Author: dazarov
Date: 2012-02-01 20:54:03 -0500 (Wed, 01 Feb 2012)
New Revision: 38390
Modified:
trunk/cdi/plugins/org.jboss.tools.cdi.ui/src/org/jboss/tools/cdi/ui/marker/DeleteAllDisposerDuplicantMarkerResolution.java
trunk/common/plugins/org.jboss.tools.common/src/org/jboss/tools/common/refactoring/MarkerResolutionUtils.java
trunk/seam/plugins/org.jboss.tools.seam.ui/src/org/jboss/tools/seam/ui/marker/AbstractSeamMarkerResolution.java
Log:
Seam quick fixes insert incorrectly formated code
https://issues.jboss.org/browse/JBIDE-10766
Modified:
trunk/cdi/plugins/org.jboss.tools.cdi.ui/src/org/jboss/tools/cdi/ui/marker/DeleteAllDisposerDuplicantMarkerResolution.java
===================================================================
---
trunk/cdi/plugins/org.jboss.tools.cdi.ui/src/org/jboss/tools/cdi/ui/marker/DeleteAllDisposerDuplicantMarkerResolution.java 2012-02-02
01:40:01 UTC (rev 38389)
+++
trunk/cdi/plugins/org.jboss.tools.cdi.ui/src/org/jboss/tools/cdi/ui/marker/DeleteAllDisposerDuplicantMarkerResolution.java 2012-02-02
01:54:03 UTC (rev 38390)
@@ -58,7 +58,6 @@
DeleteAllDisposerAnnotationsProcessor processor = new
DeleteAllDisposerAnnotationsProcessor(file, method, label);
ProcessorBasedRefactoring refactoring = new ProcessorBasedRefactoring(processor);
DeletePreviewWizard wizard = new DeletePreviewWizard(refactoring);
- //RefactoringWizardOpenOperation op = new RefactoringWizardOpenOperation(wizard);
wizard.showWizard();
}
Modified:
trunk/common/plugins/org.jboss.tools.common/src/org/jboss/tools/common/refactoring/MarkerResolutionUtils.java
===================================================================
---
trunk/common/plugins/org.jboss.tools.common/src/org/jboss/tools/common/refactoring/MarkerResolutionUtils.java 2012-02-02
01:40:01 UTC (rev 38389)
+++
trunk/common/plugins/org.jboss.tools.common/src/org/jboss/tools/common/refactoring/MarkerResolutionUtils.java 2012-02-02
01:54:03 UTC (rev 38390)
@@ -274,7 +274,7 @@
spaces += SPACE;
}
- String text = "";
+ String text = lineSeparator;
for(String line : lines){
text += spaces;
text += line;
Modified:
trunk/seam/plugins/org.jboss.tools.seam.ui/src/org/jboss/tools/seam/ui/marker/AbstractSeamMarkerResolution.java
===================================================================
---
trunk/seam/plugins/org.jboss.tools.seam.ui/src/org/jboss/tools/seam/ui/marker/AbstractSeamMarkerResolution.java 2012-02-02
01:40:01 UTC (rev 38389)
+++
trunk/seam/plugins/org.jboss.tools.seam.ui/src/org/jboss/tools/seam/ui/marker/AbstractSeamMarkerResolution.java 2012-02-02
01:54:03 UTC (rev 38390)
@@ -81,18 +81,6 @@
}
}
- private IType getType(IJavaElement element){
- IJavaElement elem = element;
- while(elem != null){
- if(elem instanceof IType)
- return (IType)elem;
- elem = elem.getParent();
- }
- return null;
- }
-
-
-
protected boolean validateComponentName(String value){
ISeamProject seamProject = getSeamProject();
Map<String, IStatus> errors =
SeamValidatorFactory.SEAM_COMPONENT_NAME_VALIDATOR.validate(value, seamProject);