From af4e0aea7a8dd0a41c774f106403bbcbb36b8236 Mon Sep 17 00:00:00 2001
From: Mathieu Bastian <mathieu.bastian@gmail.com>
Date: Tue, 27 Oct 2015 15:35:57 +0100
Subject: [PATCH] Fix additional issue on ReportPanel

---
 .../gephi/desktop/importer/ReportPanel.java   | 19 +++++++++++--------
 1 file changed, 11 insertions(+), 8 deletions(-)

diff --git a/modules/DesktopImport/src/main/java/org/gephi/desktop/importer/ReportPanel.java b/modules/DesktopImport/src/main/java/org/gephi/desktop/importer/ReportPanel.java
index 69b5b2026..b56440165 100644
--- a/modules/DesktopImport/src/main/java/org/gephi/desktop/importer/ReportPanel.java
+++ b/modules/DesktopImport/src/main/java/org/gephi/desktop/importer/ReportPanel.java
@@ -444,13 +444,10 @@ public class ReportPanel extends javax.swing.JPanel {
 
     private void initProcessors() {
         int i = 0;
-        Collection<? extends Processor> processors = Lookup.getDefault().lookupAll(Processor.class);
-        for (Processor processor : processors) {
+        for (Processor processor : Lookup.getDefault().lookupAll(Processor.class)) {
             JRadioButton radio = new JRadioButton(processor.getDisplayName());
             radio.putClientProperty(PROCESSOR_KEY, processor);
             processorGroup.add(radio);
-            GridBagConstraints constraints = new GridBagConstraints(0, i++, 1, 1, 0, (i == processors.size() ? 1.0 : 0.0), GridBagConstraints.NORTHWEST, GridBagConstraints.NONE, new Insets(0, 0, 0, 0), 0, 0);
-            processorPanel.add(radio, constraints);
         }
     }
 
@@ -459,7 +456,8 @@ public class ReportPanel extends javax.swing.JPanel {
 
             @Override
             public void run() {
-                boolean first = true;
+
+                List<AbstractButton> validButtons = new ArrayList<AbstractButton>();
                 for (Enumeration<AbstractButton> enumeration = processorGroup.getElements(); enumeration.hasMoreElements();) {
                     AbstractButton radioButton = enumeration.nextElement();
                     Processor p = (Processor) radioButton.getClientProperty(PROCESSOR_KEY);
@@ -467,13 +465,18 @@ public class ReportPanel extends javax.swing.JPanel {
                     ProcessorUI pui = getProcessorUI(p);
                     if (pui != null) {
                         boolean isValid = pui.isValid(containers);
-                        radioButton.setVisible(isValid);
                         if (isValid) {
-                            radioButton.setSelected(first);
-                            first = false;
+                            validButtons.add(radioButton);
                         }
                     }
                 }
+
+                int i = 0;
+                for (AbstractButton radio : validButtons) {
+                    radio.setSelected(i == 0);
+                    GridBagConstraints constraints = new GridBagConstraints(0, i++, 1, 1, 0, (i == validButtons.size() ? 1.0 : 0.0), GridBagConstraints.NORTHWEST, GridBagConstraints.NONE, new Insets(0, 0, 0, 0), 0, 0);
+                    processorPanel.add(radio, constraints);
+                }
             }
         });
     }
-- 
GitLab