hotline #98102 reset thumbnail for album records force re-downloading of…
hotline #98102 reset thumbnail for album records force re-downloading of thumbnail from third party thumbnail provider
Merge request reports
Activity
mentioned in commit d9bf1e5e
40 45 } 41 46 42 $record 43 ->setUrlImage('') 44 ->setUrlVignette('') 45 ->fetchUrlLocalVignette(); 47 Class_WebService_BibNumerique_Vignette::getInstance()->updateAlbum($album); 48 foreach($album->getErrors() as $error) 49 $this->_helper->notify($error); 46 50 47 $this->_helper->notify($this->_('Vignette réinitialisée pour "%s"', $record->getTitrePrincipal())); 48 $this->_redirectToReferer(); 51 $album->createThumbnail(); 52 $url_vignette = $album->getThumbnailUrl(); 53 54 $record->setUrlVignette($url_vignette); -
On utilise souvent UrlImage Et UrlVignette en dupliquant l'information.
Ca fait deux endroits où l'on utilise le setUrlImage et setUrlVignette avec les mêmes paramètres '' ou $url_vignette. Ne pourrait-on pas faire une fonction setUrlVignetteAndImage ? Et n'utiliser setUrlImage que dans le cas d'une vraie image (poster) Ou bien faire en sorte que getUrlImage retourne url_vignette si url_image vide et url_vignette non vide... Cela permettrait de ne gérer que les url_vignette d'un côté et les url_image uniquement si vraies images, éviterait d'avoir à stocker deux fois la même information dans deux champs différent, sans casser le comportement actuel.