February 2008 - Simexplorer-si-commits - mailman.cloud.codelutin.com

r1105 - trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions
by tchemit＠users.labs.libre-entreprise.org Feb. 18, 2008

Feb. 18, 2008

Author: tchemit Date: 2008-02-18 23:13:30 +0000 (Mon, 18 Feb 2008) New Revision: 1105 Modified: trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/DeleteElementAction.java Log: m?\195?\169thode delete presque ok, reste juste ?\195?\160 supprimer l'?\195?\169l?\195?\169ment de l'historique de l'onglet detail et a ne cacher l'onglet synchronize uniquement si l'objet concern?\195?\169 y est comme racine Modified: trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/DeleteElementAction.java =================================================================== --- trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/DeleteElementAction.java 2008-02-18 23:12:36 UTC (rev 1104) +++ trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/DeleteElementAction.java 2008-02-18 23:13:30 UTC (rev 1105) @@ -25,10 +25,10 @@ import fr.cemagref.simexplorer.is.ui.swing.model.DataEntityModel; import fr.cemagref.simexplorer.is.ui.swing.model.ListTabModel; import fr.cemagref.simexplorer.is.ui.swing.model.ListTableModel; +import fr.cemagref.simexplorer.is.ui.swing.model.SimExplorerTabModel; import static org.codelutin.i18n.I18n._; import javax.swing.JOptionPane; -import java.awt.Component; import java.awt.event.ActionEvent; /** @@ -64,54 +64,81 @@ if (!super.beforeAction(e)) { return false; } - // on doit retrouver l'élément à traiter à partir de la source - SimExplorerTab sourceTab = SimExplorerTab.findTab((Component) e.getSource()); - log.info("incoming tab " + sourceTab); - if (sourceTab == null) { - //fatal error - log.warn("could not find a tab for source " + e.getSource()); - return false; + if (selected == null) { + // on doit retrouver l'élément à traiter à partir de la source + selected = findSelectedItem(e); } - selected = sourceTab.getModel().getSelectedItem(); + boolean selectedUuid = selected != null && selected.getUuid() != null; + if (selectedUuid) { - return selected != null; + // demande de confirmation avant d'effacer + int result = JOptionPane.showConfirmDialog(null, _("simexplorer.confirm.delete", getDeleteMessage()), "delete", JOptionPane.YES_NO_OPTION); + if (result != JOptionPane.OK_OPTION) { + // do nothing + return false; + } + } + return selectedUuid; } + @Override protected void doAction(ActionEvent e) throws Exception { super.doAction(e); log.info("selected " + selected); - // demande de confirmation avant d'effacer - int result = JOptionPane.showConfirmDialog(null, _("simexplorer.confirm.delete", getDeleteMessage()), "delete", JOptionPane.YES_NO_OPTION); - if (result != JOptionPane.OK_OPTION) { - // do nothing - return; - } + // effectue la suppression StorageServiceHelper.deleteElement(getContext(), selected.isRemote(), selected.getUuid(), selected.getVersion().toString()); + + //TODO supprimer l'élément de l'historique de detail } @Override public void updateUI() { // on doit invalider les données apres un export - ListTabModel model; + ListTabModel listTabModel; JListTab ui; SimExplorerTab tab; if (selected.isRemote()) { // reinit de la liste locale tab = SimExplorerTab.remote; - model = (ListTabModel) tab.getModel(); + listTabModel = (ListTabModel) tab.getModel(); ui = (JListTab) tab.getUI(); } else { // reninit de la liste remote tab = SimExplorerTab.local; - model = (ListTabModel) tab.getModel(); + listTabModel = (ListTabModel) tab.getModel(); ui = (JListTab) tab.getUI(); } - updateList(model, ui, tab); + updateList(listTabModel, ui, tab); + + // update detail model + tab = SimExplorerTab.detail; + SimExplorerTabModel model = tab.getModel(); + DataEntityModel item = model.getSelectedItem(); + if (tab.isTabVisible(getTabContainer()) && item.getUuid() != null && item.getUuid().equals(selected.getUuid())) { + // on supprimer la donnée + model.synch(null); + // on doit supprimer la vue + tab.closeTab(getTabContainer()); + } + + // update synchronize model + tab = SimExplorerTab.synchronize; + //model = tab.getModel(); + //TODO verifier si l'élément se trouve dans la synchro, pour le moment + //TODO on ferme toujours la synchro + //item = model.getSelectedItem(); + //if (item.getUuid() != null && item.getUuid().equals(selected.getUuid())) { + // on supprimer la donnée + //model.synch(null); + // on doit supprimer la vue + tab.closeTab(getTabContainer()); + //} } private void updateList(ListTabModel model, JListTab ui, SimExplorerTab tab) { + model.setPagination(null); model.updateTabModel(getContext(), (ListTableModel) ui.getTable().getModel()); tab.updateUI(); }

1 0

r1104 - trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions
by tchemit＠users.labs.libre-entreprise.org Feb. 18, 2008

Feb. 18, 2008

Author: tchemit Date: 2008-02-18 23:12:36 +0000 (Mon, 18 Feb 2008) New Revision: 1104 Modified: trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/ImportAction.java Log: utilisation m?\195?\169thode updateUI au lieu de afterAction Modified: trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/ImportAction.java =================================================================== --- trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/ImportAction.java 2008-02-18 23:11:53 UTC (rev 1103) +++ trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/ImportAction.java 2008-02-18 23:12:36 UTC (rev 1104) @@ -81,8 +81,8 @@ } @Override - protected void afterAction(ActionEvent e) throws Exception { - super.afterAction(e); + public void updateUI() { + super.updateUI(); //TODO should invalidate the 'remote' list //TODO look if detail history is touch by this modification //TODO look if synchronize is touched by this modification @@ -90,7 +90,6 @@ @Override protected void clear() { - super.clear(); f = null; remote = null; // this is a multi-tab action, must reset tab after each action

1 0

r1103 - trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions
by tchemit＠users.labs.libre-entreprise.org Feb. 18, 2008

Feb. 18, 2008

Author: tchemit Date: 2008-02-18 23:11:53 +0000 (Mon, 18 Feb 2008) New Revision: 1103 Modified: trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/QuitAction.java Log: utilisation m?\195?\169thode doAction au lieu de actionPerformed Modified: trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/QuitAction.java =================================================================== --- trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/QuitAction.java 2008-02-18 23:11:11 UTC (rev 1102) +++ trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/QuitAction.java 2008-02-18 23:11:53 UTC (rev 1103) @@ -43,9 +43,9 @@ } @Override - public void actionPerformed(java.awt.event.ActionEvent e) { - super.actionPerformed(e); - //TODO Fermer tous les tabs + protected void doAction(java.awt.event.ActionEvent e) throws Exception { + //TODO Fermer tous les tabs + //TODO se deconnecter,... getMainUI().dispose(); } }

1 0

r1102 - trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions
by tchemit＠users.labs.libre-entreprise.org Feb. 18, 2008

Feb. 18, 2008

Author: tchemit Date: 2008-02-18 23:11:11 +0000 (Mon, 18 Feb 2008) New Revision: 1102 Modified: trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/ConnectAction.java trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/UnconnectAction.java Log: utilisation m?\195?\169thode updateUI au lieu de afterAction Modified: trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/ConnectAction.java =================================================================== --- trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/ConnectAction.java 2008-02-18 23:10:42 UTC (rev 1101) +++ trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/ConnectAction.java 2008-02-18 23:11:11 UTC (rev 1102) @@ -87,11 +87,6 @@ } } - @Override - protected void afterAction(ActionEvent e) throws Exception { - updateUI(); - } - /** * Rafraichit les uis après un changement de status de connexion de l'utilisateur. * <p/> @@ -130,11 +125,6 @@ //}); } - @Override - protected void clear() { - super.clear(); - } - protected void initLoginUI() { if (ui == null) { loginUIHandler = new MyLoginUIHandler(); Modified: trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/UnconnectAction.java =================================================================== --- trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/UnconnectAction.java 2008-02-18 23:10:42 UTC (rev 1101) +++ trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/UnconnectAction.java 2008-02-18 23:11:11 UTC (rev 1102) @@ -104,11 +104,6 @@ } } - @Override - protected void afterAction(ActionEvent e) throws Exception { - updateUI(); - } - /** * Rafraichit les uis après un changement de status de connexion de l'utilisateur. * <p/> @@ -169,7 +164,6 @@ @Override protected void clear() { - super.clear(); history = null; } } \ No newline at end of file

1 0

r1101 - trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions
by tchemit＠users.labs.libre-entreprise.org Feb. 18, 2008

Feb. 18, 2008

Author: tchemit Date: 2008-02-18 23:10:42 +0000 (Mon, 18 Feb 2008) New Revision: 1101 Modified: trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/CollapseAllAction.java trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/ExpandAllAction.java Log: utilisation m?\195?\169thode updateUI au lieu de afterAction Modified: trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/CollapseAllAction.java =================================================================== --- trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/CollapseAllAction.java 2008-02-18 23:09:30 UTC (rev 1100) +++ trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/CollapseAllAction.java 2008-02-18 23:10:42 UTC (rev 1101) @@ -18,9 +18,9 @@ * ##% */ package fr.cemagref.simexplorer.is.ui.swing.actions; +import fr.cemagref.simexplorer.is.ui.swing.EntityTreeNodeHelper; import fr.cemagref.simexplorer.is.ui.swing.JDetailTab; import fr.cemagref.simexplorer.is.ui.swing.JSynchronizeTab; -import fr.cemagref.simexplorer.is.ui.swing.EntityTreeNodeHelper; import fr.cemagref.simexplorer.is.ui.swing.actions.util.SimExplorerAbstractTabAction; import fr.cemagref.simexplorer.is.ui.swing.model.EntityTreeNode; @@ -84,15 +84,14 @@ } @Override - protected void afterAction(ActionEvent e) throws Exception { - super.afterAction(e); - tree.invalidate(); - getMainUI().repaint(); + public void updateUI() { + super.updateUI(); + tree.invalidate(); + getMainUI().repaint(); } @Override protected void clear() { - super.clear(); tree = null; node = null; } Modified: trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/ExpandAllAction.java =================================================================== --- trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/ExpandAllAction.java 2008-02-18 23:09:30 UTC (rev 1100) +++ trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/ExpandAllAction.java 2008-02-18 23:10:42 UTC (rev 1101) @@ -86,15 +86,14 @@ } @Override - protected void afterAction(ActionEvent e) throws Exception { - super.afterAction(e); - tree.invalidate(); + public void updateUI() { + super.updateUI(); + tree.invalidate(); getMainUI().repaint(); } @Override protected void clear() { - super.clear(); tree = null; node = null; }

1 0

r1100 - trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions
by tchemit＠users.labs.libre-entreprise.org Feb. 18, 2008

Feb. 18, 2008

Author: tchemit Date: 2008-02-18 23:09:30 +0000 (Mon, 18 Feb 2008) New Revision: 1100 Modified: trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/ChangeSizorAction.java Log: fixbug: faire synch(null) supprime aussi la propri?\195?\169t?\195?\169 remote du mod?\195?\168le ce qui n'est pas souhait?\195?\169 Modified: trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/ChangeSizorAction.java =================================================================== --- trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/ChangeSizorAction.java 2008-02-18 23:08:15 UTC (rev 1099) +++ trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/ChangeSizorAction.java 2008-02-18 23:09:30 UTC (rev 1100) @@ -18,10 +18,10 @@ * ##% */ package fr.cemagref.simexplorer.is.ui.swing.actions; +import fr.cemagref.simexplorer.is.entities.metadata.MetaData; import fr.cemagref.simexplorer.is.ui.swing.JListTab; import fr.cemagref.simexplorer.is.ui.swing.actions.util.SimExplorerAbstractTabAction; import fr.cemagref.simexplorer.is.ui.swing.model.ListTabModel; -import fr.cemagref.simexplorer.is.ui.swing.model.PaginationModel; import fr.cemagref.simexplorer.is.ui.swing.model.ListTableModel; import javax.swing.Action; @@ -44,20 +44,18 @@ private static final long serialVersionUID = 5137512273375524199L; protected Integer newWidth; - + protected ListTabModel model; public ChangeSizorAction(String name) { super(name); } @Override protected boolean beforeAction(ActionEvent e) throws Exception { - JComboBox jComboBox = getComboBox(); if (!jComboBox.isEnabled()) { return false; } - - ListTabModel model = (ListTabModel) getModel(); + model = (ListTabModel) getModel(); Integer oldIndex = model.getPagination().getWidth(); Object o = jComboBox.getSelectedItem(); if (o != null) { @@ -68,22 +66,16 @@ @Override protected void doAction(ActionEvent e) throws Exception { - - final ListTabModel model = (ListTabModel) getModel(); - final PaginationModel pagination = model.getPagination(); - - // pour le moment, chaque retaille de fenetre repositionne - // la liste à zero pour simplifier - pagination.setWidth(newWidth); - pagination.setFirstIndex(0); - model.synch(null); // on sauvegarde dans la config la nouvelle valeur - // la valeur sera utilisée apres chaque action de recherche - if (model.isRemote()) { + boolean remote = model.isRemote(); + if (remote) { getContext().getConfig().setRemoteSizor(newWidth); } else { getContext().getConfig().setLocalSizor(newWidth); } + // on supprime la pagination actuelle, elle sera recalculé dans l'update + model.setPagination(null); + model.synch(remote, (MetaData) null); // update model model.updateTabModel(getContext(), (ListTableModel) ((JListTab) getUI()).getTable().getModel()); } @@ -100,6 +92,7 @@ // multi-tab action setTab(null); newWidth = null; + model=null; } protected JComboBox getComboBox() {

1 0

r1099 - trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions
by tchemit＠users.labs.libre-entreprise.org Feb. 18, 2008

Feb. 18, 2008

Author: tchemit Date: 2008-02-18 23:08:15 +0000 (Mon, 18 Feb 2008) New Revision: 1099 Modified: trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/GoLastPageAction.java trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/GoNextPageAction.java trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/GoPreviousPageAction.java Log: fixbug : l'action de changement de page s'applique bien d?\195?\169sormais en fonction de l'onglet de l'action et non pas uniquement sur l'onglet s?\195?\169lectionn?\195?\169 (n?\195?\169cessaire pour modifier un onglet de liste lorsque celui-ci n'est pas s?\195?\169lectionn?\195?\169 : delete, export,...) Modified: trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/GoLastPageAction.java =================================================================== --- trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/GoLastPageAction.java 2008-02-18 23:07:15 UTC (rev 1098) +++ trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/GoLastPageAction.java 2008-02-18 23:08:15 UTC (rev 1099) @@ -18,6 +18,7 @@ * ##% */ package fr.cemagref.simexplorer.is.ui.swing.actions; +import fr.cemagref.simexplorer.is.ui.swing.SimExplorerTab; import fr.cemagref.simexplorer.is.ui.swing.model.ListTabModel; import java.awt.event.ActionEvent; @@ -46,14 +47,15 @@ @Override protected boolean beforeAction(ActionEvent e) throws Exception { - if (!getUIObject((String) getValue(ACTION_COMMAND_KEY)).isEnabled()) { + SimExplorerTab myTab = findTab(e); + if (myTab == null) { return false; } - ListTabModel model = (ListTabModel) getModel(); + ListTabModel model = (ListTabModel) myTab.getModel(); Integer oldPage = model.getPagination().getCurrentPage(); boolean b = oldPage < model.getPagination().getNbPages(); if (b) { - newPage = getChangePageComboBox().getItemCount() - 1; + newPage = model.getPagination().getNbPages() - 1; } return b; } Modified: trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/GoNextPageAction.java =================================================================== --- trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/GoNextPageAction.java 2008-02-18 23:07:15 UTC (rev 1098) +++ trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/GoNextPageAction.java 2008-02-18 23:08:15 UTC (rev 1099) @@ -19,6 +19,7 @@ package fr.cemagref.simexplorer.is.ui.swing.actions; import fr.cemagref.simexplorer.is.ui.swing.model.ListTabModel; +import fr.cemagref.simexplorer.is.ui.swing.SimExplorerTab; import java.awt.event.ActionEvent; @@ -46,14 +47,15 @@ @Override protected boolean beforeAction(ActionEvent e) throws Exception { - if (!getUIObject((String) getValue(ACTION_COMMAND_KEY)).isEnabled()) { + SimExplorerTab myTab = findTab(e); + if (myTab==null) { return false; } - ListTabModel model = (ListTabModel) getModel(); + ListTabModel model = (ListTabModel) myTab.getModel(); Integer oldPage = model.getPagination().getCurrentPage(); boolean b = oldPage < model.getPagination().getNbPages(); if (b) { - newPage = getChangePageComboBox().getSelectedIndex() + 1; + newPage = oldPage+1; } return b; } Modified: trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/GoPreviousPageAction.java =================================================================== --- trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/GoPreviousPageAction.java 2008-02-18 23:07:15 UTC (rev 1098) +++ trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/GoPreviousPageAction.java 2008-02-18 23:08:15 UTC (rev 1099) @@ -19,6 +19,7 @@ package fr.cemagref.simexplorer.is.ui.swing.actions; import fr.cemagref.simexplorer.is.ui.swing.model.ListTabModel; +import fr.cemagref.simexplorer.is.ui.swing.SimExplorerTab; import java.awt.event.ActionEvent; @@ -46,10 +47,11 @@ @Override protected boolean beforeAction(ActionEvent e) throws Exception { - if (!getUIObject((String) getValue(ACTION_COMMAND_KEY)).isEnabled()) { + SimExplorerTab myTab = findTab(e); + if (myTab==null) { return false; } - ListTabModel model = (ListTabModel) getModel(); + ListTabModel model = (ListTabModel) myTab.getModel(); Integer oldPage = model.getPagination().getCurrentPage(); boolean b = oldPage > 0; if (b) {

