Fix broken QgsDualView::copyCellContent() method

This is a public method, yet was unusable publicly due to the forced use of sender() to retrieve the current model index
qgis · Jun 2, 2020 · 2376263 · 2376263
1 parent 61d627a
commit 2376263
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/src/gui/attributetable/qgsdualview.cpp b/src/gui/attributetable/qgsdualview.cpp
@@ -704,14 +704,14 @@ int QgsDualView::filteredFeatureCount()
 
 void QgsDualView::copyCellContent() const
 {
-  QAction *action = qobject_cast<QAction *>( sender() );
-
-  if ( action && action->data().isValid() && action->data().canConvert<QModelIndex>() )
+  const QModelIndex currentIndex = mTableView->currentIndex();
+  if ( !currentIndex.isValid() )
   {
-    QModelIndex index = action->data().toModelIndex();
-    QVariant var = mMasterModel->data( index, Qt::DisplayRole );
-    QApplication::clipboard()->setText( var.toString() );
+    return;
   }
+
+  QVariant var = mMasterModel->data( currentIndex, Qt::DisplayRole );
+  QApplication::clipboard()->setText( var.toString() );
 }
 
 void QgsDualView::cancelProgress()