IvyDE
  1. IvyDE
  2. IVYDE-199

File resource listener on Ivy files launches resolve when it detects modifications.

    Details

    • Type: New Feature New Feature
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.1.0
    • Component/s: None
    • Labels:
      None

      Description

      Currently, the Ivy Editor launches a resolve as part of its save logic. But if the file was modified outside of the editor (e.g. a source control update, via the XML editor, etc.), the plugin does not launch a resolve.

      This issue is linked to IVYDE-197 through line 58 of IvyFileResourceListener:

      ...
      if (resource instanceof IFile && IvyPlugin.getPreferenceStoreHelper().getAutoResolveOnChange())
      ...

      Of course this feature could be added without the inclusion of IVYDE-197, but I think they complement each other nicely.

      Thanks!

      1. ivyde-199.patch
        6 kB
        Jon Schneider

        Issue Links

          Activity

          Hide
          Nicolas Lalevée added a comment -

          patch integrated with some little modification:

          • some CoreException could be thrown on refresh, I don't think the end user should bother, at least I made it logged.
          • with that new feature, when using the ivy editor, two resolve process where launched on save: one with that new listener, one with the ivy editor. So I removed the Ivy editor one and everything work fine.
          Show
          Nicolas Lalevée added a comment - patch integrated with some little modification: some CoreException could be thrown on refresh, I don't think the end user should bother, at least I made it logged. with that new feature, when using the ivy editor, two resolve process where launched on save: one with that new listener, one with the ivy editor. So I removed the Ivy editor one and everything work fine.
          Hide
          Jon Schneider added a comment -

          Attached the patch.

          Show
          Jon Schneider added a comment - Attached the patch.

            People

            • Assignee:
              Nicolas Lalevée
              Reporter:
              Jon Schneider
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development