Maven JXR
  1. Maven JXR
  2. JXR-87

Change line number anchor name pattern

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.2
    • Fix Version/s: 2.4
    • Component/s: jxr
    • Labels:
      None
    • Environment:
      Site plugin 2.2
    • Flags:
      Patch

      Description

      Linenumber achor anames are solely the line number but this brings problems. Then linking in APT to source code, it complains:

      [WARNING] [APT Parser] Modified invalid link: '77' to '../kcc-core/xref/package/Object.html#a77'
      

      This means that I am not able to link to lines anymore. Change the anchor name pattern to L\d+

      1. JXR-87.patch
        0.7 kB
        Michael Osipov

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open In Progress In Progress
        1004d 23h 54m 1 Michael Osipov 23/Nov/13 10:44
        In Progress In Progress Closed Closed
        3m 39s 1 Michael Osipov 23/Nov/13 10:48
        Mark Thomas made changes -
        Workflow jira [ 12959710 ] Default workflow, editable Closed status [ 12996806 ]
        Mark Thomas made changes -
        Project Import Mon Apr 06 00:07:28 UTC 2015 [ 1428278848796 ]
        Mark Thomas made changes -
        Flags Patch [ 10430 ]
        Patch Submitted Yes [ 10763 ]
        Mark Thomas made changes -
        Workflow jira [ 12722156 ] Default workflow, editable Closed status [ 12748919 ]
        Mark Thomas made changes -
        Project Import Sun Apr 05 10:32:48 UTC 2015 [ 1428229968272 ]
        Michael Osipov made changes -
        Status In Progress [ 3 ] Closed [ 6 ]
        Resolution Fixed [ 1 ]
        Hide
        Michael Osipov added a comment -

        Fixed with r1544832.

        Show
        Michael Osipov added a comment - Fixed with r1544832.
        Michael Osipov made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Hide
        Michael Osipov added a comment -

        Retargeting fix for 2.4.

        Show
        Michael Osipov added a comment - Retargeting fix for 2.4.
        Michael Osipov made changes -
        Fix Version/s 2.4 [ 17492 ]
        Patch Submitted Yes [ 10070 ]
        Assignee Michael Osipov [ michael-o ]
        Hide
        Michael Osipov added a comment -

        Is anyone opposed to fix this after more than two years? I will apply this patch and target it for the next release. Those who care for compat should read CHANGES anyway.

        Show
        Michael Osipov added a comment - Is anyone opposed to fix this after more than two years? I will apply this patch and target it for the next release. Those who care for compat should read CHANGES anyway.
        Hide
        Michael Osipov added a comment -

        True, but it was broken anyway. Doxia was broken too before Maven 2.2.x. That's the reason I created this patch. Btw, for those who upgrade from Maven 2.0 to 2.2 they see broken links too. So in my opinion there is little harm.

        Show
        Michael Osipov added a comment - True, but it was broken anyway. Doxia was broken too before Maven 2.2.x. That's the reason I created this patch. Btw, for those who upgrade from Maven 2.0 to 2.2 they see broken links too. So in my opinion there is little harm.
        Hide
        Neeme Praks added a comment -

        Your patch will break all the links out there that rely on the current format. I would suggest to include a configuration option to restore the old behavior for people that cannot update all the other systems that rely on the old format. But new format should be default as the current format is broken (see also http://stackoverflow.com/questions/7110556/is-it-valid-to-use-only-digits-as-uri-fragment-identifier)

        Show
        Neeme Praks added a comment - Your patch will break all the links out there that rely on the current format. I would suggest to include a configuration option to restore the old behavior for people that cannot update all the other systems that rely on the old format. But new format should be default as the current format is broken (see also http://stackoverflow.com/questions/7110556/is-it-valid-to-use-only-digits-as-uri-fragment-identifier )
        Michael Osipov made changes -
        Field Original Value New Value
        Attachment JXR-87.patch [ 56017 ]
        Show
        Michael Osipov added a comment - The suspicious line is: http://svn.apache.org/viewvc/maven/jxr/tags/jxr-2.2/maven-jxr/src/main/java/org/apache/maven/jxr/JavaCodeTransform.java?view=markup#l375
        Michael Osipov created issue -

          People

          • Assignee:
            Michael Osipov
            Reporter:
            Michael Osipov
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development