[jboss-cvs] jbosside/core/plugins/org.jboss.ide.eclipse.packages.core/src/main/org/jboss/ide/eclipse/packages/core/model/types ...
Robert Stryker
rawblem at gmail.com
Thu Dec 21 20:08:40 EST 2006
User: rawb
Date: 06/12/21 20:08:40
Modified: core/plugins/org.jboss.ide.eclipse.packages.core/src/main/org/jboss/ide/eclipse/packages/core/model/types
JARPackageType.java
Removed: core/plugins/org.jboss.ide.eclipse.packages.core/src/main/org/jboss/ide/eclipse/packages/core/model/types
AbstractPackageType.java
Log:
The Extension Manager had several errors. It used the incorrect method of instantiating a provided class (Class.forName led to class not found errors. configElement.createExecutable("attribute") is the proper method)
It also assumed all provided classes would be of type AbstractPackageType, instead of IPackageType. It then made use of methods not present in IPackageType (setId, setLabel). I've removed all references to AbstractPackageType, the JARPackageType now directly returns its information (jar, JAR), and the extension point data is essentially ignored.
Suck It.
Revision Changes Path
1.7 +9 -2 jbosside/core/plugins/org.jboss.ide.eclipse.packages.core/src/main/org/jboss/ide/eclipse/packages/core/model/types/JARPackageType.java
(In the diff below, changes in quantity of whitespace are not shown.)
Index: JARPackageType.java
===================================================================
RCS file: /cvsroot/jboss/jbosside/core/plugins/org.jboss.ide.eclipse.packages.core/src/main/org/jboss/ide/eclipse/packages/core/model/types/JARPackageType.java,v
retrieving revision 1.6
retrieving revision 1.7
diff -u -b -r1.6 -r1.7
--- JARPackageType.java 20 Dec 2006 20:43:47 -0000 1.6
+++ JARPackageType.java 22 Dec 2006 01:08:40 -0000 1.7
@@ -21,10 +21,9 @@
* The name of the resulting JAR will be the project's name followed by a ".jar" extension.
* @author Marshall
*/
-public class JARPackageType extends AbstractPackageType {
+public class JARPackageType implements IPackageType {
public static final String TYPE_ID = "jar";
-
public IPackage createDefaultConfiguration(IProject project, IProgressMonitor monitor)
{
Assert.isNotNull(project);
@@ -79,4 +78,12 @@
return SUPPORT_NONE;
}
+
+ public String getId() {
+ return "jar";
+ }
+
+ public String getLabel() {
+ return "JAR";
+ }
}
More information about the jboss-cvs-commits
mailing list