Maven PDF Plugin
  1. Maven PDF Plugin
  2. MPDF-11

Improve warning message for invalid links

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Won't Fix
    • Affects Version/s: 1.0
    • Fix Version/s: None
    • Labels:
      None
    • Environment:
      1.0-SNAPSHOT r772248

      Description

      Currently I'm getting several warnings like this:

      [WARNING] Modified invalid link: notification/index.html

      It would be nice if the message said why this is invalid and/or what to do to fix it (usually, prepend ./ ).

        Issue Links

          Activity

          Wendy Smoak created issue -
          Hide
          Wendy Smoak added a comment -

          Including the problem .apt filename would be good, so the user knows where to edit.

          Show
          Wendy Smoak added a comment - Including the problem .apt filename would be good, so the user knows where to edit.
          Hide
          Lukas Theussl added a comment -

          I added some additional warning for the most common use case. This won't catch all mistakes though, as some invalid local links pass through as valid internal links, see the definition.

          Also indicating the source file name is currently not possible because doxia parsers don't get the source document passed in, see DOXIA-224.
          However, if you run with -X you should see the file that is being processed when the warning is emitted.

          Show
          Lukas Theussl added a comment - I added some additional warning for the most common use case. This won't catch all mistakes though, as some invalid local links pass through as valid internal links, see the definition . Also indicating the source file name is currently not possible because doxia parsers don't get the source document passed in, see DOXIA-224 . However, if you run with -X you should see the file that is being processed when the warning is emitted.
          Lukas Theussl made changes -
          Field Original Value New Value
          Link This issue is superceded by DOXIA-224 [ DOXIA-224 ]
          Hide
          Lukas Theussl added a comment -

          Should be fixed in doxia.

          Show
          Lukas Theussl added a comment - Should be fixed in doxia.
          Lukas Theussl made changes -
          Status Open [ 1 ] Closed [ 6 ]
          Resolution Won't Fix [ 2 ]
          Assignee Lukas Theussl [ lukas ]
          Lukas Theussl made changes -
          Link This issue is superceded by DOXIA-434 [ DOXIA-434 ]
          Mark Thomas made changes -
          Project Import Sun Apr 05 12:00:08 UTC 2015 [ 1428235208998 ]
          Mark Thomas made changes -
          Project Import Mon Apr 06 00:22:36 UTC 2015 [ 1428279756475 ]
          Mark Thomas made changes -
          Workflow jira [ 12960289 ] Default workflow, editable Closed status [ 12997131 ]
          Mark Thomas made changes -
          Assignee Lukas JOSEFIK [ lukas ] Lukas Theussl [ ltheussl ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Closed Closed
          630d 22h 22m 1 Lukas Theussl 27/Jan/11 07:14

            People

            • Assignee:
              Lukas Theussl
              Reporter:
              Wendy Smoak
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development