[jbosstools-commits] JBoss Tools SVN: r41326 - trunk/common/plugins/org.jboss.tools.common.ui/src/org/jboss/tools/common/ui/preferences.

[jbosstools-commits at lists.jboss.org]

Author: scabanovich
Date: 2012-05-23 19:49:23 -0400 (Wed, 23 May 2012)
New Revision: 41326

Modified:
   trunk/common/plugins/org.jboss.tools.common.ui/src/org/jboss/tools/common/ui/preferences/SeverityConfigurationBlock.java
Log:
JBIDE-11552
https://issues.jboss.org/browse/JBIDE-11552
Check for qualifier is redundant when all preferences used on the page are defined in the same plugin, and it is erroneous when page composed of preferences defined in several plugins.


Modified: trunk/common/plugins/org.jboss.tools.common.ui/src/org/jboss/tools/common/ui/preferences/SeverityConfigurationBlock.java
===================================================================
--- trunk/common/plugins/org.jboss.tools.common.ui/src/org/jboss/tools/common/ui/preferences/SeverityConfigurationBlock.java	2012-05-23 23:36:56 UTC (rev 41325)
+++ trunk/common/plugins/org.jboss.tools.common.ui/src/org/jboss/tools/common/ui/preferences/SeverityConfigurationBlock.java	2012-05-23 23:49:23 UTC (rev 41326)
@@ -332,10 +332,10 @@
 	}
 	
 	public void doFilter(String prefId){
-		String qualifier = getQualifier();
+//		String qualifier = getQualifier();
 		Key key = null;
 		for(Key k : fAllKeys){
-			if(k.getQualifier().equals(qualifier) && k.getName().equals(prefId))
+			if(/*k.getQualifier().equals(qualifier) &&*/ k.getName().equals(prefId))
 				key = k;
 		}
 		if(key != null){