[izpack-changes] r2022 - in izpack-src/trunk: . src/lib/com/izforge/izpack/panels src/lib/com/izforge/izpack/util

[noreply at berlios.de]

Author: jponge
Date: 2008-01-26 16:52:21 +0100 (Sat, 26 Jan 2008)
New Revision: 2022

Modified:
   izpack-src/trunk/Versions.txt
   izpack-src/trunk/src/lib/com/izforge/izpack/panels/PacksPanelBase.java
   izpack-src/trunk/src/lib/com/izforge/izpack/panels/TreePacksPanel.java
   izpack-src/trunk/src/lib/com/izforge/izpack/util/Log.java
Log:
PacksPanelBase: the packs would be selected / deselected on any list event (e.g., by navigating using the keys). This funky behavior has been fixed (Julien Ponge).

Modified: izpack-src/trunk/Versions.txt
===================================================================
--- izpack-src/trunk/Versions.txt	2008-01-25 22:27:53 UTC (rev 2021)
+++ izpack-src/trunk/Versions.txt	2008-01-26 15:52:21 UTC (rev 2022)
@@ -101,6 +101,8 @@
 - eng.xml: Grammatical corrections, added UserPathPanel headline, messages, and captions,
   added UserInputPanel.dir.* messages and captions, added UserInputPanel.file.* messages and captions.
   (Need to add/test with alternative lang packs)  (Jeff Gordon)
+- PacksPanelBase: the packs would be selected / deselected on any list event (e.g., by navigating using the keys).
+  This funky behavior has been fixed (Julien Ponge).
 
 
   > 3.10.2 (build 2007.05.11)

Modified: izpack-src/trunk/src/lib/com/izforge/izpack/panels/PacksPanelBase.java
===================================================================
--- izpack-src/trunk/src/lib/com/izforge/izpack/panels/PacksPanelBase.java	2008-01-25 22:27:53 UTC (rev 2021)
+++ izpack-src/trunk/src/lib/com/izforge/izpack/panels/PacksPanelBase.java	2008-01-26 15:52:21 UTC (rev 2022)
@@ -280,16 +280,14 @@
     public void valueChanged(ListSelectionEvent e)
     {
         VariableSubstitutor vs = new VariableSubstitutor(idata.getVariables());
-
         int i = packsTable.getSelectedRow();
         if (i < 0) return;
         
         // toggle the value stored in the packsModel
-        Integer checked = (Integer)packsModel.getValueAt(i, 0);
-        if (checked.intValue() == 0) {
-          packsModel.setValueAt(new Integer(1), i, 0);
-        } else if (checked.intValue() == 1) {
-          packsModel.setValueAt(new Integer(0), i, 0);
+        if (e.getValueIsAdjusting())
+        {
+            Integer checked = (Integer) packsModel.getValueAt(i, 0);
+            packsModel.setValueAt(checked, i, 0);
         }
         
         // Operations for the description

Modified: izpack-src/trunk/src/lib/com/izforge/izpack/panels/TreePacksPanel.java
===================================================================
--- izpack-src/trunk/src/lib/com/izforge/izpack/panels/TreePacksPanel.java	2008-01-25 22:27:53 UTC (rev 2021)
+++ izpack-src/trunk/src/lib/com/izforge/izpack/panels/TreePacksPanel.java	2008-01-26 15:52:21 UTC (rev 2022)
@@ -595,7 +595,6 @@
    /**
     * Updates a value for pack in PacksModel with data from a checkbox node
     * 
-    * @param id pack id
     * @param cbnode This is the checkbox node which contains model values
     */
    public void setModelValue(CheckBoxNode cbnode)

Modified: izpack-src/trunk/src/lib/com/izforge/izpack/util/Log.java
===================================================================
--- izpack-src/trunk/src/lib/com/izforge/izpack/util/Log.java	2008-01-25 22:27:53 UTC (rev 2021)
+++ izpack-src/trunk/src/lib/com/izforge/izpack/util/Log.java	2008-01-26 15:52:21 UTC (rev 2022)
@@ -205,7 +205,6 @@
         // get the debug setting
         // ----------------------------------------------------
         String temp = System.getProperty(DEBUG_SWITCH);
-        System.out.println("temp: " + temp);
         if ((temp != null) && (temp.toUpperCase().equals("ON")))
         {
             debugActive = true;