Bug 332329

Hi Robert, yes it will be useful to have samples from you.

I set this bug as a release_blocker for 2.8.1.

Created attachment 85637 [details]
that's the file «from 2.7.2»

Created attachment 85638 [details]
and that's the corrupted one

The additions are
the last line in table liens with licleunik=1072
lines in table lieide with licleunik=1072
Links in queries design mode are between "??cleunik" of same names

Thanks Robert. It's better for you to go back to 2.7.x until there's a fix.

la 19/03/2014 17:36, Jarosław Staniek skribis (esperanto estas la unua 
internacia lingvo)
> https://bugs.kde.org/show_bug.cgi?id=332329
>
> --- Comment #5 from Jarosław Staniek <staniek@kde.org> ---
> Thanks Robert. It's better for you to go back to 2.7.x until there's a fix.
>

So, I desactivated deb-src 
http://ppa.launchpad.net/kubuntu-ppa/backports/ubuntu/ saucy main and 
recovered the 2.7.2
Good luck, and thanks for being so reactive

Perhaps related to this change of behavior: https://bugs.kde.org/show_bug.cgi?id=331259. I guess I am on right track.

I just had a glance at 2.9 within NEON   . The bug is there.

2.9 is Pre-Alpha now (calligra master branch) so yes, it's unfixed. Your case helps a lot, not for the first time :)

I am focused on the " sometimes the SQL of the queries no longer shows the keys prefixed by the name of a table" part of the report, hoping the rest may be related to this misbehavior.

indeed that's the case of the corrupted files

I'm presently  adding my new datas to the file under 2.7.2, and something has not been clear. I'll do again .
Could the problem (at least part of it)  be in the file ? 

Is there a way to launch a verbose Kexi ?

@Robert I'd like to keep this bug report devoted to one issue, if there's something else (corruption?) I encourage to discuss it in details on the forum first, then maybe file another bug report when we can spot the issue.

The test file https://bugs.kde.org/attachment.cgi?id=85637&action=edit has query 'liensidentite' that's already invalid:
SELECT civilite, nom, prenom, courriel, adresse1, adresse2, code, commune, trilien, licleunik, idcleunik, remarque, idcleunik FROM lieide, identite, liens, localis, codecomm, pays WHERE (codecomm.cocleunik = localis.cocleunik AND localis.locleunik = identite.locleunik AND identite.idcleunik = lieide.idcleunik AND liens.licleunik = lieide.licleunik AND pays.pacleunik = codecomm.pacleunik) AND (liens.licleunik = 48) ORDER BY identite.nom, identite.prenom

The reason why it's invalid: "Both table "lieide" and "liens" have defined "licleunik" field. Use ".licleunik" notation to specify table name."

I reduced the test to a query "SELECT identite.idcleunik FROM lieide, identite WHERE identite.idcleunik = lieide.idcleunik". Note the use of "identite." prefix. So the query is valid. The query 'liensidentite' can be fixed the same way.

Switching to design view breaks the fix and we're again with invalid query. Forthcoming fix addresses this.

Review of a fix is in progress: https://git.reviewboard.kde.org/r/116986/

@Robert: Changed name of this bug. Should you have separate bugs we'll need them reported separately.

Git commit bdab8c724e91e59e247835ea1cc7dbacf39f752e by Jaroslaw Staniek.
Committed on 22/03/2014 at 23:11.
Pushed by staniek into branch 'calligra/2.8'.

Do not remove table prefix when it is needed
REVIEW:116986

M  +2    -1    libs/db/connection.cpp
M  +5    -4    libs/db/expression.cpp

http://commits.kde.org/calligra/bdab8c724e91e59e247835ea1cc7dbacf39f752e

Git commit 5728406cd72a42add30163cba03cdcd643febfd2 by Jaroslaw Staniek.
Committed on 22/03/2014 at 23:11.
Pushed by staniek into branch 'master'.

Do not remove table prefix when it is needed

M  +2    -1    libs/db/connection.cpp
M  +5    -4    libs/db/expression.cpp

http://commits.kde.org/calligra/5728406cd72a42add30163cba03cdcd643febfd2

So I understand that master (and NEON) includes yet the correction.

For Mint I'll check for the next upgrade of calligra in https://launchpad.net/~kubuntu-ppa/+archive/backports/+packages

Thanks