Skip to main content
aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorVikas Chandra2018-01-11 13:19:45 +0000
committerVikas Chandra2018-01-12 10:01:44 +0000
commit0a8dbf87973b400fbe02c3f32471ad9e15fabc4c (patch)
tree9c1ebbca69f61d0ad53de81ef6e8db3cd0744346
parent54ded9c322dc068d854a2a857a19898b469b4798 (diff)
downloadeclipse.jdt.ui-0a8dbf87973b400fbe02c3f32471ad9e15fabc4c.tar.gz
eclipse.jdt.ui-0a8dbf87973b400fbe02c3f32471ad9e15fabc4c.tar.xz
eclipse.jdt.ui-0a8dbf87973b400fbe02c3f32471ad9e15fabc4c.zip
Bug 527609 - [9] Investigate inconsistency of "Is not modular" node
during drag and drop. Change-Id: Id8df7b29015ccdb53da48072dd9106cab9e84277 Signed-off-by: Vikas Chandra <Vikas.Chandra@in.ibm.com>
-rw-r--r--org.eclipse.jdt.ui/ui/org/eclipse/jdt/internal/ui/wizards/buildpaths/LibrariesWorkbookPage.java8
1 files changed, 7 insertions, 1 deletions
diff --git a/org.eclipse.jdt.ui/ui/org/eclipse/jdt/internal/ui/wizards/buildpaths/LibrariesWorkbookPage.java b/org.eclipse.jdt.ui/ui/org/eclipse/jdt/internal/ui/wizards/buildpaths/LibrariesWorkbookPage.java
index 8edc2c8cc9..5c9f500948 100644
--- a/org.eclipse.jdt.ui/ui/org/eclipse/jdt/internal/ui/wizards/buildpaths/LibrariesWorkbookPage.java
+++ b/org.eclipse.jdt.ui/ui/org/eclipse/jdt/internal/ui/wizards/buildpaths/LibrariesWorkbookPage.java
@@ -199,6 +199,12 @@ public class LibrariesWorkbookPage extends BuildPathBasePage {
// only for update
cpe= checkAndUpdateIfModularJRE(cpe);
}
+ if (cpe.getEntryKind() == IClasspathEntry.CPE_CONTAINER) {
+ if (cpe.getAttribute(CPListElement.MODULE) == null && !isJREContainer(cpe.getPath())) {
+ cpe.updateExtraAttributeOfClasspathEntry();
+ cpe.setAttribute(CPListElement.MODULE, null);
+ }
+ }
Object mod= cpe.getAttribute(CPListElement.MODULE);
if(mod == null) {
rootClasspath.addCPListElement(cpe);
@@ -366,7 +372,7 @@ public class LibrariesWorkbookPage extends BuildPathBasePage {
return false;
String[] segments= path.segments();
for (String seg : segments) {
- if (seg.contains("JRE_CONTAINER")) { //$NON-NLS-1${
+ if (seg.equals(JavaRuntime.JRE_CONTAINER)) {
return true;
}
}

Back to the top