adam/Piwigo
1
0 Fork 0
mirror of https://github.com/Piwigo/Piwigo.git synced 2026-10-30 21:52:00 +01:00
Code Issues 657 Packages Projects Releases Wiki Activity

[albums] Erreurs SQL lors de la suppression d'une categorie virtuelle déjà supprimée #41

New Issue
Closed
opened 2026-10-04 23:41:08 +02:00 by adam · 2 comments
adam commented 2026-10-04 23:41:08 +02:00
Owner
Copy link
Copy source

Reported by gouranga on 7 Apr 2007 10:58

Version: 1.6.2

Une suppression d'une catégorie virtuelle déjà supprimée (par un rafraichissement par exemple de la page) ou juste une suppression d'une catégorie inexistante (par changement dans la barre d'adresse) créé une vilaine erreur SQL.

Steps to reproduce: Dans administration, cliquer sur une catégorie virtuelle à supprimer et valider. Faire F5.

Additional information: L'erreur se situe dans admin/include/functions.php. La fonction delete_categories($ids) vérifie si le tableau d'identifiant n'est pas vide avant d'appeler la méthode get_subcat_ids qui elle renvoit dans le cas mentionné un tableau vide ce qui créé une requête SQL invalide. Un simple déplacement de l'appel à get_subcat_ids avant le test ou alors duplication du test devrait suffir.

Piwigo Bugtracker #673

**Reported by gouranga on 7 Apr 2007 10:58** **Version:** 1.6.2 Une suppression d&#039;une catégorie virtuelle déjà supprimée (par un rafraichissement par exemple de la page) ou juste une suppression d&#039;une catégorie inexistante (par changement dans la barre d&#039;adresse) créé une vilaine erreur SQL. **Steps to reproduce:** Dans administration, cliquer sur une catégorie virtuelle à supprimer et valider. Faire F5. **Additional information:** L&#039;erreur se situe dans admin/include/functions.php. La fonction delete_categories($ids) vérifie si le tableau d&#039;identifiant n&#039;est pas vide avant d&#039;appeler la méthode get_subcat_ids qui elle renvoit dans le cas mentionné un tableau vide ce qui créé une requête SQL invalide. Un simple déplacement de l&#039;appel à get_subcat_ids avant le test ou alors duplication du test devrait suffir. [Piwigo Bugtracker #673](http://piwigo.org/bugs/view.php?id=673) <!-- github-sync-id:97225338 -->
adam closed this issue 2026-10-04 23:41:09 +02:00
adam commented 2026-10-04 23:41:10 +02:00
Author
Owner
Copy link
Copy source

Comment posted by rvelices on 7 Apr 2007 16:51

j'ai encore mieux: essaie d'executer
\rm -rf *
c'est vraiment boggue car apres il n'y a plus rien qui marche... :-) desole je n'ai pas pu m'empecher :-)

**Comment posted by rvelices on 7 Apr 2007 16:51** j&#039;ai encore mieux: essaie d&#039;executer \rm -rf \* c&#039;est vraiment boggue car apres il n&#039;y a plus rien qui marche... :-) desole je n&#039;ai pas pu m&#039;empecher :-) <!-- github-comment-id:124889862 -->
adam commented 2026-10-04 23:41:10 +02:00
Author
Owner
Copy link
Copy source

Comment posted by mathiasm on 29 Jan 2008 13:42

>Une suppression d'une catégorie virtuelle déjà supprimée (par un >rafraichissement par exemple de la page) ou juste une suppression d'une >catégorie inexistante (par changement dans la barre d'adresse) créé une >vilaine erreur SQL.

Logique, mais ne devrait-on pas avoir une page de redirection qui nettoie le $_POST ?

**Comment posted by mathiasm on 29 Jan 2008 13:42** &gt;Une suppression d&#039;une catégorie virtuelle déjà supprimée (par un &gt;rafraichissement par exemple de la page) ou juste une suppression d&#039;une &gt;catégorie inexistante (par changement dans la barre d&#039;adresse) créé une &gt;vilaine erreur SQL. Logique, mais ne devrait-on pas avoir une page de redirection qui nettoie le $_POST ? <!-- github-comment-id:124889882 -->
Sign in to join this conversation.
No Branch/Tag Specified
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
adam (Adam Wojcieszonek)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: adam/Piwigo#41