[Libreoffice-bugs] [Bug 117090] EDITING: Firebird: migration: View definitions are lost from embedded hsql files using functions unknown to Firebird

[bugzilla-daemon at bugs.documentfoundation.org]

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

Julien Nabet <serval2412 at yahoo.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |lionel at mamane.lu,
                   |                            |serval2412 at yahoo.fr

--- Comment #6 from Julien Nabet <serval2412 at yahoo.fr> ---
Just for curiosity, would it be ok to detect the presence of CONCAT (as a
function, not as a variable called CONCAT1 for example) and display a warning
message inviting the user to change CONCAT syntax to || syntax before
migrating?

Indeed, replacing CONCAT syntax with || syntax is not easy since:
- you can have imbricated CONCAT => recursive algo
- you can have some spaces before/after concat, parenthesis, ...
- you can have values named <CONCAT something>
- you must find matching end parenthesis of the beginning parenthesis of the
CONCAT
I found some algo which are not straightforward and that they don't even take
into account parenthesis as values + deal with values declared with quote or
double quotes: eg :
CONCAT(a, ')', CONCAT(b, ")", c))

Unless there's an existing parsing mechanism to call which would already know
how to deal with all this...

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/libreoffice-bugs/attachments/20180530/9915b6f3/attachment.html>