forked from openlp/openlp
Forgot a few things
This commit is contained in:
parent
678a42263d
commit
bef7c9df13
@ -119,7 +119,8 @@ class EditSongForm(QtWidgets.QDialog, Ui_EditSongDialog, RegistryProperties):
|
|||||||
cache.append(obj.name)
|
cache.append(obj.name)
|
||||||
combo.setItemData(row, obj.id)
|
combo.setItemData(row, obj.id)
|
||||||
set_case_insensitive_completer(cache, combo)
|
set_case_insensitive_completer(cache, combo)
|
||||||
combo.setEditText('')
|
combo.setCurrentIndex(-1)
|
||||||
|
combo.setCurrentText('')
|
||||||
|
|
||||||
def _add_author_to_list(self, author, author_type):
|
def _add_author_to_list(self, author, author_type):
|
||||||
"""
|
"""
|
||||||
@ -353,7 +354,8 @@ class EditSongForm(QtWidgets.QDialog, Ui_EditSongDialog, RegistryProperties):
|
|||||||
self.authors_combo_box.setItemData(row, author.id)
|
self.authors_combo_box.setItemData(row, author.id)
|
||||||
self.authors.append(author.display_name)
|
self.authors.append(author.display_name)
|
||||||
set_case_insensitive_completer(self.authors, self.authors_combo_box)
|
set_case_insensitive_completer(self.authors, self.authors_combo_box)
|
||||||
self.authors_combo_box.setEditText('')
|
self.authors_combo_box.setCurrentIndex(-1)
|
||||||
|
self.authors_combo_box.setCurrentText('')
|
||||||
|
|
||||||
# Types
|
# Types
|
||||||
self.author_types_combo_box.clear()
|
self.author_types_combo_box.clear()
|
||||||
|
@ -106,4 +106,5 @@ class TestEditSongForm(TestCase, TestMixin):
|
|||||||
mocked_cache.append.assert_called_once_with('Charles')
|
mocked_cache.append.assert_called_once_with('Charles')
|
||||||
mocked_combo.setItemData.assert_called_once_with(0, 1)
|
mocked_combo.setItemData.assert_called_once_with(0, 1)
|
||||||
mocked_set_case_insensitive_completer.assert_called_once_with(mocked_cache, mocked_combo)
|
mocked_set_case_insensitive_completer.assert_called_once_with(mocked_cache, mocked_combo)
|
||||||
mocked_combo.setEditText.assert_called_once_with('')
|
mocked_combo.setCurrentIndex.assert_called_once_with(-1)
|
||||||
|
mocked_combo.setCurrentText.assert_called_once_with('')
|
||||||
|
Loading…
Reference in New Issue
Block a user