Microoptimise opening attribute table - don't call sort on model when…

… no sort order set
qgis · May 23, 2017 · 381eef0 · 381eef0
1 parent ebee101
commit 381eef0
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/src/gui/attributetable/qgsattributetablefiltermodel.cpp b/src/gui/attributetable/qgsattributetablefiltermodel.cpp
@@ -215,7 +215,8 @@ void QgsAttributeTableFilterModel::setAttributeTableConfig( const QgsAttributeTa
     }
   }
 
-  sort( config.sortExpression(), config.sortOrder() );
+  if ( !config.sortExpression().isEmpty() )
+    sort( config.sortExpression(), config.sortOrder() );
 }
 
 void QgsAttributeTableFilterModel::sort( const QString &expression, Qt::SortOrder order )

diff --git a/src/gui/attributetable/qgsattributetablemodel.cpp b/src/gui/attributetable/qgsattributetablemodel.cpp
@@ -257,7 +257,7 @@ void QgsAttributeTableModel::editCommandEnded()
 void QgsAttributeTableModel::attributeDeleted( int idx )
 {
   if ( mSortCacheAttributes.contains( idx ) )
-    prefetchSortData( QLatin1String( "" ) );
+    prefetchSortData( QString() );
 }
 
 void QgsAttributeTableModel::layerDeleted()
@@ -785,7 +785,7 @@ void QgsAttributeTableModel::prefetchColumnData( int column )
 {
   if ( column == -1 || column >= mAttributes.count() )
   {
-    prefetchSortData( QLatin1String( "" ) );
+    prefetchSortData( QString() );
   }
   else
   {