r1229 - in trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing: actions actions/util model - Simexplorer-si-commits

25 Feb 2008

Author: tchemit
Date: 2008-02-25 14:06:59 +0000 (Mon, 25 Feb 2008)
New Revision: 1229

Modified:
   trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/ShowDetailTabAction.java
   trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/util/ShowListTabAbstractAction.java
   trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/model/ListTabModel.java
   trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/model/PaginationModel.java
Log:
javadoc

Modified: trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/ShowDetailTabAction.java
===================================================================

--- trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/ShowDetailTabAction.java	2008-02-25 14:05:03 UTC (rev 1228)
+++ trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/ShowDetailTabAction.java	2008-02-25 14:06:59 UTC (rev 1229)
@@ -22,8 +22,8 @@
 import fr.cemagref.simexplorer.is.entities.metadata.Version;
 import fr.cemagref.simexplorer.is.ui.swing.SimExplorerActionManager;
 import fr.cemagref.simexplorer.is.ui.swing.StorageServiceHelper;
+import fr.cemagref.simexplorer.is.ui.swing.actions.util.SelectedAction;
 import fr.cemagref.simexplorer.is.ui.swing.actions.util.ShowTabAbstractAction;
-import fr.cemagref.simexplorer.is.ui.swing.actions.util.SelectedAction;
 import fr.cemagref.simexplorer.is.ui.swing.model.DataEntityModel;
 import fr.cemagref.simexplorer.is.ui.swing.model.DetailTabModel;
 import fr.cemagref.simexplorer.is.ui.swing.model.EntityTreeNode;
@@ -45,11 +45,7 @@
 
 /**
  * Action pour afficher le tab de détail d'un LoggableElement.
- * <p/>
- * Les deux listeners (sur l'arbre de navigation et sur la liste des versions)
- * écoutent les changements provenant du model de détail afin
- * de pouvoir optimiser les rafraichissement d'ui.
- *
+ * 
  * @author chemit
  */
 @jaxx.runtime.builder.ActionConfig(
@@ -62,9 +58,17 @@
         hideActionText = true
 )
 public class ShowDetailTabAction extends ShowTabAbstractAction<JDetailTab, DetailTabModel> implements SelectedAction {
-    
+
     /** la donnée à afficher dans l'onglet de détail */
     protected DataEntityModel selected;
+    /** le listener sur le changement de noeud dans l'arbre de navigation */
+    protected NavigationSelectionListener navigationSelectionListener;
+    /** le listener sur les actions propres à l'arbre de navigation */
+    protected TreeActionUpdater treeActionUpdater;
+    /** le listener sur le changement de version */
+    protected VersionSelectionListener versionSelectionListener;
+    /** le listener sur le changement de fichier attaché */
+    protected AttachmentSelectionListener attachmentSelectionListener;
 
     private static final long serialVersionUID = 6643631041114311643L;
 
@@ -73,6 +77,11 @@
     }
 
     @Override
+    protected void setStatus(String status) {
+        // no log
+    }
+
+    @Override
     public String getI18nToolTipText() {
         return "simexplorer.action.showTab_detail.tooltip";
     }
@@ -100,7 +109,7 @@
                 // on doit retrouver l'élément à traiter à partir de la source
                 selected = findSelectedItem(e);
             }
-        }        
+        }
         boolean selectedUuid = selected != null && selected.getUuid() != null;
         if (!selectedUuid) {
             return false;
@@ -113,15 +122,10 @@
         detailModel.setSelectedItem(selected);
 
         // chargement de l'élément à partir du service
-        // délégué au model de selection        
+        // délégué au model de selection
         return detailModel.getSelectedItem().getLe(getContext()) != null;
     }
 
