arx-deidentifier/arx

[BUG] Deselect All creates an invalid state

ZachHaber opened this issue · 2 comments

Describe the bug
If you use Deselect All to deselect all columns, it prevents you from moving on to the next step even after selecting a column.

In addition, if you deselect all manually (one at a time), you can still move forward to the next step, but there's an error in the preview.

To Reproduce
Steps to reproduce the behavior:

  1. Import data with < 100 columns (#399)
  2. Move to the "Columns" Step
  3. Right click and use "Deselect All"
  4. Right click a column and select it
  5. Observe that you still can't move to the next page
  6. To continue, past this bugged state, you have to go back a step to the "CSV" step
  7. Then go to the "Columns" step again
  8. Observe that the Next button is now enabled again

For the error state, you can do the same steps as before, but after step 6:

  1. disable the column you enabled
  2. press Next
  3. Observe the error state

Expected behavior
I expect that you shouldn't be able to move forward when there are no selected columns regardless of how they are deselected.

In addition, if you select a column again, you should be able to move forward.

Screenshots
Before using Deselect All
image
After using Deselect All
image
After Selecting a column
image


After deselecting all columns manually and moving to the next step:
image

ARX GUI (please complete the following information):

  • OS: Windows 10 (Version 10.0.19044 Build 19044)
  • Version 3.9.0

Thanks a lot for reporting this bug. We will fix this.

Should be fixed by commit 6275841. If not, please re-open.