Suppression du formulaire `FORMULAIRE_CHANGER_FICHIER_DOCUMENT` qui n'est plus utilisé par SPIP (la fonctionnalité se trouve directement dans `FORMULAIRE_EDITER_DOCUMENT`) #4899

Merged
b_b merged 2 commits from maieul/medias:code_deprecie into master 7 months ago
maieul commented 7 months ago
Collaborator

En voulant débuguer #4872, j'ai vu qu'on avait conservé le code de FORMULAIRE_CHANGER_FICHIER_DOCUMENT, même si le plugin ne l'utilise plus (la fonctionnalité est désormais directement dans FORMULAIRE_EDITER_DOCUMENT). C'était piégeux car du coup j'ai cherché à debug au mauvais endroit.

Cette PR propose donc la suppression, reste à décider sur quelle version de SPIP, pour que ce soit connu pour les éventuelles personnes qui l'utilisaient directement.

En voulant débuguer #4872, j'ai vu qu'on avait conservé le code de `FORMULAIRE_CHANGER_FICHIER_DOCUMENT`, même si le plugin ne l'utilise plus (la fonctionnalité est désormais directement dans `FORMULAIRE_EDITER_DOCUMENT`). C'était piégeux car du coup j'ai cherché à debug au mauvais endroit. Cette PR propose donc la suppression, reste à décider sur quelle version de SPIP, pour que ce soit connu pour les éventuelles personnes qui l'utilisaient directement.
maieul added 1 commit 7 months ago
rastapopoulos approved these changes 7 months ago
b_b approved these changes 7 months ago
b_b left a comment
Owner

Pour la 4.2 uniquement amha.

Pour la 4.2 uniquement amha.
Poster
Collaborator

Je pense aussi 4.2 seulement.

Je pense aussi 4.2 seulement.
marcimat approved these changes 7 months ago
b_b added 1 commit 7 months ago
b_b merged commit 2c99d54592 into master 7 months ago
b_b deleted branch code_deprecie 7 months ago

Reviewers

rastapopoulos approved these changes 7 months ago
b_b approved these changes 7 months ago
marcimat approved these changes 7 months ago
The pull request has been merged as 2c99d54592.
Sign in to join this conversation.
No Milestone
No Assignees
4 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.