From 38eef2ff3ceeef707b4940702dfc1a98e2dff16f Mon Sep 17 00:00:00 2001 From: Dmitry Degtyarev Date: Wed, 7 Apr 2021 18:22:05 +0400 Subject: [PATCH] fix find results selection bug was getting selection incorrectly caused wrong object to be added to parent select dialog and maybe other problems closes #193 --- src/admc/find_results.cpp | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/admc/find_results.cpp b/src/admc/find_results.cpp index c6fc12e3..97bbab33 100644 --- a/src/admc/find_results.cpp +++ b/src/admc/find_results.cpp @@ -155,11 +155,11 @@ void FindResults::load(const QHash &search_results) { } QList> FindResults::get_selected_rows() const { - const QList selected_rows = view->current_view()->selectionModel()->selectedRows(); + const QList selected_indexes = view->get_selected_indexes(); QList> out; - for (const QModelIndex row_index : selected_rows) { + for (const QModelIndex row_index : selected_indexes) { const int row = row_index.row(); QList row_copy; @@ -276,7 +276,7 @@ void FindResults::enable_disable_helper(const bool disabled) { // First, hide all actions, then show whichever actions are // appropriate for current console selection void FindResults::update_actions_visibility() { - const QList selected_indexes = view->current_view()->selectionModel()->selectedRows(); + const QList selected_indexes = view->get_selected_indexes(); object_actions->update_actions_visibility(selected_indexes); // Always hide find action because opening a find dialog @@ -287,8 +287,8 @@ void FindResults::update_actions_visibility() { QHash FindResults::get_selected_dns_and_indexes() { QHash out; - const QList indexes = view->current_view()->selectionModel()->selectedRows(); - for (const QModelIndex &index : indexes) { + const QList selected_indexes = view->get_selected_indexes(); + for (const QModelIndex &index : selected_indexes) { const QString dn = index.data(ObjectRole_DN).toString(); out[dn] = QPersistentModelIndex(index); }