ROU-4519: Issue with dropdown column with dependency triggering OnCellValueChange event twice

[OS-giulianasilva]

This PR is for fix the dropdown child triggering the OnCellValueChange more than once.

What was happening

• When the Grid has a dropdown column with dependency, the OnCellValueChange event was triggered twice when the parent cell value changed.

What was done

• Add validation to check if currentValue is different than empty when adding the child undo action and triggering the OnCellValueChange event.
  • If the currentValue is already empty, we don't need to set the cell value to empty again and add it to the undo stack.
• Change the way we get the last parent cell's undo action on the undo stack.
  • We want to ensure that the child's undo action is added to the last parent's undo action in the undo stack.
• Disabled the trigger in the validateCell method to trigger the OnCellValueChange event just once.
  • The onCellValueChange is already explicitly being triggered in the _parentCellValueChangeHandler, We don't need to do this twice.
• Trigger the OnCellValueChange with the original value instead of currentValue.
  • The OnCellValueChange event must return the original value and not the previous cell value.
• Added an else statement in ValidationMark's validateCell method to validate the cell event even when not triggering the columns event.

Test Steps

1. Go to a page containing a Grid with dropdown dependency.
2. Change the parent cell value
3. Check that the child's OnCellValueChange is triggered just once for the child dropdown.
4. Press Ctrl+Z to undo
5. Check that the child's OnCellValueChange is triggered just once for the child dropdown.

Screenshots

Before the changes:

After the changes:

Checklist

• tested locally
• documented the code
• clean all warnings and errors of eslint
• requires changes in OutSystems (if so, provide a module with changes)
• requires new sample page in OutSystems (if so, provide a module with changes)

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication