Fix first entry is not selected when a search is performed (keepassxr…

…eboot#9868)
libklein · Sep 25, 2023 · cd7a53a · cd7a53a
1 parent 6b67f58
commit cd7a53a
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/src/gui/entry/EntryView.cpp b/src/gui/entry/EntryView.cpp
@@ -219,11 +219,12 @@ void EntryView::displaySearch(const QList<Entry*>& entries)
  m_model->setEntries(entries);
  header()->showSection(EntryModel::ParentGroup);
 
+ setFirstEntryActive();
+
  // Reset sort column to 'Group', overrides DatabaseWidgetStateSync
  m_sortModel->sort(EntryModel::ParentGroup, Qt::AscendingOrder);
  sortByColumn(EntryModel::ParentGroup, Qt::AscendingOrder);
 
- setFirstEntryActive();
  m_inSearchMode = true;
 }
 

diff --git a/tests/gui/TestGui.cpp b/tests/gui/TestGui.cpp
@@ -1082,6 +1082,13 @@ void TestGui::testSearch()
  QCOMPARE(groupView->currentGroup(), m_db->rootGroup());
  QVERIFY(!m_dbWidget->isSearchActive());
 
+ // check if first entry is selected after search
+ QTest::keyClicks(searchTextEdit, "some");
+ QTRY_VERIFY(m_dbWidget->isSearchActive());
+ QTRY_COMPARE(entryView->selectedEntries().length(), 1);
+ QModelIndex index_current = entryView->indexFromEntry(entryView->currentEntry());
+ QTRY_COMPARE(index_current.row(), 0);
+
  // Try to edit the first entry from the search view
  // Refocus back to search edit
  QTest::mouseClick(searchTextEdit, Qt::LeftButton);