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; }