spip
/
medias
16
1
Fork 7
Code Issues 31 Pull Requests 2 Releases Wiki Activity
Labels Milestones

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
Conversation 1 Commits 2 Files Changed 3
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.
maieul commented 7 months ago
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.
Reviewers
Request review
No reviewers
rastapopoulos
b_b
marcimat
Labels
Apply labels
Clear labels
amélioration
Amélioration, nouvelle fonctionnalité bug
Ca ne fonctionne pas documentation
Documentation souhaite doublon
Ce ticket est un doublon invalide
Ticket invalide refusé
Ignoré, c'est comme Ca...
No Label amélioration bug documentation doublon invalide refusé
Milestone
Set milestone
Clear milestone
No items
No Milestone
Assignees
Assign users
Clear assignees
No Assignees
4 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch 'maieul/medias:code_deprecie'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes