Ticket #630 (closed bug: fixed)

Opened 2 months ago

Last modified 2 months ago

Requête SQL se baladant en plein millieu de jacl2-admin

Reported by: nuks Assigned to:
Priority: high Milestone: Jelix 1.1 beta 1
Component: jelix:acl Version: trunk
Severity: minor Keywords: dao jacl2 admin sql request
Cc: Php version:
Review: review+ Hosting Provider:
Documentation needed: 0 Blocking:

Description

Dans le contrôleur build/config/jelix-dist-dev.ini de jacl2_admin, il y a une requête SQL en plein milieu du code. Vus que jacl2.db se base sur les DAO, elle n'a rien a faire là. De plus, elle n'est pas compatible avec PgSQL: 

[exception 1] (403)Erreur dans la requête (ERROR: column "nbgrp" does not exist LINE 2: GROUP BY login HAVING nbgrp < 2 ORDER BY... ^

SELECT login, count(id_aclgrp) as nbgrp FROM jacl2_user_group GROUP BY login HAVING nbgrp < 2 ORDER BY login

Vus que je suis entrain de faire l'administration de jBlog, ça urge donc je m'en charge.

Attachments

fixed_buggy_sql_request.diff (1.2 kB) - added by nuks on 06/24/08 13:54:46.
Adaptation de la requête pour pgsql
fixed_2buggy_sql_requests.diff (1.4 kB) - added by nuks on 06/24/08 14:19:49.
Un autre bug corrigé juste à coté

Change History

06/24/08 13:13:36 changed by nuks

  • owner deleted.

Il faut d'abord effectué le ticket #579 + ajouter une clause having aux DAOs

06/24/08 13:54:46 changed by nuks

  • attachment fixed_buggy_sql_request.diff added.

Adaptation de la requête pour pgsql

06/24/08 13:57:32 changed by nuks

Voilà j'ai fais un patch qui crée une requête alternative pour pgsql. Ca fonctionne avec pgsql mais je n'ai pas testé sous mysql & le reste (mais il ne devrait pas y avoir de problème vus que je n'ai pas changé la requête pour les autres bdd.

06/24/08 14:19:49 changed by nuks

  • attachment fixed_2buggy_sql_requests.diff added.

Un autre bug corrigé juste à coté

06/25/08 21:32:41 changed by laurentj

  • status changed from new to closed.
  • resolution set to fixed.
  • review set to review+.
  • component changed from module:jxacl to jelix:acl.
  • milestone set to Jelix 1.1 beta 1.

Its ok. Checked in in the trunk. Thanks for this patch.

06/25/08 21:38:55 changed by nuks

np, but i don't think it's finished 'coz it's better to use a DAO than a SQL request in a script like this. As you see ;)

Download in other formats: Comma-delimited Text Tab-delimited Text RSS Feed