fix #4969

qgis · Jun 18, 2012 · 8eb323a · 8eb323a
1 parent d133099
commit 8eb323a
Showing 1 changed file with 6 additions and 23 deletions.
diff --git a/src/providers/postgres/qgspgtablemodel.cpp b/src/providers/postgres/qgspgtablemodel.cpp
@@ -90,22 +90,13 @@ void QgsPgTableModel::addTableEntry( QgsPostgresLayerProperty layerProperty )
   QStandardItem *sridItem = new QStandardItem( layerProperty.srid );
   sridItem->setEditable( false );
 
-  QString pkText, pkCol = "";
-  if ( layerProperty.pkCols.size() == 0 )
+  QString pkCol = "";
+  if ( layerProperty.pkCols.size() > 0 )
   {
-    pkText = "";
-  }
-  else
-  {
-    if ( layerProperty.pkCols.size() > 1 )
-    {
-      pkText = tr( "Select..." );
-    }
-
     pkCol = layerProperty.pkCols[0];
   }
 
-  QStandardItem *pkItem = new QStandardItem( pkText );
+  QStandardItem *pkItem = new QStandardItem( pkCol );
   if ( layerProperty.pkCols.size() > 1 )
     pkItem->setFlags( pkItem->flags() | Qt::ItemIsEditable );
   else
@@ -133,15 +124,11 @@ void QgsPgTableModel::addTableEntry( QgsPostgresLayerProperty layerProperty )
   bool detailsFromThread = wkbType == QGis::WKBUnknown ||
                            ( wkbType != QGis::WKBNoGeometry && layerProperty.srid.isEmpty() );
 
-  if ( detailsFromThread || pkText == tr( "Select..." ) )
+  if ( detailsFromThread )
   {
-    Qt::ItemFlags flags = Qt::ItemIsSelectable;
-    if ( detailsFromThread )
-      flags |= Qt::ItemIsEnabled;
-
     foreach( QStandardItem *item, childItemList )
     {
-      item->setFlags( item->flags() & ~flags );
+      item->setFlags( item->flags() & ~( Qt::ItemIsSelectable | Qt::ItemIsEnabled ) );
     }
   }
 
@@ -274,13 +261,9 @@ void QgsPgTableModel::setGeometryTypesForTable( QgsPostgresLayerProperty layerPr
 
         row[ dbtmSrid ]->setText( sridList.at( 0 ) );
 
-        Qt::ItemFlags flags = Qt::ItemIsEnabled;
-        if ( layerProperty.pkCols.size() < 2 )
-          flags |= Qt::ItemIsSelectable;
-
         foreach( QStandardItem *item, row )
         {
-          item->setFlags( item->flags() | flags );
+          item->setFlags( item->flags() | Qt::ItemIsSelectable | Qt::ItemIsEnabled );
         }
 
         for ( int j = 1; j < typeList.size(); j++ )