Wrap field name in quotes when added to expression string

qgis · Dec 30, 2011 · d7b1499 · d7b1499
1 parent 9b93f9b
commit d7b1499
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 20 deletions.
diff --git a/src/gui/qgsexpressionbuilderwidget.cpp b/src/gui/qgsexpressionbuilderwidget.cpp
@@ -162,7 +162,7 @@ void QgsExpressionBuilderWidget::loadFieldNames( QgsFieldMap fields )
   {
     QString fieldName = field.name();
     fieldNames << fieldName;
-    registerItem( tr( "Fields" ), fieldName, " " + fieldName + " ", "", QgsExpressionItem::Field );
+    registerItem( tr( "Fields" ), fieldName, " \"" + fieldName + "\" ", "", QgsExpressionItem::Field );
   }
   highlighter->addFields( fieldNames );
 }

diff --git a/src/gui/qgsexpressionhighlighter.cpp b/src/gui/qgsexpressionhighlighter.cpp
@@ -20,25 +20,6 @@ QgsExpressionHighlighter::QgsExpressionHighlighter( QTextDocument *parent)
 {
   HighlightingRule rule;
 
-//     keywordFormat.setForeground(Qt::darkBlue);
-//     keywordFormat.setFontWeight(QFont::Bold);
-//     QStringList keywordPatterns;
-//     keywordPatterns << "\\bchar\\b" << "\\bclass\\b" << "\\bconst\\b"
-//                     << "\\bdouble\\b" << "\\benum\\b" << "\\bexplicit\\b"
-//                     << "\\bfriend\\b" << "\\binline\\b" << "\\bint\\b"
-//                     << "\\blong\\b" << "\\bnamespace\\b" << "\\boperator\\b"
-//                     << "\\bprivate\\b" << "\\bprotected\\b" << "\\bpublic\\b"
-//                     << "\\bshort\\b" << "\\bsignals\\b" << "\\bsigned\\b"
-//                     << "\\bslots\\b" << "\\bstatic\\b" << "\\bstruct\\b"
-//                     << "\\btemplate\\b" << "\\btypedef\\b" << "\\btypename\\b"
-//                     << "\\bunion\\b" << "\\bunsigned\\b" << "\\bvirtual\\b"
-//                     << "\\bvoid\\b" << "\\bvolatile\\b";
-//     foreach (const QString &pattern, keywordPatterns) {
-//         rule.pattern = QRegExp(pattern);
-//         rule.format = keywordFormat;
-//         highlightingRules.append(rule);
-//     }
-
   quotationFormat.setForeground( Qt::darkGreen );
   rule.pattern = QRegExp( "\'.*\'" );
   rule.format = quotationFormat;