Skip to content

Commit 6f0a64f

Browse files
authored
Merge pull request #341 from AndreSonntag/cleanUpFix
cleanUpProject method fix
2 parents 58cd553 + 47d9fb4 commit 6f0a64f

1 file changed

Lines changed: 11 additions & 10 deletions

File tree

  • plugins/de.cognicrypt.codegenerator/src/main/java/de/cognicrypt/codegenerator/generator

plugins/de.cognicrypt.codegenerator/src/main/java/de/cognicrypt/codegenerator/generator/CodeGenerator.java

Lines changed: 11 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -322,26 +322,27 @@ memberPath, new File(this.project
322322
* @throws CoreException
323323
* {@link DeveloperProject#refresh() refresh()} and {@link DeveloperProject#getPackagesOfProject(String) getPackagesOfProject()}
324324
*/
325-
protected void cleanUpProject(final IEditorPart editor) throws CoreException {
325+
protected void cleanUpProject(IEditorPart editor) throws CoreException {
326326
this.project.refresh();
327-
boolean fileOpen = false;
327+
final ICompilationUnit[] generatedCUnits = this.project.getPackagesOfProject(Constants.PackageNameAsName).getCompilationUnits();
328+
boolean anyFileOpen = false;
328329

329-
//prevent Organize Imports Problems
330-
if (editor.getTitle().equals(this.temporaryOutputFile)) {
331-
fileOpen = true;
332-
Utils.closeEditor(editor);
330+
if(editor == null && generatedCUnits[0].getResource().getType() == IResource.FILE) {
331+
IFile genClass = (IFile) generatedCUnits[0].getResource();
332+
IDE.openEditor(Utils.getCurrentlyOpenPage(), genClass);
333+
editor = Utils.getCurrentlyOpenPage().getActiveEditor();
334+
anyFileOpen = true;
333335
}
334336

335337
final OrganizeImportsAction organizeImportsActionForAllFilesTouchedDuringGeneration = new OrganizeImportsAction(editor.getSite());
336338
final FormatAllAction faa = new FormatAllAction(editor.getSite());
337-
final ICompilationUnit[] generatedCUnits = this.project.getPackagesOfProject(Constants.PackageNameAsName).getCompilationUnits();
338339
faa.runOnMultiple(generatedCUnits);
339340
organizeImportsActionForAllFilesTouchedDuringGeneration.runOnMultiple(generatedCUnits);
340341

341-
if (fileOpen) {
342-
final IFile outputFile = this.project.getIFile(this.temporaryOutputFile);
343-
IDE.openEditor(Utils.getCurrentlyOpenPage(), outputFile);
342+
if (anyFileOpen) {
343+
Utils.closeEditor(editor);
344344
}
345+
345346
final ICompilationUnit openClass = JavaCore.createCompilationUnitFrom(Utils.getCurrentlyOpenFile(editor));
346347
organizeImportsActionForAllFilesTouchedDuringGeneration.run(openClass);
347348
faa.runOnMultiple(new ICompilationUnit[] { openClass });

0 commit comments

Comments
 (0)