SQLRequest et table de jointure

Répondre
doc
Messages : 187
Enregistré le : dim. févr. 26, 2017 4:39 pm

lun. mai 22, 2017 2:28 pm

Bonjour les devs,

J'ai besoin d'effectuer un filtrage sur un ElementComboBox (SITE_CLIENT) avec une table de jointure et ne voyant pas trop comment faire avec la SQLRequest, je joue avec les WHERE, les or() et les and(), est ce qu'il y a déjà qqchose d'implémenté plus proprement que je n'ai pas compris ou passé à côté ??
Il y a bien JoinSQLElement et SQLElementLink mais j'avoue ne pas vraiment comprendre son utilisation...

Requête que je souhaite faire pour infos:

Code : Tout sélectionner

SELECT * FROM "SITE_CLIENT" 
JOIN "GESTIONNAIRE_SITE"
ON "SITE_CLIENT"."ID_CLIENT" = "GESTIONNAIRE_SITE"."ID_CLIENT_SITES"
WHERE "GESTIONNAIRE_SITE"."ID_CLIENT_MANAGER" = XXX 
OR "SITE_CLIENT"."ID_CLIENT" = XXX
OR "SITE_CLIENT"."ID_CLIENT_GERANT" = XXX
GROUP BY "SITE_CLIENT"."ID"
(Sélectionner tous les sites du client ou du gérant ou du gestionnaire)

STRUCTURE TABLE:

---------------
SITE_CLIENT
ID
ID_CLIENT -> FK vers table CLIENT
..
----------------
GESTIONNAIRE_SITE
ID
ID_CLIENT_MANAGER -> FK vers table CLIENT
ID_CLIENT_SITES -> FK vers table CLIENT
----------------


Merci.
Haut
doc
Messages : 187
Enregistré le : dim. févr. 26, 2017 4:39 pm

lun. juin 26, 2017 12:40 pm

Bonjour,

Je m'arrache les cheveux depuis quelques jours sur ce problème que je n'arrives pas à résoudre (en tout cas proprement), je rajoute un transformer sur la requête de mon combi, lequel génère une requête correcte mais lors de son execution, j'ai une erreur car dans la requête est ajouté un

Code : Tout sélectionner

FOR SHARE OF "SITE_CLIENT"
qui m'empêche d'utiliser DISTINCT ou GROUP BY ainsi qu'une condition

Code : Tout sélectionner

AND (1=0))
dont je ne sais d'où elle sors et qui renvoi toujours FALSE.

Code : Tout sélectionner

// à chaque changement de client, on ré-initialise le combo avec les sites du client
comboClient.addModelListener("wantedID", new PropertyChangeListener() {

    @Override
    public void propertyChange(PropertyChangeEvent evt) {
        int wantedID = comboClient.getWantedID();

        if (wantedID != SQLRow.NONEXISTANT_ID && wantedID >= SQLRow.MIN_VALID_ID) {
            final SQLRow rowClient = getTable().getForeignTable("ID_CLIENT").getRow(wantedID);
            /*Where clause = new Where(comboSite.getRequest().getPrimaryTable().getField("ID_CLIENT"), "=", rowClient.getID());
            Where clause2 = new Where(comboSite.getRequest().getPrimaryTable().getField("ID_CLIENT_GERANT"), "=", rowClient.getID());
            SQLElement managersElement = Configuration.getInstance().getDirectory().getElement("GESTIONNAIRE_SITE");
            SQLTable managersTable = managersElement.getListRequest().getPrimaryTable();
            Where clause3 = new Where(managersTable.getField("ID_CLIENT_MANAGER"), "=", rowClient.getID());
            Where clause4 = new Where(comboSite.getRequest().getPrimaryTable().getField("ID_CLIENT"), "=", managersTable.getField("ID_CLIENT_SITES"));
            Where clause5 = new Where(comboSite.getRequest().getPrimaryTable().getField("ID_CLIENT"), "<>", rowClient.getID());
            comboSite.getRequest().setWhere(clause.or(clause2).or(clause3.and(clause4)));
            */
            comboSite.getRequest().setSelectTransf(new ITransformer<SQLSelect, SQLSelect>() {
                @Override
                public SQLSelect transformChecked(SQLSelect sel) {
                	SQLElement managersElement = Configuration.getInstance().getDirectory().getElement("GESTIONNAIRE_SITE");
                	SQLTable managersTable = managersElement.getListRequest().getPrimaryTable();
                	SQLTable siteTable = comboSite.getRequest().getPrimaryTable();
                	TableRef aTableRef = new AliasedTable(managersTable, "g");
                	Where joinClause = new Where(siteTable.getField("ID_CLIENT"), "=", aTableRef.getField("ID_CLIENT_SITES"));
                    final SQLSelectJoin join = sel.addJoin("INNER", aTableRef, joinClause);
                    Where w = new Where(siteTable.getField("ID_CLIENT"), "=", rowClient.getID());
                    w = w.or(new Where(siteTable.getField("ID_CLIENT_GERANT"), "=", rowClient.getID()));
                    w = w.or(new Where(join.getJoinedTable().getField("ID_CLIENT_MANAGER"), "=", rowClient.getID()));
                    join.setWhere(w);
                    
                    sel.setDistinct(true);
                    
                    System.err.println(sel.asString());
                    return sel;
                }
            });

        } else {
        	comboSite.getRequest().setWhere(null);
        }
    }
});
Et l'erreur qui en résulte:

