From 184bf418ca01d19cd5ffa08ab3dc6b89c5f20a6b Mon Sep 17 00:00:00 2001
From: vmora <vincent.mora@oslandia.com>
Date: Wed, 10 Jul 2013 11:13:24 +0200
Subject: [PATCH] - removed useless function getPrimaryKey since a member
 already exist   for that - modified the test to have multiple column primary
 key (it fails for   the moment

---
 .../spatialite/qgsspatialiteprovider.cpp      | 37 +------------------
 .../spatialite/qgsspatialiteprovider.h        |  4 --
 .../src/python/test_qgsspatialiteprovider.py  |  2 +-
 3 files changed, 2 insertions(+), 41 deletions(-)

diff --git a/src/providers/spatialite/qgsspatialiteprovider.cpp b/src/providers/spatialite/qgsspatialiteprovider.cpp
index 374774a256c..359a42bf4bf 100644
--- a/src/providers/spatialite/qgsspatialiteprovider.cpp
+++ b/src/providers/spatialite/qgsspatialiteprovider.cpp
@@ -3506,10 +3506,7 @@ bool QgsSpatiaLiteProvider::addFeatures( QgsFeatureList & flist )
   QString values;
   QString separator;
   int ia, ret;
-
   
-  const QString primaryKey = getPrimaryKey();
-
   if ( flist.size() == 0 )
     return true;
   const QgsAttributes & attributevec = flist[0].attributes();
@@ -3609,7 +3606,7 @@ bool QgsSpatiaLiteProvider::addFeatures( QgsFeatureList & flist )
         continue;
 
       // replace primary key with NULL so that sqlite will generate one for us
-      if ( primaryKey == fieldname )
+      if ( mPrimaryKey == fieldname )
       { 
         v = QVariant();
         assert(v.toString().isEmpty());
@@ -5208,35 +5205,3 @@ QGISEXTERN bool deleteLayer( const QString& dbPath, const QString& tableName, QS
   return true;
 }
 
-QString QgsSpatiaLiteProvider::getPrimaryKey()
-{
-  char **results;
-  int rows;
-  int columns;
-  char *errMsg = NULL;
-  QString sql = QString( "PRAGMA table_info(%1)" ).arg( mQuery );
-
-  int ret = sqlite3_get_table( sqliteHandle, sql.toUtf8().constData(), &results, &rows, &columns, &errMsg );
-  if ( ret != SQLITE_OK )
-  {
-    QgsDebugMsg( "sqlite error: " + QString::fromUtf8( errMsg ) );
-  }
-
-  int pkColIdx=0;
-  for (; (pkColIdx < columns) && (QString("pk") != results[pkColIdx]) ; pkColIdx++){}
-  assert(pkColIdx<columns); // we must find it
-
-  int nameIdx=0;
-  for (; (nameIdx < columns) && (QString("name") != results[nameIdx]) ; nameIdx++){}
-  assert(nameIdx<columns); // we must find it
-
-  for (int r=0; r<rows; r++)
-  {
-    if ( QString("1") == results[r*columns + pkColIdx] )
-    { 
-      return QString::fromUtf8( results[r*columns + nameIdx] );
-    }
-  }
-
-  return QString();
-}
diff --git a/src/providers/spatialite/qgsspatialiteprovider.h b/src/providers/spatialite/qgsspatialiteprovider.h
index b97a400e6e5..fc5b938a4aa 100644
--- a/src/providers/spatialite/qgsspatialiteprovider.h
+++ b/src/providers/spatialite/qgsspatialiteprovider.h
@@ -124,10 +124,6 @@ class QgsSpatiaLiteProvider: public QgsVectorDataProvider
     */
     virtual void updateExtents();
 
-    /**  * Get the name of the primary key for the layer
-    */
-    QString getPrimaryKey();
-
     /**
       * Get the field information for the layer
       * @return vector of QgsField objects
diff --git a/tests/src/python/test_qgsspatialiteprovider.py b/tests/src/python/test_qgsspatialiteprovider.py
index b03d41950c5..20dec04de30 100644
--- a/tests/src/python/test_qgsspatialiteprovider.py
+++ b/tests/src/python/test_qgsspatialiteprovider.py
@@ -41,7 +41,7 @@ class TestQgsSpatialiteProvider(TestCase):
         cur = con.cursor()
         sql = "SELECT InitSpatialMetadata()"
         cur.execute(sql)
-        sql = "CREATE TABLE test_pg (id INTEGER NOT NULL PRIMARY KEY, name TEXT NOT NULL)"
+        sql = "CREATE TABLE test_pg (id INTEGER NOT NULL, name TEXT NOT NULL, PRIMARY KEY (id, name))"
         cur.execute(sql)
         sql = "SELECT AddGeometryColumn('test_pg', 'geometry', 4326, 'POLYGON', 'XY')"
         cur.execute(sql)