Skip to content

Commit

Permalink
Disable validation of experiment name on restart
Browse files Browse the repository at this point in the history
  • Loading branch information
frode-aarstad committed Aug 6, 2024
1 parent 7273944 commit 5652b19
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 0 deletions.
10 changes: 10 additions & 0 deletions src/ert/gui/ertwidgets/stringbox.py
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,8 @@ def __init__(
self._validation = ValidationSupport(self)
self._validator: Optional[ArgumentDefinition] = None
self._model = model
self._disable_validation = False

if placeholder_text:
self.setPlaceholderText(placeholder_text)
self.editingFinished.connect(self.stringBoxChanged)
Expand All @@ -53,6 +55,8 @@ def __init__(
self.modelChanged()

def validateString(self) -> None:
if self._disable_validation:
return
string_to_validate = str(self.text())
if not string_to_validate and self.placeholderText():
string_to_validate = self.placeholderText()
Expand Down Expand Up @@ -108,3 +112,9 @@ def isValid(self) -> bool:
@property
def get_text(self) -> str:
return self.text() if self.text() else self.placeholderText()

def enable_validation(self) -> None:
self._disable_validation = False

def disable_validation(self) -> None:
self._disable_validation = True
2 changes: 2 additions & 0 deletions src/ert/gui/simulation/multiple_data_assimilation_panel.py
Original file line number Diff line number Diff line change
Expand Up @@ -153,11 +153,13 @@ def update_experiment_name(self) -> None:
@Slot(bool)
def update_experiment_edit(self, checked: bool) -> None:
if checked:
self._experiment_name_field.disable_validation()
self._experiment_name_field.setText(
self._ensemble_selector.selected_ensemble.experiment.name
)
self._experiment_name_field.setEnabled(False)
else:
self._experiment_name_field.enable_validation()
self._experiment_name_field.clear()
self._experiment_name_field.setEnabled(True)

Expand Down

0 comments on commit 5652b19

Please sign in to comment.