Options pour masquer certains modèles pas prise en compte dans la barre d'édition #13

Closed
opened 8 months ago by tcharlss · 3 comments
Owner

Insérer modèles 2.0.1 / SPIP 4.1

Dans la configuration du plugin il y a une option « Ne pas proposer les modèles suivants »

Elle est bien prise en compte dans le formulaire latéral :

Mais pas dans la barre d'édition :

Insérer modèles 2.0.1 / SPIP 4.1 Dans la configuration du plugin il y a une option « Ne pas proposer les modèles suivants » Elle est bien prise en compte dans le formulaire latéral : ![](https://git.spip.net/attachments/e6fc7ef5-eeee-4330-95dc-e04da57da0e1) Mais pas dans la barre d'édition : ![](https://git.spip.net/attachments/6c65a3fa-952e-4297-89ae-678d92e42caa)
Poster
Owner

Donc on a une fonction inserer_modeles_lister_formulaires_modeles() qui donne la liste des modèles.

Le formulaire l'utilise puis retire ceux désactivés dans la config : https://git.spip.net/spip-contrib-extensions/inserer_modeles/src/branch/master/formulaires/inserer_modeles.php#L18-L23

$modeles_dispo = inserer_modeles_lister_formulaires_modeles();
include_spip('inc/config');
$config_ignorer = lire_config('inserer_modeles/ignorer_modeles') ?? [];
foreach ($config_ignorer as $ignorer) {
	unset($modeles_dispo[$ignorer]);
}

La barre d'édition fait de même, sauf qu'elle ne retire pas ceux désactivés : https://git.spip.net/spip-contrib-extensions/inserer_modeles/src/branch/master/inserer_modeles_pipelines.php#L39

À priori il faudrait que ça soit la fonction de listing qui fasse elle-même ce tri, soit par défaut, soit en option.

Donc on a une fonction [inserer_modeles_lister_formulaires_modeles()](https://git.spip.net/spip-contrib-extensions/inserer_modeles/src/branch/master/inc/inserer_modeles.php#L12) qui donne la liste des modèles. Le formulaire l'utilise puis retire ceux désactivés dans la config : https://git.spip.net/spip-contrib-extensions/inserer_modeles/src/branch/master/formulaires/inserer_modeles.php#L18-L23 ```php $modeles_dispo = inserer_modeles_lister_formulaires_modeles(); include_spip('inc/config'); $config_ignorer = lire_config('inserer_modeles/ignorer_modeles') ?? []; foreach ($config_ignorer as $ignorer) { unset($modeles_dispo[$ignorer]); } ``` La barre d'édition fait de même, sauf qu'elle ne retire pas ceux désactivés : https://git.spip.net/spip-contrib-extensions/inserer_modeles/src/branch/master/inserer_modeles_pipelines.php#L39 À priori il faudrait que ça soit la fonction de listing qui fasse elle-même ce tri, soit par défaut, soit en option.

+1 pour dans la fonction directement avec une option (à voir lequel par défaut, tout ou filtré)

+1 pour dans la fonction directement avec une option (à voir lequel par défaut, tout ou filtré)
Poster
Owner

PR !14

J'ai gardé le défaut tel quel, des fois que la fonction soit utilisée ailleurs dans la nature.

Le nouveau param est un booléen pour exclure les modèles désactivés.

J'ai envisagé faire à la place un tableau d'options des fois que dans le futur il y ait besoin d'autres filtres, mais bon, j'en vois pas vraiment d'autre pour l'instant.

PR !14 J'ai gardé le défaut tel quel, des fois que la fonction soit utilisée ailleurs dans la nature. Le nouveau param est un booléen pour exclure les modèles désactivés. J'ai envisagé faire à la place un tableau d'options des fois que dans le futur il y ait besoin d'autres filtres, mais bon, j'en vois pas vraiment d'autre pour l'instant.
maieul closed this issue 4 months ago
Sign in to join this conversation.
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.