valeur par defaut de la balise #CONFIG ne fonctionne pas #3999

Closed
opened 5 years ago by tofulm · 4 comments
tofulm commented 5 years ago
Collaborator

Dans la "doc":https://www.spip.net/fr_article4335.html de la balise #CONFIG : ??Il est possible de fournir une valeur par défaut si le contenu de la meta est vide.??

Cette valeur par défaut fonctionne s'il n'y a pas d'entrée correspondant au nom de la valeur dans la table spip_meta, par contre si le nom existe et que la valeur est vide, alors la valeur par défaut n'est pas utilisée

Dans la "doc":https://www.spip.net/fr_article4335.html de la balise #CONFIG : ??Il est possible de fournir une valeur par défaut si le contenu de la meta est vide.?? Cette valeur par défaut fonctionne s'il n'y a pas d'entrée correspondant au nom de la valeur dans la table spip_meta, par contre si le nom existe et que la valeur est vide, alors la valeur par défaut n'est pas utilisée
Owner

Salut

Le comportement me semble logique, on peut avoir une meta dont la
valeur est vide. J'opterai pour corriger la documentation.
"Il est possible de fournir une valeur par défaut si la meta n'existe pas."

Salut Le comportement me semble logique, on peut avoir une meta dont la valeur est vide. J'opterai pour corriger la documentation. "Il est possible de fournir une valeur par défaut si la meta n'existe pas."
Poster
Collaborator

si dans https://core.spip.net/projects/spip/repository/entry/branches/spip-3.1/ecrire/inc/config.php#L110
on remplace par :

		$r = (isset($GLOBALS['meta'][$cfg]) and $GLOBALS['meta'][$cfg]) ?

ca fonctionne

si dans https://core.spip.net/projects/spip/repository/entry/branches/spip-3.1/ecrire/inc/config.php#L110 on remplace par : <pre> $r = (isset($GLOBALS['meta'][$cfg]) and $GLOBALS['meta'][$cfg]) ? </pre> ca fonctionne
Owner

Les valeurs 0 (zéro) peuvent être une valeur, tout comme une chaine vide ''.
Je pense qu’il vaut mieux éviter d’y toucher.
À la limite on pourrait considérer null et dans ce cas renvoyer la valeur par défaut.
Mais tous les autres cas ne me semblent à ne pas changer.

Les valeurs 0 (zéro) peuvent être une valeur, tout comme une chaine vide ''. Je pense qu’il vaut mieux éviter d’y toucher. À la limite on pourrait considérer `null` et dans ce cas renvoyer la valeur par défaut. Mais tous les autres cas ne me semblent à ne pas changer.
b_b commented 5 years ago
Owner

Même avis que marcimat, amha cela pourrait générer des effets de bord de modifier ce comportement, on ferme :)
Statut changé à Fermé

Même avis que marcimat, amha cela pourrait générer des effets de bord de modifier ce comportement, on ferme :) **Statut changé à Fermé**
Sign in to join this conversation.
No Milestone
No project
No Assignees
4 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.