id_objet pas forcément "oui" en cas de création ? #9

Open
opened 4 months ago by nicod_ · 3 comments
Collaborator

Une remarque / question suite à 3d2805109b

Il me semble que l'id_objet, en cas de création, n'est pas forcément "oui" mais peut être "new" voir autre chose, par exemple dans des plugins qui gèrent des objets éditoriaux non issus du core, qui ne suivent pas forcément les même conventions.

Est ce qu'un test intval() ne serait pas plus générique ?

Une remarque / question suite à https://git.spip.net/spip-contrib-extensions/cci/commit/3d2805109be2f3c6aa62a6714573b48fe524a268 Il me semble que l'id_objet, en cas de création, n'est pas forcément "oui" mais peut être "new" voir autre chose, par exemple dans des plugins qui gèrent des objets éditoriaux non issus du core, qui ne suivent pas forcément les même conventions. Est ce qu'un test `intval()` ne serait pas plus générique ?
Collaborator

@nicod_ c'est une bonne solution à adopter pour
cci_pipelines.php. Pour action/ajouter_imagecci.php, est-ce que l'on ne pourrait pas être plus restrictif?

@nicod_ c'est une bonne solution à adopter pour cci_pipelines.php. Pour action/ajouter_imagecci.php, est-ce que l'on ne pourrait pas être plus restrictif?
Poster
Collaborator

Pour action/ajouter_imagecci.php, est-ce que l'on ne pourrait pas être plus restrictif?

Plus restrictif ? C'est à dire ?

> Pour action/ajouter_imagecci.php, est-ce que l'on ne pourrait pas être plus restrictif? Plus restrictif ? C'est à dire ?
Collaborator

@nicod_ à bien y réfléchir, on peut le faire car on a utilisé securiser_action dans action/ajouter_imagecci.php

La correction est dans la branche master.

@nicod_ à bien y réfléchir, on peut le faire car on a utilisé securiser_action dans `action/ajouter_imagecci.php` La [correction](https://git.spip.net/spip-contrib-extensions/cci/pulls/10/files) est dans la branche master.
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 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-contrib-extensions/cci#9
Loading…
There is no content yet.