Merge pull request #1589 from slarosa/fix11247

[expression builder] fixes #11247
qgis · Sep 23, 2014 · d37f75f · d37f75f
2 parents 72a33ea + bad94e0
commit d37f75f
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 3 deletions.
diff --git a/src/gui/qgscodeeditor.cpp b/src/gui/qgscodeeditor.cpp
@@ -106,6 +106,14 @@ void QgsCodeEditor::setFoldingVisible( bool folding )
   }
 }
 
+void QgsCodeEditor::insertText( QString theText )
+{
+  int line, index;
+  getCursorPosition( &line, &index );
+  insertAt( theText, line, index );
+  setCursorPosition( line, index + theText.length() );
+}
+
 // Settings for font and fontsize
 bool QgsCodeEditor::isFixedPitch( const QFont& font )
 {

diff --git a/src/gui/qgscodeeditor.h b/src/gui/qgscodeeditor.h
@@ -60,6 +60,11 @@ class GUI_EXPORT QgsCodeEditor : public QsciScintilla
     void setFoldingVisible( bool folding );
     bool foldingVisible() { return mFolding; }
 
+    /** Isert text at cursor position
+     * @param theText The text to be inserted
+     */
+    void insertText( QString theText );
+
   protected:
 
     bool isFixedPitch( const QFont& font );

diff --git a/src/gui/qgsexpressionbuilderwidget.cpp b/src/gui/qgsexpressionbuilderwidget.cpp
@@ -158,7 +158,7 @@ void QgsExpressionBuilderWidget::on_expressionTree_doubleClicked( const QModelIn
     return;
 
   // Insert the expression text.
-  txtExpressionString->insert( item->getExpressionText() );
+  txtExpressionString->insertText( item->getExpressionText() );
   txtExpressionString->setFocus();
 }
 
@@ -392,14 +392,14 @@ void QgsExpressionBuilderWidget::on_lblPreview_linkActivated( QString link )
 
 void QgsExpressionBuilderWidget::on_mValueListWidget_itemDoubleClicked( QListWidgetItem *item )
 {
-  txtExpressionString->insert( " " + item->text() + " " );
+  txtExpressionString->insertText( " " + item->text() + " " );
   txtExpressionString->setFocus();
 }
 
 void QgsExpressionBuilderWidget::operatorButtonClicked()
 {
   QPushButton* button = dynamic_cast<QPushButton*>( sender() );
-  txtExpressionString->insert( " " + button->text() + " " );
+  txtExpressionString->insertText( " " + button->text() + " " );
   txtExpressionString->setFocus();
 }