Type of issue (keep only one) / Type de demande (ne garder qu'une ligne)
New feature / Nouvelle fonctionnalité
Description
Actuellement quand un champ ne peut être édité, seules les données associées au champ sont nettoyées par EntryManager->assignRestrictedFields:
Si le champ utilise d'autres données dans le formulaire qui ont un nom différent de $field->getPropertyName(), celles-ci ne sont pas nettoyées.
Actuellement, le seul moyen est de détecter dans $field->formatValuesBeforeSave() que le champ ne pouvait pas être édité et alors de ne pas tenir compte de ces données supplémentaires et les nettoyer.
La proposition est de ne plus faire ce test dans $field->formatValuesBeforeSave() mais de créer une nouvelle méthode pour BazarField exemple : cleanNotEditableValues($entry): array qui pour chaque champ ferait le nettoyage des champs concernés.
Un temps de concertation orale ou en présence entre développeurs est souhaité.
VeveQNV
commented
2 years ago
Type of issue (keep only one) / Type de demande (ne garder qu'une ligne) New feature / Nouvelle fonctionnalité
Description Actuellement quand un champ ne peut être édité, seules les données associées au champ sont nettoyées par
EntryManager->assignRestrictedFields:Si le champ utilise d'autres données dans le formulaire qui ont un nom différent de
$field->getPropertyName(), celles-ci ne sont pas nettoyées. Actuellement, le seul moyen est de détecter dans$field->formatValuesBeforeSave()que le champ ne pouvait pas être édité et alors de ne pas tenir compte de ces données supplémentaires et les nettoyer.La proposition est de ne plus faire ce test dans
$field->formatValuesBeforeSave()mais de créer une nouvelle méthode pourBazarFieldexemple :cleanNotEditableValues($entry): arrayqui pour chaque champ ferait le nettoyage des champs concernés.Un temps de concertation orale ou en présence entre développeurs est souhaité.
La présente issue permet de recueillir les écrits en attendant cette concertation. Elle fait suite à des échanges situés dans https://github.com/YesWiki/yeswiki/pull/953
Additionnal informations / Informations complémentaires
future doryphore 4.4ou plus tard