Permissions sur user importé

Bug #659286 reported by nozof
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Bilboplanet
Fix Committed
High
Gregoire de Hemptinne

Bug Description

PHP version : 5.2.6-1+lenny9
MySQL version : 5.0.51a
Apache/2.2.9

BilboPlanet version : 1.0 Unstab

J'ai importé mon contenu depuis la 0.3.2.

Lorsque je veux changer les permissions pour un utilisateur, je clique sur la liste déroulante, fais mon choix et ça revient sur "normal user" et un message s'affiche en haut de la page : User xxxx xxxx is now know as.
Une entrée est créé dans la table permission dans la base de données

Si je crée un nouvel user ça fonctionne?

Revision history for this message
nozof (nozof) wrote :

En fait si je crée un user avec une espace dans le champ: user id, ça coince .

Pour les users importés dont le nom ne comporte pas d'espace ça fonctionne.

Beaucoup de mes anciens user ont des espace dans leur nom.

Changed in bilboplanet:
assignee: nobody → theClimber (greg-theclimber)
importance: Undecided → High
status: New → Confirmed
Revision history for this message
Gregoire de Hemptinne (greg-theclimber) wrote :

J'ai commité une correction. En gros, la solution est de ne plus permettre de user_id contenant un espace. On peut seulement utiliser des espaces dans "fullname" : ce qui a plus de sens ... car c'est ça le "vrai" nom !

J'ai donc modifié l'algo d'import pour qu'il transforme les espaces en "_" pour le user_id. J'ai aussi modifié le truc dans la gestion des utilisateurs.

Merci d'avoir reporté le bug. N'hésite pas à retester pour vérifier que ça marche bien maintenant.

PS: pour le problème des langues, je reviendrais dessus plus tard

Changed in bilboplanet:
status: Confirmed → Fix Committed
Revision history for this message
nozof (nozof) wrote :

J'ai recuperé la derniere version 15h25

j'exporte avec ou sans la modification https://bugs.launchpad.net/bilboplanet/+bug/659164/comments/16

Lors de l'import j'ai cette erreur :

Fatal error: Uncaught exception 'Exception' with message 'Column 'site_id' cannot be null (1048)' in /home/www/bilboplanet/inc/clearbricks/dblayer/class.mysql.php:105 Stack trace: #0 /home/www/bilboplanet/inc/clearbricks/dblayer/class.mysql.php(115): mysqlConnection->db_query(Resource id #15, 'INSERT INTO `bp...') #1 /home/www/bilboplanet/inc/clearbricks/dblayer/dblayer.php(337): mysqlConnection->db_exec(Resource id #15, 'INSERT INTO `bp...') #2 /home/www/bilboplanet/inc/clearbricks/dblayer/class.cursor.php(138): dbLayer->execute('INSERT INTO `bp...') #3 /home/www/bilboplanet/admin/manage-database.php(161): cursor->insert() #4 {main} thrown in /home/www/bilboplanet/inc/clearbricks/dblayer/class.mysql.php on line 105

les users sont bien importés le problème de l'espace est résolu
seulement 2/22 flux importés
seulement 10/900 importés

Revision history for this message
Gregoire de Hemptinne (greg-theclimber) wrote :

Le problème est plus conséquent que je ne pensais.

Si j'ai bien compris le bug, c'est aussi lié à l'espace dans les noms des utilisateurs. J'ai commité un fix (je met le fichier de correction en pièce jointe, comme ça tu dois pas spécialement re-télécharger tout le code)

J'espère que ça ira maintenant !

Changed in bilboplanet:
status: Fix Committed → In Progress
Revision history for this message
Gregoire de Hemptinne (greg-theclimber) wrote :

Oups, j'ai envoyé le mauvais fichier, voici le bon

Revision history for this message
nozof (nozof) wrote :

Ça marche !

Changed in bilboplanet:
status: In Progress → Fix Committed
Revision history for this message
nozof (nozof) wrote :

J'ai retéléchargé le code et apparemment le fichier n'est pas le bon.

Je l'ai remplacé par celui du post #5 et la ça marche

Revision history for this message
Gregoire de Hemptinne (greg-theclimber) wrote :

Ca c'est pas normal. T'es sur de ce que t'avances. Qu'est-ce qui ne marche pas plus exactement? Peux-tu expliquer quel test tu fais?

Normalement les anciens user avec un espace dans leur noms ont un user_id avec l'espace qui a été remplacé par un underscore "_". Ce qui est sensé régler le problème.

Il est vrai, ceci dit, qu'il est possible que les accents et les caractères spéciaux ne soient pas gérés. Si c'est le cas, alors fais-moi signe.

Revision history for this message
nozof (nozof) wrote :

 Ça fait comme au post #3

Si je change le fichier manage-database du post #5 c'est ok

Je pense que c'est le fichier du post #4 qui à été commité.

Revision history for this message
Gregoire de Hemptinne (greg-theclimber) wrote :

Bizarre, pourtant j'ai vérifié et les modifs ont été commitées...

Revision history for this message
nozof (nozof) wrote :

Lors del'import de ma dase 0.3.2 vers beta1, j'ai ce message:

Fatal error: Uncaught exception 'Exception' with message 'Column 'feed_status' cannot be null (1048)' in /home/www/bilboplanet/inc/clearbricks/dblayer/class.mysql.php:105 Stack trace: #0 /home/www/bilboplanet/inc/clearbricks/dblayer/class.mysql.php(115): mysqlConnection->db_query(Resource id #15, 'INSERT INTO `bp...') #1 /home/www/bilboplanet/inc/clearbricks/dblayer/dblayer.php(337): mysqlConnection->db_exec(Resource id #15, 'INSERT INTO `bp...') #2 /home/www/bilboplanet/inc/clearbricks/dblayer/class.cursor.php(138): dbLayer->execute('INSERT INTO `bp...') #3 /home/www/bilboplanet/admin/manage-database.php(179): cursor->insert() #4 {main} thrown in /home/www/bilboplanet/inc/clearbricks/dblayer/class.mysql.php on line 105

Lorsque je remplace le fichier manage_database. php de la beta1 par celui qui se trouve au post #5 l'import se passe correctement.

J'ai fait un diff des fichiers.

Revision history for this message
Gregoire de Hemptinne (greg-theclimber) wrote :

Bon, je vais regarder ça de plus près

Changed in bilboplanet:
status: Fix Committed → Confirmed
Revision history for this message
Gregoire de Hemptinne (greg-theclimber) wrote :

Ok, j'ai pigé le bug. En fait j'ai fait ça dépend des colonnes de la table. Donc ça dépend des gens. Mais normalement si tu avais exporté depuis la toute dernière 0.3.2 ça aurait fonctionné... mais visiblement tu avais (à l'époque) fait une migration de 0.3.1 vers 0.3.2 donc il te manquait une colonne dans ta table "flux" (même si ça ne t'a pas marqué car cette colonne n'était pas utilisée).

Donc voilà, j'en ai tenu compte dans le code et maintenant ça devrait marcher (j'espère)

Revision history for this message
nozof (nozof) wrote :

9a marche !

Changed in bilboplanet:
status: Confirmed → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.