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

Julien Nabet <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected],
                   |                            |[email protected]

--- Comment #6 from Julien Nabet <[email protected]> ---
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.
_______________________________________________
Libreoffice-bugs mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

Reply via email to