Details

    • Type: Bug Bug
    • Status: Resolved Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.5.2.Release
    • Fix Version/s: 2.6.1.M1
    • Component/s: Editor, Search
    • Labels:
    • Patch Submitted:
      Yes
    • Number of attachments :
      1

      Description

      Seems like the work in GRECLIPSE-1233 is not complete. Mark occurrences is not correct when there are default parameter references coming from other files. Eg-

      class C {
          def xxx(a, b = 2) {}
      } 
      

      and in another file:

      def obj
      new C().xxx()
      new C().xxx(obj)
      new C().xxx(obj, obj)
      new C().xxx(obj, obj, obj) 
      

      Select the first, third or fourth method reference and the first, third, and fourth reference highlights. Select the second, and only the second highlights.

      The correct behavior is that selecting any method reference should select all other references. This is the behavior that you see when all are inside the same file.

        Issue Links

          Activity

          Hide
          Kris De Volder added a comment -

          This is a patch I received from one of last nights hackathon participants.

          It solves the highlighting issue by ignoring parameters in the case where the references is a 'JDTMethodNode'.

          Show
          Kris De Volder added a comment - This is a patch I received from one of last nights hackathon participants. It solves the highlighting issue by ignoring parameters in the case where the references is a 'JDTMethodNode'.
          Hide
          Andrew Eisenberg added a comment -

          Thanks for the patch. Reasonable fix even if behavior is not completely preserved. See GRECLIPSE-1256 for a better long term solution to this bug.

          Show
          Andrew Eisenberg added a comment - Thanks for the patch. Reasonable fix even if behavior is not completely preserved. See GRECLIPSE-1256 for a better long term solution to this bug.

            People

            • Assignee:
              Andrew Eisenberg
              Reporter:
              Andrew Eisenberg
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: