[PATCH] Fix fdo#32059: Commands merge with variables

Marcos Souza (via Code Review) gerrit at gerrit.libreoffice.org
Mon May 20 19:27:13 PDT 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/3983

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/83/3983/1

Fix fdo#32059: Commands merge with variables

Verify if there is a space before insert a new command to not merge
variables.

Change-Id: Ia5ddb4c4c8233b3779c0ee0c7009e181cfe5d1c3
---
M starmath/source/edit.cxx
1 file changed, 6 insertions(+), 0 deletions(-)



diff --git a/starmath/source/edit.cxx b/starmath/source/edit.cxx
index fc88ba7..2c452ca 100644
--- a/starmath/source/edit.cxx
+++ b/starmath/source/edit.cxx
@@ -714,6 +714,12 @@
 
         OSL_ENSURE( pEditView, "NULL pointer" );
         OUString aText = SM_RESSTR(nCommand);
+
+        // put an space before put a new command
+        OUString aTerm(pEditView->GetEditEngine()->GetText());
+        if (!aTerm.endsWith(" "))
+            aText = " " + aText;
+
         pEditView->InsertText(aText);
 
         if (HasMark(aText))

-- 
To view, visit https://gerrit.libreoffice.org/3983
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia5ddb4c4c8233b3779c0ee0c7009e181cfe5d1c3
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Marcos Souza <marcos.souza.org at gmail.com>



More information about the LibreOffice mailing list