developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.
#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)
Change History (6)
comment:1 Changed 13 years ago by nuks
- Owner nuKs deleted
comment:2 Changed 13 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.
comment:3 Changed 13 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 13 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 ;)
Il faut d'abord effectué le ticket #579 + ajouter une clause having aux DAOs