developer.jelix.org is not used any more and exists only for history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#630 closed bug (fixed)

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

Reported by: nuks Owned by:
Priority: high Milestone: Jelix 1.1 beta 1
Component: jelix:acl Version: trunk
Severity: minor Keywords: dao jacl2 admin sql request
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

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 (2)

fixed_buggy_sql_request.diff​ (1.2 KB) - added by nuks 12 years ago.
Adaptation de la requête pour pgsql
fixed_2buggy_sql_requests.diff​ (1.4 KB) - added by nuks 12 years ago.
Un autre bug corrigé juste à coté

Download all attachments as: .zip

Change History (6)

comment:1 Changed 12 years ago by nuks

  • Owner nuKs deleted

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

Changed 12 years ago by nuks

Adaptation de la requête pour pgsql

comment:2 Changed 12 years ago 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.

Changed 12 years ago by nuks

Un autre bug corrigé juste à coté

comment:3 Changed 12 years ago by laurentj

  • Component changed from module:jxacl to jelix:acl
  • Milestone set to Jelix 1.1 beta 1
  • Resolution set to fixed
  • review set to review+
  • Status changed from new to closed

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

comment:4 Changed 12 years ago 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 ;)

Note: See TracTickets for help on using tickets.