https://bugs.documentfoundation.org/show_bug.cgi?id=130672
Julien Nabet <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |[email protected], | |[email protected] --- Comment #7 from Julien Nabet <[email protected]> --- On pc Debian x86-64 with master sources updated today, I could reproduce this. Lionel: I could make it work with 2 changes: 1) diff --git a/connectivity/source/commontools/predicateinput.cxx b/connectivity/source/commontools/predicateinput.cxx index c4360f981222..9eefddba525f 100644 --- a/connectivity/source/commontools/predicateinput.cxx +++ b/connectivity/source/commontools/predicateinput.cxx @@ -250,6 +250,7 @@ namespace dbtools sTransformedText, m_xConnection, m_xFormatter, _rxField, OUString(), rParseContext.getPreferredLocale(), OUString(nDecSeparator), &rParseContext ); + sTransformedText = sTransformedText.replaceAll(" ", ""); _rPredicateValue = sTransformedText; bSuccess = true; I noticed that, after having typed "-2" for example, if I switch to another appli and go back to LO, "-2" is automatically changed into "- 2" (with a space) normalizePredicateString is called from OParameterDialog::CheckValueForError() (see https://opengrok.libreoffice.org/xref/core/dbaccess/source/ui/dlg/paramdialog.cxx?r=d52a4dba#157) I tried this to avoid modifying sqlnode.cxx (a bit afraid about regressions) 2) diff --git a/dbaccess/source/ui/dlg/paramdialog.cxx b/dbaccess/source/ui/dlg/paramdialog.cxx index ad1c23baa9f4..934ea809d09d 100644 --- a/dbaccess/source/ui/dlg/paramdialog.cxx +++ b/dbaccess/source/ui/dlg/paramdialog.cxx @@ -215,10 +215,11 @@ namespace dbaui { Reference< XPropertySet > xParamAsSet; m_xParams->getByIndex(i) >>= xParamAsSet; - +/* OUString sValue; pValues->Value >>= sValue; pValues->Value = m_aPredicateInput.getPredicateValue( sValue, xParamAsSet ); +*/ } } catch(Exception&) This one replaces too "-2" by "- 2". I must recognize I don't understand what's the use of the block I commented knowing that CheckValueForError from 1) already made the check. Any thoughts here? -- You are receiving this mail because: You are the assignee for the bug.
