en regardant un peu dans le code du plugin album 3, j’aurais bien ajouté 2 variables _ALBUMS_AFFICHER_TITRE et _ALBUMS_AFFICHER_DESCRIPTIF pour pouvoir forcer l’affichage des titre et descriptif sur tout le site via mes_options.php.
Voici les modifs que je propose : (L80 et 82) Qu’en dites vous ? J’ai également repéré 2 ou 3 pétouilles, notamment au niveau du titre personnalisé qui n’a pas l’air de fonctionner et la variable _ALBUMS_TAILLE_PREVIEW qui n’a pas l’air prise en compte à partir du moment où la taille des miniatures dans les options avancées est configurée… jean marie
Ok pour prendre en compte ces 2 variables dans les modèles.
En parlant de constantes, j'avais ajouté 13 constantes pour définir les valeurs par défaut des modèles dans albums.options.php : Connexion · GitLab
Mais il me semble que c'est pas top pour les perfs de déclarer trop de constantes, en fait l'idée c'était de pouvoir prendre connaissance de toutes les constantes utilisables dans le modèle en jetant un coup d'oeil au fichier php. Mais c'était pas une très bonne idée je pense.
Je crois que je vais retirer tout ça, et les référencer dans la doc tout simplement.
Merci aussi pour le retour sur les 2 constantes qui ne sont pas prises en compte correctement, je vais regarder ça.
tcharlss
Le 12/01/2016 11:51, Jean Marie Grall a écrit :
Salut,
en regardant un peu dans le code du plugin album 3, j'aurais bien ajouté 2 variables _ALBUMS_AFFICHER_TITRE et _ALBUMS_AFFICHER_DESCRIPTIF pour pouvoir forcer l'affichage des titre et descriptif sur tout le site via mes_options.php.
J'ai également repéré 2 ou 3 pétouilles, notamment au niveau du titre personnalisé qui n'a pas l'air de fonctionner et la variable _ALBUMS_TAILLE_PREVIEW qui n'a pas l'air prise en compte à partir du moment où la taille des miniatures dans les options avancées est configurée...
jean marie
--
Cousu Main
*Jean Marie Grall*
Votre site internet sur mesure
06.10.07.76.29
www.cousumain.info <http://www.cousumain.info>
jeanmarie@cousumain.info <mailto:jeanmarie@cousumain.info>
En parlant de constantes, j'avais ajouté 13 constantes pour définir les valeurs par défaut des modèles dans
albums.options.php : Connexion · GitLab
Mais il me semble que c'est pas top pour les perfs de déclarer trop de constantes,
rohhh l'impact des constantes sur les perfs
est surement bien moindre que l'impact d'une mauvaise lisibilité.
JL
en fait l'idée c'était de pouvoir
prendre connaissance de toutes les constantes utilisables dans le modèle en jetant un coup d'oeil au fichier php. Mais
c'était pas une très bonne idée je pense.
Je crois que je vais retirer tout ça, et les référencer dans la doc tout simplement.
Merci aussi pour le retour sur les 2 constantes qui ne sont pas prises en compte correctement, je vais regarder ça.
tcharlss
Le 12/01/2016 11:51, Jean Marie Grall a écrit :
Salut,
en regardant un peu dans le code du plugin album 3, j'aurais bien ajouté 2 variables _ALBUMS_AFFICHER_TITRE et
_ALBUMS_AFFICHER_DESCRIPTIF pour pouvoir forcer l'affichage des titre et descriptif sur tout le site via mes_options.php.
J'ai également repéré 2 ou 3 pétouilles, notamment au niveau du titre personnalisé qui n'a pas l'air de fonctionner et
la variable _ALBUMS_TAILLE_PREVIEW qui n'a pas l'air prise en compte à partir du moment où la taille des miniatures
dans les options avancées est configurée...
jean marie
--
Cousu Main
*Jean Marie Grall*
Votre site internet sur mesure
06.10.07.76.29
www.cousumain.info <http://www.cousumain.info>
jeanmarie@cousumain.info <mailto:jeanmarie@cousumain.info>
c’est fait : Le problème avec le titre personnalisé est que titre doit être égale à oui ou true pour déclencher l’affichage, donc si on passe titre=Mon titre perso, on n’affiche pas le titre. Il faudrait sans doute mettre en place une variable titre_perso. Pour ce qui est de _ALBUMS_TAILLE_PREVIEW, si une taille est configurée dans le menu Fonctions avancées, la variable n’est pas prise en compte car elle est dans un |sinon{}. jean marie