Code : Tout sélectionner

SELECT DISTINCT "SITE_CLIENT"."ID", "SITE_CLIENT"."NUMERO", "SITE_CLIENT"."CODE_IMPLANT", "SITE_CLIENT"."NOM", "SITE_CLIENT"."ORDRE"
 FROM "OpenConcerto"."OpenConcerto48"."SITE_CLIENT"
 INNER JOIN "OpenConcerto"."OpenConcerto48"."GESTIONNAIRE_SITE" "g" on ((("SITE_CLIENT"."ID_CLIENT" = "g"."ID_CLIENT_SITES") AND ("g"."ARCHIVE" = 0)) AND ("g"."ID" <> -1)) AND ((("SITE_CLIENT"."ID_CLIENT" = 2) OR ("SITE_CLIENT"."ID_CLIENT_GERANT" = 2)) OR ("g"."ID_CLIENT_MANAGER" = 2))
 WHERE ("SITE_CLIENT"."ID" <> 1) AND ("SITE_CLIENT"."ARCHIVE" = 0)
 ORDER BY "SITE_CLIENT"."ORDRE"
java.util.concurrent.ExecutionException: java.lang.IllegalStateException: Impossible d'accéder au résultat de SELECT DISTINCT "SITE_CLIENT"."ID", "SITE_CLIENT"."NUMERO", "SITE_CLIENT"."CODE_IMPLANT", "SITE_CLIENT"."NOM", "SITE_CLIENT"."ORDRE"
 FROM "OpenConcerto"."OpenConcerto48"."SITE_CLIENT"
 INNER JOIN "OpenConcerto"."OpenConcerto48"."GESTIONNAIRE_SITE" "g" on ((("SITE_CLIENT"."ID_CLIENT" = "g"."ID_CLIENT_SITES") AND ("g"."ARCHIVE" = 0)) AND ("g"."ID" <> -1)) AND ((("SITE_CLIENT"."ID_CLIENT" = 2) OR ("SITE_CLIENT"."ID_CLIENT_GERANT" = 2)) OR ("g"."ID_CLIENT_MANAGER" = 2))
 WHERE ("SITE_CLIENT"."ID" <> 1) AND (("SITE_CLIENT"."ARCHIVE" = 0) AND (1=0))
 ORDER BY "SITE_CLIENT"."ORDRE" FOR SHARE OF "SITE_CLIENT"
 in jdbc:postgresql://127.0.0.1:5432/OpenConcerto
	at java.util.concurrent.FutureTask$Sync.innerGet(Unknown Source)
	at java.util.concurrent.FutureTask.get(Unknown Source)
	at org.openconcerto.utils.SwingWorker2.get(SwingWorker2.java:412)
	at org.openconcerto.sql.sqlobject.IComboModel$4.done(IComboModel.java:351)
	at org.openconcerto.utils.SwingWorker2$5.run(SwingWorker2.java:536)
	at org.openconcerto.utils.SwingWorker2$DoSubmitAccumulativeRunnable.run(SwingWorker2.java:619)
	at sun.swing.AccumulativeRunnable.run(Unknown Source)
	at org.openconcerto.utils.SwingWorker2$DoSubmitAccumulativeRunnable.actionPerformed(SwingWorker2.java:631)
	at javax.swing.Timer.fireActionPerformed(Unknown Source)
	at javax.swing.Timer$DoPostEvent.run(Unknown Source)
	at java.awt.event.InvocationEvent.dispatch(Unknown Source)
	at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
	at java.awt.EventQueue.access$400(Unknown Source)
	at java.awt.EventQueue$2.run(Unknown Source)
	at java.awt.EventQueue$2.run(Unknown Source)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.AccessControlContext$1.doIntersectionPrivilege(Unknown Source)
	at java.awt.EventQueue.dispatchEvent(Unknown Source)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.run(Unknown Source)
