Skip to content

Comments

Fix alternative criteria reordering when adding second alternative#63

Draft
AAClause wants to merge 1 commit intodevelopfrom
bugfix/second_alternative
Draft

Fix alternative criteria reordering when adding second alternative#63
AAClause wants to merge 1 commit intodevelopfrom
bugfix/second_alternative

Conversation

@AAClause
Copy link
Member

  • Explicitly show sequence order controls when nbAlternatives > 1
  • Use valid default index (nbAlternatives - 1) instead of out-of-range value
  • Add bounds validation for criteriaIndex before setting selection

Fixes issue where entering data for a new alternative and changing its
order would cause the data to be lost, duplicating the existing alternative's
data instead.

Fixes #62

- Explicitly show sequence order controls when nbAlternatives > 1
- Use valid default index (nbAlternatives - 1) instead of out-of-range value
- Add bounds validation for criteriaIndex before setting selection

Fixes issue where entering data for a new alternative and changing its
order would cause the data to be lost, duplicating the existing alternative's
data instead.

Fixes #62
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Adding a new alternative to a rule with one existing alternative loses entered data when reordering

1 participant