diff --git a/openlp/core/ui/projector/editform.py b/openlp/core/ui/projector/editform.py index aaf02eab2..1c5a6e14f 100644 --- a/openlp/core/ui/projector/editform.py +++ b/openlp/core/ui/projector/editform.py @@ -159,7 +159,7 @@ class ProjectorEditForm(QDialog, Ui_ProjectorEditForm): self.button_box.helpRequested.connect(self.help_me) self.button_box.rejected.connect(self.cancel_me) - def exec_(self, projector=None, test=False): + def exec_(self, projector=None): if projector is None: self.projector = Projector() self.new_projector = True @@ -168,8 +168,6 @@ class ProjectorEditForm(QDialog, Ui_ProjectorEditForm): self.new_projector = False self.retranslateUi(self) reply = QDialog.exec_(self) - if not test: - self.projector = None return reply @pyqtSlot() diff --git a/tests/interfaces/openlp_core_ui/test_projectoreditform.py b/tests/interfaces/openlp_core_ui/test_projectoreditform.py index 46cb73a36..c2b8ccce7 100644 --- a/tests/interfaces/openlp_core_ui/test_projectoreditform.py +++ b/tests/interfaces/openlp_core_ui/test_projectoreditform.py @@ -89,7 +89,7 @@ class TestProjectorEditForm(TestCase, TestMixin): with patch('openlp.core.ui.projector.editform.QDialog.exec_'): # WHEN: Calling edit form with no parameters - self.projector_form.exec_(test=True) + self.projector_form.exec_() item = self.projector_form.projector # THEN: Should be creating a new instance @@ -108,7 +108,7 @@ class TestProjectorEditForm(TestCase, TestMixin): with patch('openlp.core.ui.projector.editform.QDialog.exec_'): # WHEN: Calling edit form with existing projector instance - self.projector_form.exec_(projector=TEST1_DATA, test=True) + self.projector_form.exec_(projector=TEST1_DATA) item = self.projector_form.projector # THEN: Should be editing an existing entry