MantisBT - Piwigo
View Issue Details
0000160Piwigousers & groupspublic2005.09.22 14:252005.10.05 19:47
noiry 
plg 
normalmajoralways
closedfixed 
1.5.0RC1 
1.5.0RC2 
IE
Apache 1.3.x
0000160: en vrac : pb dans les requetes si gestion des users dans tables exterieures (phpBB)
!!!! 2 BUGS DIFFERENTS !!!!

admin/user_list.php :

ligne 64 :
----------
SELECT id
  FROM '.USERS_TABLE.'
  WHERE id != '.$conf['guest_id'].'
;';

remplacé par :
--------------
SELECT '.$conf['user_fields']['id'].' AS id
  FROM '.USERS_TABLE.'
  WHERE '.$conf['user_fields']['id'].' != '.$conf['guest_id'].'
;';

!!!!!!!!!!!!!!!

Ensuite, les updates massifs plantent :

admin/include/functions.php ligne 708 :

CREATE TABLE '.$temporary_tablename.'
(
'.implode(",\n", $columns).'
PRIMARY KEY(id)
)
;'

Forcement, quand id c'est user_id ca deconne... J'ai retiré la clé. Ca semble marcher... Mais je ne suis pas sur que ce soit très propre. Si vous avez une bonne solution...
No tags attached.
Issue History
2005.09.22 14:25noiryNew Issue
2005.09.22 14:25noirybrowser => IE
2005.09.22 14:25noiryWeb server => Apache 1.3.x
2005.09.28 21:44plgCategorydatabase => user or group management
2005.09.28 21:44plgProduct Version => 1.5.0RC1
2005.09.29 22:31plgNote Added: 0000278
2005.09.29 22:31plgAssigned To => plg
2005.09.29 22:31plgStatusnew => assigned
2005.10.05 19:47plgNote Added: 0000283
2005.10.05 19:47plgStatusassigned => closed
2005.10.05 19:47plgResolutionopen => fixed
2005.10.05 19:47plgFixed in Version => 1.5.0RC2

Notes
(0000278)
plg   
2005.09.29 22:31   
J'ai corrigé le premier bug en reprenant complètement la gestion de la liste des utilisateurs au niveau du code PHP. En effet, le bout de code dans lequel tu as détecté un bug est mauvais: si je choisis "tout" comme cible de mes modifs (suppresions, associations aux groupes, status, propriétés d'affichage), on s'attend à n'affecter que la liste filtrée et pas la liste complète.

Le code est bien meilleur maintenant car centralisé dans une fonction unique get_filtered_user_list.

J'aurai bien clos le bug, mais tu en as soumis 2 en même temps, donc je fermerai plus tard, le temps de corriger le second bug.
(0000283)
plg   
2005.10.05 19:47   
Second bug corrigé. Puisque les colonnes faisant parti de la clef sont fourni dans $dbfields['primary'], autant les utiliser).

https://mail.gna.org/public/phpwebgallery-cvs/2005-10/msg00000.html [^]