#52: always execute disableRequiredPropCheckBoxes() when updating pan…

…el list in the report UI
sepinf-inc · Feb 13, 2023 · ad2034f · ad2034f
1 parent 31dae56
commit ad2034f
Showing 1 changed file with 8 additions and 6 deletions.
diff --git a/iped-app/src/main/java/iped/app/ui/columns/ColumnsSelectReportUI.java b/iped-app/src/main/java/iped/app/ui/columns/ColumnsSelectReportUI.java
@@ -76,19 +76,14 @@ protected ColumnsSelectReportUI() {
         } else {
             columnsManager.checkOnlySelectedProperties(Arrays.asList(basicReportProps));
         }
-        disableRequiredPropertiesCheckBoxes();
         updatePanelList();
     }
 
     @Override
     public void actionPerformed(ActionEvent e) {
         super.actionPerformed(e);
-        if (e.getSource().equals(selectVisibleButton)) {
-            disableRequiredPropertiesCheckBoxes();
-            updatePanelList();
-        } else if (e.getSource().equals(clearButton)) {
+        if (e.getSource().equals(clearButton)) {
             columnsManager.uncheckAllProperties();
-            disableRequiredPropertiesCheckBoxes();
             updatePanelList();
         } else if (e.getSource() instanceof JCheckBox || e.getSource().equals(selectVisibleButton)) {
             if (columnsManager.getSelectedProperties().size() > PROPERTIES_LIMIT_NUM) {
@@ -109,6 +104,13 @@ public void actionPerformed(ActionEvent e) {
         }
     }
 
+    @Override
+    protected void updatePanelList() {
+        disableRequiredPropertiesCheckBoxes();
+        super.updatePanelList();        
+    }
+
+
     public void disableRequiredPropertiesCheckBoxes() {
         columnsManager.allCheckBoxesState.put(IndexItem.ID_IN_SOURCE, new CheckBoxState(true, false));
         columnsManager.allCheckBoxesState.put(BasicProps.PATH, new CheckBoxState(true, false));