1 0

r1098 - trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions
by tchemit＠users.labs.libre-entreprise.org Feb. 18, 2008

Feb. 18, 2008

Author: tchemit Date: 2008-02-18 23:07:15 +0000 (Mon, 18 Feb 2008) New Revision: 1098 Modified: trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/ChangePageAction.java Log: fixbug : l'action de changement de page s'applique bien d?\195?\169sormais en fonction de l'onglet de l'action et non pas uniquement sur l'onglet s?\195?\169lectionn?\195?\169 (n?\195?\169cessaire pour modifier un onglet de liste lorsque celui-ci n'est pas s?\195?\169lectionn?\195?\169 : delete, export,...) Modified: trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/ChangePageAction.java =================================================================== --- trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/ChangePageAction.java 2008-02-18 23:06:02 UTC (rev 1097) +++ trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/ChangePageAction.java 2008-02-18 23:07:15 UTC (rev 1098) @@ -20,6 +20,7 @@ import fr.cemagref.simexplorer.is.ui.swing.JListTab; import fr.cemagref.simexplorer.is.ui.swing.SimExplorerActionManager; +import fr.cemagref.simexplorer.is.ui.swing.SimExplorerTab; import fr.cemagref.simexplorer.is.ui.swing.actions.util.SimExplorerAbstractTabAction; import fr.cemagref.simexplorer.is.ui.swing.model.ListTabModel; import fr.cemagref.simexplorer.is.ui.swing.model.PaginationModel; @@ -32,6 +33,9 @@ import javax.swing.JComponent; import java.awt.event.ActionEvent; +import jaxx.runtime.swing.JAXXTab; +import jaxx.runtime.JAXXObject; + /** * Action pour lancer une recherche d'application * @@ -56,11 +60,15 @@ @Override protected boolean beforeAction(ActionEvent e) throws Exception { - JComboBox jComboBox = getChangePageComboBox(); + SimExplorerTab myTab = findTab(e); + if (myTab==null) { + return false; + } + JComboBox jComboBox = getChangePageComboBox(myTab.getUI()); if (jComboBox==null || !jComboBox.isEnabled()) { return false; } - ListTabModel model = (ListTabModel) getModel(); + ListTabModel model = (ListTabModel) myTab.getModel(); Integer oldPage = model.getPagination().getCurrentPage(); Object o = jComboBox.getSelectedItem(); if (o != null) { @@ -98,8 +106,8 @@ newPage = null; } - protected JComboBox getChangePageComboBox() { - JComponent uiObject = getUIObject((String) getValue(Action.ACTION_COMMAND_KEY)); + protected JComboBox getChangePageComboBox(JAXXTab myTab) { + JComponent uiObject = getUIObject((String) getValue(Action.ACTION_COMMAND_KEY), (JAXXObject) myTab); if (uiObject instanceof JComboBox) { return (JComboBox) uiObject; }

1 0

r1097 - trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/util
by tchemit＠users.labs.libre-entreprise.org Feb. 18, 2008

Feb. 18, 2008

Author: tchemit Date: 2008-02-18 23:06:02 +0000 (Mon, 18 Feb 2008) New Revision: 1097 Modified: trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/util/SimExplorerAbstractAction.java Log: par default la methode clear nefait (ainsi, pas besoin d'appeler le super pour chaque action) Modified: trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/util/SimExplorerAbstractAction.java =================================================================== --- trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/util/SimExplorerAbstractAction.java 2008-02-18 23:04:17 UTC (rev 1096) +++ trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/util/SimExplorerAbstractAction.java 2008-02-18 23:06:02 UTC (rev 1097) @@ -63,6 +63,7 @@ } catch (Exception e1) { SimExplorer.showError(e1); } finally { + this.e=null; // always clear action after use : actions are staless clear(); } @@ -106,7 +107,7 @@ } protected void clear() { - e=null; + // nothing by default } protected JComponent getUIObject(String name, JAXXObject container) {

1 0

r1096 - trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/util
by tchemit＠users.labs.libre-entreprise.org Feb. 18, 2008

Feb. 18, 2008

Author: tchemit Date: 2008-02-18 23:04:17 +0000 (Mon, 18 Feb 2008) New Revision: 1096 Modified: trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/util/SimExplorerAbstractAction.java trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/util/SimExplorerAbstractTabAction.java Log: ajout m?\195?\169thode findTab pour retrouver l'onglet ?\195?\160 partir d'ActionEvent suppression methode afterAction, on utilise directement updateUI Modified: trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/util/SimExplorerAbstractAction.java =================================================================== --- trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/util/SimExplorerAbstractAction.java 2008-02-18 23:03:19 UTC (rev 1095) +++ trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/util/SimExplorerAbstractAction.java 2008-02-18 23:04:17 UTC (rev 1096) @@ -39,11 +39,13 @@ static protected Log log = LogFactory.getLog(SimExplorerAbstractAction.class); + protected ActionEvent e; + public void actionPerformed(java.awt.event.ActionEvent e) { log.info("------------------------------------------------------------"); - log.info("event : " + e); - log.info("source : " + e.getSource()); + log.debug("event : " + e); + log.debug("source : " + e.getSource()); try { boolean accepted = beforeAction(e); log.info("action : " + this); @@ -55,7 +57,8 @@ if (accepted) { setStatus(toString()); doAction(e); - afterAction(e); + this.e=e; + updateUI(); } } catch (Exception e1) { SimExplorer.showError(e1); @@ -98,16 +101,12 @@ // nothing by default } - protected void afterAction(ActionEvent e) throws Exception { - updateUI(); - } - public void updateUI() { // nothing by default } protected void clear() { - // nothing by default + e=null; } protected JComponent getUIObject(String name, JAXXObject container) { @@ -116,10 +115,4 @@ } return (JComponent) container.getObjectById(name); } - - protected String getToken() { - String token; - token = getContext().getToken(); - return token; - } } Modified: trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/util/SimExplorerAbstractTabAction.java =================================================================== --- trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/util/SimExplorerAbstractTabAction.java 2008-02-18 23:03:19 UTC (rev 1095) +++ trunk/simexplorer-is/simexplorer-is-swing/src/java/fr/cemagref/simexplorer/is/ui/swing/actions/util/SimExplorerAbstractTabAction.java 2008-02-18 23:04:17 UTC (rev 1096) @@ -122,6 +122,20 @@ return getTab() == SimExplorerTab.remote; } + /** + * @param e l'évènement d'action + * @return le tab d'où l'action a été lancée + */ + protected SimExplorerTab findTab(ActionEvent e) { + SimExplorerTab sourceTab; + sourceTab = SimExplorerTab.findTab((Component) e.getSource()); + return sourceTab; + } + + /** + * @param e l'évènement d'action + * @return le dataentity sélectionné dans le tab d'où a été lancée l'action + */ protected DataEntityModel findSelectedItem(ActionEvent e) { SimExplorerTab sourceTab = SimExplorerTab.findTab((Component) e.getSource()); log.info("incoming tab " + sourceTab); @@ -131,12 +145,8 @@ return null; } DataEntityModel entityModel = sourceTab.getModel().getSelectedItem(); - if (entityModel!=null) { - try { - entityModel = entityModel.clone(); - } catch (CloneNotSupportedException e1) { - e1.printStackTrace(); - } + if (entityModel != null) { + entityModel = entityModel.cloneSafe(); } return entityModel; }

1 0