Développement #101084
Utiliser combo_cell_loader au lieur de combo_load_cell
0%
Description
Si la fonction évolue dans combo il faut qu'elle bouge aussi dans les themes.
Ca ne devrait pas être nécessaire après ça
Related issues
History
Updated by Gael Pasgrimaud 2 months ago
- Related to Développement #101014: Cellules fiche, pouvoir passer des filtres en url added
Updated by Robot Gitea 2 months ago
- Status changed from Nouveau to En cours
Gael Pasgrimaud (gpasgrimaud) a ouvert une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/publik-base-theme/pulls/799
- Titre : WIP: misc: take care of combo_load_cell new signature (#101084)
- Modifications : https://git.entrouvert.org/entrouvert/publik-base-theme/pulls/799/files
Updated by Frédéric Péters 2 months ago
Ca ne devrait pas être nécessaire après ça
Le truc serait alors d'introduire une nouvelle API, et garder combo_load_cell() comme une fonction qui appellerait combo_load_cell_v2({"element": elem}).
(idéalement avec un meilleur nom)
Alternativement, pour ce cas précis, modifier combo_load_cell pour commencer par regarder le type du premier paramètre, et gérer les anciens appelants.
Updated by Gael Pasgrimaud 2 months ago
Je vois l'idée de garder la compat pour du code qui serait pas à nous. Je vais voir
Updated by Gael Pasgrimaud 2 months ago
- Subject changed from Prendre en compte la nouvelle api de combo_load_cell to Utiliser combo_cell_loader au lieur de combo_load_cell
combo_load_cell est marqué dépréciéé
pas trouvé telement mieux comme nom que combo_cell_loader
je garde tout de même cette PR car ça n'a pas d'intérêt de continuer d'utiliser l'ancienne fonction