1
0
mirror of https://github.com/altlinux/admc.git synced 2025-03-24 22:50:29 +03:00

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
This commit is contained in:
Dmitry Degtyarev 2021-04-07 18:22:05 +04:00
parent 87f54264a3
commit 38eef2ff3c

View File

@ -155,11 +155,11 @@ void FindResults::load(const QHash<QString, AdObject> &search_results) {
}
QList<QList<QStandardItem *>> FindResults::get_selected_rows() const {
const QList<QModelIndex> selected_rows = view->current_view()->selectionModel()->selectedRows();
const QList<QModelIndex> selected_indexes = view->get_selected_indexes();
QList<QList<QStandardItem *>> out;
for (const QModelIndex row_index : selected_rows) {
for (const QModelIndex row_index : selected_indexes) {
const int row = row_index.row();
QList<QStandardItem *> 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<QModelIndex> selected_indexes = view->current_view()->selectionModel()->selectedRows();
const QList<QModelIndex> 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<QString, QPersistentModelIndex> FindResults::get_selected_dns_and_indexes() {
QHash<QString, QPersistentModelIndex> out;
const QList<QModelIndex> indexes = view->current_view()->selectionModel()->selectedRows();
for (const QModelIndex &index : indexes) {
const QList<QModelIndex> selected_indexes = view->get_selected_indexes();
for (const QModelIndex &index : selected_indexes) {
const QString dn = index.data(ObjectRole_DN).toString();
out[dn] = QPersistentModelIndex(index);
}