diff options
author | Mike Kaganski <mike.kaganski@collabora.com> | 2019-11-29 21:56:38 +0300 |
---|---|---|
committer | Mike Kaganski <mike.kaganski@collabora.com> | 2019-11-29 21:20:56 +0100 |
commit | d997addd504c92fea58cf62fddc2b1e9b8216a7d (patch) | |
tree | 2107ed71d325008a4b7c1527b76f9b5c24fbfa05 | |
parent | b1832658951d9f03c871b665b270a73deacf884f (diff) |
tdf#129007: fix autocompletion
Using EditView::SelectCurrentWord in arbitrary selection positions
is not a correct approach. When cursor is after "_" in "=_+FUNC",
the method will select "=_+", i.e. the substring that cannot start
any Calc identifier (and actually ending *after* current cursor).
When matching identifiers like "bb.bbb" in "=bb.bb", the end result
will be "=bb.bb.bbb", etc.
So instead of trying to match words and hack around problems, let's
simply find longest match in the string starting from beginning of
the line, and ending at current cursor position. Testing trailing
parts of that string of decreasing length, the first found match is
the longest, which is what we need. This also avoids multiple calls
to expensive EditEngine methods.
Change-Id: Ia3470cc85a4ba9f9ab34cbe9db3250bf28e93350
Reviewed-on: https://gerrit.libreoffice.org/84087
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
-rw-r--r-- | sc/source/ui/app/inputhdl.cxx | 49 |
1 files changed, 15 insertions, 34 deletions
diff --git a/sc/source/ui/app/inputhdl.cxx b/sc/source/ui/app/inputhdl.cxx index 640ea40060e0..2ea4ba7a6c2d 100644 --- a/sc/source/ui/app/inputhdl.cxx +++ b/sc/source/ui/app/inputhdl.cxx @@ -1478,48 +1478,29 @@ void ScInputHandler::NextFormulaEntry( bool bBack ) namespace { -bool needToExtendSelection(const OUString& rSelectedText, const OUString& rInsertText) -{ - return !ScGlobal::GetpTransliteration()->isMatch( rSelectedText, rInsertText); -} - void completeFunction( EditView* pView, const OUString& rInsert, bool& rParInserted ) { if (pView) { ESelection aSel = pView->GetSelection(); - --aSel.nStartPos; - --aSel.nEndPos; - pView->SetSelection(aSel); - pView->SelectCurrentWord(); - - // a dot and underscore are word separators so we need special - // treatment for any formula containing a dot or underscore - if(rInsert.indexOf(".") != -1 || rInsert.indexOf("_") != -1) + { - // need to make sure that we replace also the part before the dot - // go through the word to find the match with the insert string - aSel = pView->GetSelection(); - ESelection aOldSelection = aSel; - OUString aSelectedText = pView->GetSelected(); - if ( needToExtendSelection( aSelectedText, rInsert ) ) - { - while(needToExtendSelection(aSelectedText, rInsert)) - { - assert(aSel.nStartPos > 0); - --aSel.nStartPos; - aSel.nEndPos = aSel.nStartPos; - pView->SetSelection(aSel); - pView->SelectCurrentWord(); - aSelectedText = pView->GetSelected(); - } - aSel.nStartPos = aSel.nEndPos - ( aSelectedText.getLength() - 1 ); - } - else + const sal_Int32 nMinLen = std::max(aSel.nEndPos - aSel.nStartPos, sal_Int32(1)); + // Since transliteration service is used to test for match, the replaced string could be + // longer than rInsert, so in order to find longest match before the cursor, test whole + // string from start to current cursor position (don't limit to length of rInsert) + aSel.nStartPos = 0; + pView->SetSelection(aSel); + const OUString aAll = pView->GetSelected(); + OUString aMatch; + for (sal_Int32 n = aAll.getLength(); n >= nMinLen && aMatch.isEmpty(); --n) { - aSel.nStartPos = aSel.nEndPos - aSelectedText.getLength(); + const OUString aTest = aAll.copy(aAll.getLength() - n); // n trailing chars + if (ScGlobal::GetpTransliteration()->isMatch(aTest, rInsert)) + aMatch = aTest; // Found => break the loop } - aSel.nEndPos = aOldSelection.nEndPos; + + aSel.nStartPos = aSel.nEndPos - aMatch.getLength(); pView->SetSelection(aSel); } |