-    NavigationSelectionListener navigationSelectionListener;
-    TreeActionUpdater treeActionUpdater;
-    VersionSelectionListener versionSelectionListener;
-    AttachmentSelectionListener attachmentSelectionListener;
-
     protected void initFirstUsage(final JDetailTab ui, final DetailTabModel model) {
         // add a listener on navigation tree
         navigationSelectionListener = new NavigationSelectionListener();
@@ -151,28 +155,28 @@
     protected void updateUI(JDetailTab ui, DetailTabModel model) {
 
         // ajout de l'élément dans l'historique
-        SimExplorerActionManager.fireSelectedAction("historyAdd", e.getSource(),selected);
+        SimExplorerActionManager.fireSelectedAction("historyAdd", e.getSource(), selected);
     }
 
     @Override
     public void disposeUI() {
         super.disposeUI();
-        if (navigationSelectionListener!=null) {
+        if (navigationSelectionListener != null) {
             getTabUI().getNavigationTree().removeTreeSelectionListener(navigationSelectionListener);
-            navigationSelectionListener=null;
+            navigationSelectionListener = null;
         }
-        if (treeActionUpdater!=null) {
+        if (treeActionUpdater != null) {
             getTabUI().getNavigationTree().removeTreeSelectionListener(treeActionUpdater);
-            treeActionUpdater=null;
+            treeActionUpdater = null;
         }
-        if (versionSelectionListener!=null) {
+        if (versionSelectionListener != null) {
             getTabUI().getDetailVersions().removeItemListener(versionSelectionListener);
-            versionSelectionListener=null;
+            versionSelectionListener = null;
         }
 
-        if (attachmentSelectionListener!=null) {
+        if (attachmentSelectionListener != null) {
             getTabUI().getDetailAttachments().removeListSelectionListener(attachmentSelectionListener);
-            attachmentSelectionListener=null;
+            attachmentSelectionListener = null;
         }
     }
 

Modified: trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/util/ShowListTabAbstractAction.java
===================================================================
--- trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/util/ShowListTabAbstractAction.java	2008-02-25 14:05:03 UTC (rev 1228)
+++ trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/util/ShowListTabAbstractAction.java	2008-02-25 14:06:59 UTC (rev 1229)
@@ -50,11 +50,13 @@
 
     /** flag pour indiquer si la liste est locale ou remote */
     protected boolean remote;
+    /** le listener sur le changement de meta selectionnée */
+    protected ListMetaSelectionListener listMetaSelectionListener;
+    /** le listener sur le changement de tri */
+    protected TableRowSorterListener rowSorterListener;
+    /** le trieur de table */
+    protected MyTableRowSorter rowSorter;
 
-    ListMetaSelectionListener listMetaSelectionListener;
-    TableRowSorterListener rowSorterListener;
-    MyTableRowSorter rowSorter;
-
     public ShowListTabAbstractAction(String name, boolean remote) {
         super(name);
         this.remote = remote;
@@ -64,7 +66,7 @@
     protected void setStatus(String status) {
         // no log
     }
-    
+
     @Override
     protected boolean beforeAction(ActionEvent e) throws Exception {
         super.beforeAction(e);
@@ -81,17 +83,17 @@
     @Override
     public void disposeUI() {
         super.disposeUI();
-        if (listMetaSelectionListener!=null) {
+        if (listMetaSelectionListener != null) {
             getTabUI().getTable().getSelectionModel().removeListSelectionListener(listMetaSelectionListener);
-            listMetaSelectionListener=null;
+            listMetaSelectionListener = null;
         }
-        if (rowSorterListener!=null) {
+        if (rowSorterListener != null) {
             getTabUI().getTable().getRowSorter().removeRowSorterListener(rowSorterListener);
-            rowSorterListener=null;
+            rowSorterListener = null;
         }
-        if (rowSorter!=null) {
+        if (rowSorter != null) {
             getTabUI().getTable().setRowSorter(rowSorter);
-            rowSorter=null;
+            rowSorter = null;
         }
     }
 
@@ -113,7 +115,7 @@
             @Override
             public void keyReleased(KeyEvent e) {
                 // rafraichissement des actions de recherche
-                getListUpdater( getModel().isRemote()).refreshSearchActions();
+                getListUpdater(getModel().isRemote()).refreshSearchActions();
             }
         });
 
@@ -132,7 +134,7 @@
         rowSorter.addRowSorterListener(rowSorterListener);
 
         // init updater listens the model
-        getListUpdater( model.isRemote()).initListeners();
+        getListUpdater(model.isRemote()).initListeners();
 
         // init model
         model.initTabModel(remote);

Modified: trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/model/ListTabModel.java
===================================================================
--- trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/model/ListTabModel.java	2008-02-25 14:05:03 UTC (rev 1228)
+++ trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/model/ListTabModel.java	2008-02-25 14:06:59 UTC (rev 1229)
@@ -77,7 +77,6 @@
         this.selectedIndex = selectedIndex;
         if (selectedIndex!=-1) {
             synch(isRemote(),get(selectedIndex));
-            
         }
         firePropertyChange(SELECTED_ROW_PROPERTY_CHANGED, oldSelectedIndex, selectedIndex);
     }
@@ -232,7 +231,7 @@
                 query.getQuery(),
                 (int) page.getFirstIndex(),
                 page.getWidthInt(),
-                query.getSortColumn().getColumn(),
+                query.getSortColumn(),
                 query.isSortAscending()
         );
 

Modified: trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/model/PaginationModel.java
===================================================================
--- trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/model/PaginationModel.java	2008-02-25 14:05:03 UTC (rev 1228)
+++ trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/model/PaginationModel.java	2008-02-25 14:06:59 UTC (rev 1229)
@@ -129,6 +129,9 @@
     }
 
     void setWidth(SimplePaginationEnum width) {
+        if (width==null) {
+            throw new NullPointerException("can not set a null width");
+        }
         this.width = width;
         this.pageNumber = 0;
     }

    

r1229 - in trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing: actions actions/util model

tchemit＠users.labs.libre-entreprise.org

tags

participants (1)