More tests for json values, fix boolean representation

qgis · Jun 10, 2019 · 660e451 · 660e451
1 parent 114affc
commit 660e451
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/src/providers/postgres/qgspostgresconn.cpp b/src/providers/postgres/qgspostgresconn.cpp
@@ -1063,8 +1063,6 @@ QString QgsPostgresConn::quotedJsonValue( const QVariant &value )
 {
   if ( value.isNull() || !value.isValid() )
     return QStringLiteral( "null" );
-  if ( value.type() == QVariant::Bool )
-    return value.toBool() ?  QStringLiteral( "true" ) : QStringLiteral( "false" );
   const auto j { QgsJsonUtils::jsonFromVariant( value ) };
   return quotedString( QString::fromStdString( j.dump() ) );
 }

diff --git a/src/providers/postgres/qgspostgresconn.h b/src/providers/postgres/qgspostgresconn.h
@@ -285,6 +285,7 @@ class QgsPostgresConn : public QObject
 
     /**
      * Quote a json(b) value for placement in a SQL string.
+     * \note a null value will be represented as a NULL and not as a json null.
      */
     static QString quotedJsonValue( const QVariant &value );
 

diff --git a/tests/src/python/test_provider_postgres.py b/tests/src/python/test_provider_postgres.py
@@ -619,9 +619,11 @@ def testJson(self):
 
         attrs = (
             123,
-            None,
-            "String l'iteral with \"quotes\" and other funny chars ;#/",
             1233.45,
+            None,
+            True,
+            False,
+            r"String literal with \"quotes\" 'and' other funny chars []{};#/èé*",
             [1, 2, 3.4, None],
             [True, False],
             {'a': 123, 'b': 123.34, 'c': 'a string', 'd': [1, 2, 3], 'e': {'a': 123, 'b': 123.45}}