[FINNA-2579] & [FINNA-2875] Add file size to LIDO resource and fix resource play buttons for sr #3118
Conversation
…ms with resource links and player.
| * | ||
| * @return array | ||
| */ | ||
| protected function getModel( | ||
| string $url, | ||
| string $format, | ||
| string $type, | ||
| \SimpleXmlElement $measurements = null, |
There was a problem hiding this comment.
Tämän tyypin pitäisi olla ?\SimpleXmlElement
| * | ||
| * @return array | ||
| */ | ||
| protected function getAudio( | ||
| string $url, | ||
| string $format, | ||
| string $description | ||
| string $description, | ||
| \SimpleXmlElement $measurements = null |
There was a problem hiding this comment.
Tämän tyypin pitäisi olla ?\SimpleXmlElement
| @@ -923,6 +951,9 @@ protected function getAudio( | |||
| 'codec' => $format, | |||
| 'type' => 'audio', | |||
| 'embed' => 'audio', | |||
| 'data' => $this->formatResourceMeasurements( | |||
There was a problem hiding this comment.
Ei voi kutsua formatResourceMeasurements-metodia varmistamatta, ettei $measurements ole null.
| @@ -955,6 +988,9 @@ protected function getVideo( | |||
| 'url' => $url, | |||
| 'embed' => 'iframe', | |||
| 'format' => $format, | |||
| 'data' => $this->formatResourceMeasurements( | |||
There was a problem hiding this comment.
Ei voi kutsua formatResourceMeasurements-metodia varmistamatta, ettei $measurements ole null.
| @@ -966,6 +1002,9 @@ protected function getVideo( | |||
| 'src' => $url, | |||
| 'type' => $mediaType, | |||
| ], | |||
| 'data' => $this->formatResourceMeasurements( | |||
There was a problem hiding this comment.
Ei voi kutsua formatResourceMeasurements-metodia varmistamatta, ettei $measurements ole null.
|
@siiriylonen Saisiko saamen käännöksen myös? |
| * @param string $url Model url | ||
| * @param string $format Model format | ||
| * @param string $type Model type | ||
| * @param \SimpleXmlElement $measurements Measurements SimpleXmlElement |
There was a problem hiding this comment.
Kysymysmerkki tähänkin tyypin eteen. Jälkimmäinen SimpleXmlElement on turha – sehän on jo parametrin tyypissä.
| @@ -1,8 +1,13 @@ | |||
| <!-- START of: finna - RecordDriver/DefaultRecord/audio-information.phtml --> | |||
| <?php | |||
| if ($fileSize = $url['data']['size'] ?? '') { | |||
| $fileSize = !empty($fileSize['value']) && $fileSize['unit'] === 'byte' ? '(' . $this->fileSize(preg_replace('/[^0-9]/', '', $fileSize['value'])) . ')' : ''; | |||
There was a problem hiding this comment.
Miksi yksikön pitää olla byte? Jos tietueessa on koko vaikka kilotavuina, niin jää näyttämättä, eikö?
| @@ -1,7 +1,7 @@ | |||
| <!-- START of: finna - RecordDriver/DefaultRecord/audio-information.phtml --> | |||
| <?php | |||
| if ($fileSize = $url['data']['size'] ?? '') { | |||
| $fileSize = !empty($fileSize['value']) && $fileSize['unit'] === 'byte' ? '(' . $this->fileSize(preg_replace('/[^0-9]/', '', $fileSize['value'])) . ')' : ''; | |||
| $fileSize = !empty($fileSize['value']) ? '(' . $this->fileSize(preg_replace('/[^0-9]/', '', $fileSize['value'])) . ')' : ''; | |||
There was a problem hiding this comment.
Eihän tämä kuitenkaan näin voi mennä. Jos unit on jotain muuta kuin byte, niin ei fileSize anna sille oikeaa arvoa. Toisin sanoen pitää joko konvertoida tavuiksi SolrLido:ssa, välittää myös yksikkö fileSize:lle tai näyttää koko jotenkin vaihtoehtoisesti.
There was a problem hiding this comment.
Lisäksi pitää huomioida, että muu kuin tavu voi olla desimaaliluku, vaikka "3.5", jolloin ei voi suoraan vain poistaa ei-numeroita stringistä. Muuten olisi kyllä parempi tehdä siivous SolrLidon puolella, jotta sitä ei tarvitse erikseen tehdä monessa paikassa templaateissa.
…x-focus-for-resource-btn
No description provided.