Caused by: java.lang.IllegalStateException: Impossible d'accéder au résultat de SELECT DISTINCT "SITE_CLIENT"."ID", "SITE_CLIENT"."NUMERO", "SITE_CLIENT"."CODE_IMPLANT", "SITE_CLIENT"."NOM", "SITE_CLIENT"."ORDRE"
 FROM "OpenConcerto"."OpenConcerto48"."SITE_CLIENT"
 INNER JOIN "OpenConcerto"."OpenConcerto48"."GESTIONNAIRE_SITE" "g" on ((("SITE_CLIENT"."ID_CLIENT" = "g"."ID_CLIENT_SITES") AND ("g"."ARCHIVE" = 0)) AND ("g"."ID" <> -1)) AND ((("SITE_CLIENT"."ID_CLIENT" = 2) OR ("SITE_CLIENT"."ID_CLIENT_GERANT" = 2)) OR ("g"."ID_CLIENT_MANAGER" = 2))
 WHERE ("SITE_CLIENT"."ID" <> 1) AND (("SITE_CLIENT"."ARCHIVE" = 0) AND (1=0))
 ORDER BY "SITE_CLIENT"."ORDRE" FOR SHARE OF "SITE_CLIENT"
 in jdbc:postgresql://127.0.0.1:5432/OpenConcerto
	at org.openconcerto.sql.model.SQLDataSource.execute(SQLDataSource.java:640)
	at org.openconcerto.sql.model.SQLDataSource.execute(SQLDataSource.java:545)
	at org.openconcerto.sql.model.SQLRowValuesListFetcher.fetch(SQLRowValuesListFetcher.java:1217)
	at org.openconcerto.sql.model.SQLRowValuesListFetcher.fetch(SQLRowValuesListFetcher.java:1150)
	at org.openconcerto.sql.model.SQLRowValuesListFetcher.fetch(SQLRowValuesListFetcher.java:1132)
	at org.openconcerto.sql.request.ComboSQLRequest.getComboItems(ComboSQLRequest.java:317)
	at org.openconcerto.sql.request.ComboSQLRequest.getComboItems(ComboSQLRequest.java:293)
	at org.openconcerto.sql.sqlobject.IComboModel$4.doInBackground(IComboModel.java:333)
	at org.openconcerto.sql.sqlobject.IComboModel$4.doInBackground(IComboModel.java:1)
	at org.openconcerto.utils.SwingWorker2$1.call(SwingWorker2.java:115)
	at java.util.concurrent.FutureTask$Sync.innerRun(Unknown Source)
	at java.util.concurrent.FutureTask.run(Unknown Source)
	at org.openconcerto.utils.SwingWorker2.run(SwingWorker2.java:163)
	at java.util.concurrent.ThreadPoolExecutor$Worker.runTask(Unknown Source)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
	at java.lang.Thread.run(Unknown Source)
Caused by: java.sql.SQLException: second exec failed: ERREUR: FOR SHARE n'est pas autorisé avec la clause DISTINCT
	at org.openconcerto.sql.model.SQLDataSource.executeTwice(SQLDataSource.java:965)
	at org.openconcerto.sql.model.SQLDataSource.execute(SQLDataSource.java:608)
	... 15 more
Caused by: org.postgresql.util.PSQLException: ERREUR: FOR SHARE n'est pas autorisé avec la clause DISTINCT
	at org.postgresql.core.v3.QueryExecutorImpl.receiveErrorResponse(QueryExecutorImpl.java:2157)
	at org.postgresql.core.v3.QueryExecutorImpl.processResults(QueryExecutorImpl.java:1886)
	at org.postgresql.core.v3.QueryExecutorImpl.execute(QueryExecutorImpl.java:255)
	at org.postgresql.jdbc2.AbstractJdbc2Statement.execute(AbstractJdbc2Statement.java:555)
	at org.postgresql.jdbc2.AbstractJdbc2Statement.executeWithFlags(AbstractJdbc2Statement.java:403)
	at org.postgresql.jdbc2.AbstractJdbc2Statement.execute(AbstractJdbc2Statement.java:395)
	at org.apache.commons.dbcp.DelegatingStatement.execute(DelegatingStatement.java:264)
	at org.openconcerto.sql.model.SQLDataSource$ExecutorThread.run(SQLDataSource.java:1157)
Qqn peux t'il me donner un petit coup de main, j'aimerais faire de manière propre plutôt quel méthode que j'utilise actuellement.

Merci.
Haut
Avatar du membre
guillaume
Messages : 2433
Enregistré le : ven. févr. 11, 2011 7:15 pm

lun. juin 26, 2017 2:04 pm

Bonjour,

Pour le 1=0 c'est la seule façon de créer un "false" compatible avec toutes les bases de données supportées, certaines n'ont pas de "false" ni de booléen, il vient d'un Where.FALSE .

Pour retirer le for share : setLockSelect(false) sur la request.

Cordialement,
Directeur technique d'OpenConcerto qui dans son temps libre s'occupe du forum.
Pour une assistance pro, nous sommes joignables à ILM Informatique contre quelques jetons.
Pensez aussi à lire le manuel !
Haut
doc
Messages : 187
Enregistré le : dim. févr. 26, 2017 4:39 pm

mar. juin 27, 2017 8:17 am

Merci Guillaume pour le petit coup de main, effectivement j’initialisai le combo avec un

Code : Tout sélectionner

comboSite.getRequest().setWhere(Where.FALSE);
Je pensais que les conditions étaient réinitialisées lors du transformChecked(), maintenant que j'y pense transformChecked(), le mot transform aurait dû me mettre la puce à l'oreille.
Quant à setLockSelect(), cette fonction avait déjà attirée mon attention sans pour autant l'avoir testée, tout est rentré dans l'ordre ;).

Encore quelques petites améliorations et je mettrai également ce module à disposition de la communauté (module de gestion clients multi-sites).
Haut
Répondre

Retourner vers « General discussion »