GRECLIPSE

Formatter produces error when source file is not on an Eclipse source folder

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 2.0.1Release
  • Fix Version/s: 2.0.2Release
  • Component/s: None
  • Labels:
    None
  • Patch Submitted:
    Yes
  • Number of attachments :
    1

Description

I dont know if this is really an error. I think it should be considered as a bug.

I had a Gant script in a Grails project residing in scripts/; that folder was not added to the Build Path. When I tried to format this source file, the plugin always outputed an error. Formatting should work for every file, that should not be dependant on the build path.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Kris De Volder added a comment -

GroovyCompilationUnit inherits a getUnderlyingResource method that only appears to work right when on the build path. I've patched this up by implementing a method to handle this as a special case and delegate to super method in all other cases.

Show
Kris De Volder added a comment - GroovyCompilationUnit inherits a getUnderlyingResource method that only appears to work right when on the build path. I've patched this up by implementing a method to handle this as a special case and delegate to super method in all other cases.
Hide
Permalink
Andrew Eisenberg added a comment -

Patch is applied.

Show
Andrew Eisenberg added a comment - Patch is applied.
Hide
Permalink
Andrew Eisenberg added a comment -

And committed.

Show
Andrew Eisenberg added a comment - And committed.
Hide
Permalink
Kris De Volder added a comment -

Resolving issue. This should be fixed now.

Please reopen issue if this is still a problem in the latest snapshot build.

Show
Kris De Volder added a comment - Resolving issue. This should be fixed now. Please reopen issue if this is still a problem in the latest snapshot build.

People

  • Assignee:
    Kris De Volder
    Reporter:
    Jean-Noël Rivasseau
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: