filtre trim #5597

Open
opened 2 weeks ago by nicod_ · 1 comments
nicod_ commented 2 weeks ago
Owner

En référence à spip-contrib-extensions/fabrique#30

Ajout d'un filtre trim pour éviter des warnings Deprecated sur un texte null dans les squelettes, où on veut dans tous les cas une chaîne.

En référence à https://git.spip.net/spip-contrib-extensions/fabrique/issues/30 Ajout d'un filtre trim pour éviter des warnings Deprecated sur un texte null dans les squelettes, où on veut dans tous les cas une chaîne.
nicod_ commented 2 weeks ago
Poster
Owner

J'allais faire une PR en ce sens, mais les Deprecated sur trim(null) ont été corrigés entre temps dans la fabrique :
54b8d839bc
ainsi que sur strtoupper :
d60960ffa6

Quelle est la bonne approche officielle du coup : corriger au niveau des squelettes en testant tout le temps, ou mettre ça "sous le tapis" avec un filtre, en considérant que dans un squelette on ne manipule jamais des null de toute façon ?

J'allais faire une PR en ce sens, mais les Deprecated sur `trim(null)` ont été corrigés entre temps dans la fabrique : https://git.spip.net/spip-contrib-extensions/fabrique/commit/54b8d839bcd59a5f91a52d4ac4e40b6f27fdd2b2 ainsi que sur strtoupper : https://git.spip.net/spip-contrib-extensions/fabrique/commit/d60960ffa6d22c2d105b26e01a30c1f7cdc1df8b Quelle est la bonne approche officielle du coup : corriger au niveau des squelettes en testant tout le temps, ou mettre ça "sous le tapis" avec un filtre, en considérant que dans un squelette on ne manipule jamais des `null` de toute façon ?
b_b added this to the 4.2 milestone 2 weeks ago
b_b added the
filtres et balises
label 2 weeks ago
Sign in to join this conversation.
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: spip/spip#5597
Loading…
There is no content yet.