Skip to content

Commit 9e9de09

Browse files
Merge pull request #1601 from bug-or-feature/bug-1521-roll-status-msg
Fixes #1521
2 parents d7746d9 + 2ab8224 commit 9e9de09

File tree

1 file changed

+4
-6
lines changed

1 file changed

+4
-6
lines changed

sysproduction/interactive_update_roll_status.py

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -624,7 +624,7 @@ def get_roll_state_required(
624624
)
625625
print("")
626626
if okay_to_change is None:
627-
return no_change_required
627+
return roll_data.original_roll_status
628628

629629
if okay_to_change:
630630
# happy
@@ -635,7 +635,8 @@ def get_roll_state_required(
635635
continue
636636
else:
637637
print("No change")
638-
return no_change_required
638+
return roll_data.original_roll_status
639+
return None
639640

640641

641642
def setup_roll_data_with_state_reporting(
@@ -690,10 +691,7 @@ def modify_roll_state(
690691
roll_state_required: RollState,
691692
confirm_adjusted_price_change: bool = True,
692693
):
693-
roll_state_is_unchanged = (roll_state_required is no_change_required) or (
694-
roll_state_required is original_roll_state
695-
)
696-
if roll_state_is_unchanged:
694+
if roll_state_required == original_roll_state:
697695
return
698696

699697
if original_roll_state is no_open_state:

0 commit comments

Comments
 (0)