hide setting and value control in setSortOrder

qgis · May 10, 2017 · 8f862a5 · 8f862a5
1 parent a82b8cf
commit 8f862a5
Showing 1 changed file with 2 additions and 7 deletions.
diff --git a/src/core/qgsattributetableconfig.cpp b/src/core/qgsattributetableconfig.cpp
@@ -196,13 +196,8 @@ void QgsAttributeTableConfig::readXml( const QDomNode& node )
   }
 
   mSortExpression = configNode.toElement().attribute( "sortExpression" );
-  mSortOrder = static_cast<Qt::SortOrder>( configNode.toElement().attribute( "sortOrder" ).toInt() );
-  // fix https://hub.qgis.org/issues/15803
-  // because static_cast give umpredictable value if value is not in the enum range
-  if ( mSortOrder != Qt::AscendingOrder && mSortOrder != Qt::DescendingOrder )
-  {
-    mSortOrder = Qt::AscendingOrder;
-  }
+  Qt::SortOrder sortOrder = static_cast<Qt::SortOrder>( configNode.toElement().attribute( "sortOrder" ).toInt() );
+  setSortOrder( sortOrder );
 }
 
 QString QgsAttributeTableConfig::sortExpression() const