https://bugs.documentfoundation.org/show_bug.cgi?id=123368

Julien Nabet <serval2...@yahoo.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever confirmed|0                           |1
                 CC|                            |btom...@gmail.com,
                   |                            |lio...@mamane.lu

--- Comment #6 from Julien Nabet <serval2...@yahoo.fr> ---
Thank you for your feedback, I could reproduce this.

Tamas: debugging a bit on gdb shows the pb is in utils::getTableNameFromStmt
See
https://opengrok.libreoffice.org/xref/core/dbaccess/source/filter/hsqldb/utils.cxx?r=6a736800#92

This method is called from CREATE TABLE and ALTER TABLE statements.

We can't do like in
https://cgit.freedesktop.org/libreoffice/core/commit/?id=2a08db9a0473e2f87715aacfc7b481beaa490dcf
since we got:
ALTER TABLE \"Plant Database\" ALTER COLUMN \"ID\" RESTART WITH 1497

With no space in table, perhaps we would have had this:
ALTER TABLE Plant_Database ALTER COLUMN \"ID\" RESTART WITH 1497
So can't test the presence of ".

In brief, it shows we need to use a real sql parser here because for the moment
we're using only bandaid fixes. Of course, it's more easy to tell than to do I
suppose, if not, it should have already be implemented.

-- 
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

Reply via email to