Ghislain Loas (04e34a6e) at 27 Mar 17:44
wip
Ghislain Loas (55b278a3) at 27 Mar 11:30
hotline#194779: fix put patron route erase informations in Koha
... and 64 more commits
Rajouter du typage
À supprimer
- correctif #197682 : Administration : correction du sytème de mise en cache de l'explorateur de fichiers qui empéchait la création de dossier.
Bon, il n'y a pas vraiment de test qui assert le changement si ce n'est que ça passe toujours. On peut peut-être mettre une public sur le record pour donner l'état de la variable et vérifier qu'elle contient bien l'id du model et le model en cache.
Voir s'il faut rajouter aussi dans TearDown.php
Ajouter et utiliser $id = $this->_model->getId();
public static function resetUrlsByModelCache(): void
À changer :
- correctif #197905 : Performance : ajout d'un cache sur l'affichage des documents pour améliorer les performances d'affichage du portail.
protected static $_urls_by_model_cache = [];
$models = [];
foreach ($rows as $row) {
if ( ! ($url = $this->_url($this->_unserializeParams($row->getCommParams()))))
continue;
$models[$url] ??= 0;
$models[$url] ++;
}
return $models;
Et on peut supprimer la classe UrlCount.
Hum ça semble dupliquer des mécanismes qui sont dans Class_Url. Même si malheureusement Class_Url ne fournit pas la méthode souhaité "out of box". Voir Class_Url::secureIfNeeded(string $url) si on peut réutiliser sans le Class_Url::_isSecure()
$url = trim($url, " \n\r\t\v\x00\/");
cf https://www.php.net/manual/fr/function.trim.php on peut passer en paramètre les caractères.
- correctif #198520 : SIGB Nanook : Correction du mode Nanook unique.
En regardant le fichier je trouve que des appels à _dateNoveltyFrom avec $default = '' ou pas passer du coup le paramètre me semble inutile.
Aussi, on peut remplacer 2000-01-01 par une const.
return $this->_addDefaultNovelty($this->getPMBExemplaire($champ,
$ex,
$ret,
$code_barres,
$cotes), $novelty_date);
});
Attention à la variable $champs_nouveaute qui est à la fois dans le foreach et qui se voit ajouter des éléments.
Est-ce que ce profil de données doit remplacer Class_IntProfilDonnées::forPMB ?