You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@ant.apache.org by hi...@apache.org on 2015/01/11 17:40:39 UTC

[3/5] ant-ivyde git commit: Fix for the user-selected configurations not being checked in the viewer

Fix for the user-selected configurations not being checked in the viewer

When you have an ivy.xml that extends another module and that other
module defines some configurations, it occurred that calling

  settingsSetupTab.init(conf.isSettingsProjectSpecific(),
conf.getIvySettingsSetup());
        
would cause the configurations being set again, because the
ModifyListener of ivyUserDirEditor would fire and cause
reinitialization.


Project: http://git-wip-us.apache.org/repos/asf/ant-ivyde/repo
Commit: http://git-wip-us.apache.org/repos/asf/ant-ivyde/commit/ba8906b9
Tree: http://git-wip-us.apache.org/repos/asf/ant-ivyde/tree/ba8906b9
Diff: http://git-wip-us.apache.org/repos/asf/ant-ivyde/diff/ba8906b9

Branch: refs/heads/master
Commit: ba8906b926d6c171fa9ba818134e139f1fd4ad14
Parents: 4953a50
Author: Carsten Pfeiffer <ca...@gebit.de>
Authored: Mon Aug 18 16:20:46 2014 +0200
Committer: Nicolas Lalevée <ni...@hibnet.org>
Committed: Sun Jan 11 16:54:54 2015 +0100

----------------------------------------------------------------------
 .../ivyde/internal/eclipse/cpcontainer/IvydeContainerPage.java     | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ant-ivyde/blob/ba8906b9/org.apache.ivyde.eclipse/src/java/org/apache/ivyde/internal/eclipse/cpcontainer/IvydeContainerPage.java
----------------------------------------------------------------------
diff --git a/org.apache.ivyde.eclipse/src/java/org/apache/ivyde/internal/eclipse/cpcontainer/IvydeContainerPage.java b/org.apache.ivyde.eclipse/src/java/org/apache/ivyde/internal/eclipse/cpcontainer/IvydeContainerPage.java
index b4bedb5..e66eb23 100644
--- a/org.apache.ivyde.eclipse/src/java/org/apache/ivyde/internal/eclipse/cpcontainer/IvydeContainerPage.java
+++ b/org.apache.ivyde.eclipse/src/java/org/apache/ivyde/internal/eclipse/cpcontainer/IvydeContainerPage.java
@@ -383,9 +383,9 @@ public class IvydeContainerPage extends NewElementWizardPage implements IClasspa
 
     private void loadFromConf() {
         ivyFilePathText.init(conf.getIvyXmlPath());
-        confTableViewer.init(conf.getConfs());
 
         settingsSetupTab.init(conf.isSettingsProjectSpecific(), conf.getIvySettingsSetup());
+        confTableViewer.init(conf.getConfs()); // *after* settingsSetupTab.init()!
         classpathSetupTab.init(conf.isClassthProjectSpecific(), conf.getClasspathSetup());
         mappingSetupTab.init(conf.isMappingProjectSpecific(), conf.getMappingSetup());
         // project == null <==> container in a launch config: always resolve before launch