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

Improve warning message for invalid links

    Details

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

      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

          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.
          Hide
          Lukas Theussl added a comment -

          Should be fixed in doxia.

          Show
          Lukas Theussl added a comment - Should be fixed in doxia.

            People

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

              Dates

              • Created:
                Updated:
                Resolved: