Maven 2.x PDF Plugin

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

is superceded by

Bug - A problem which impairs or prevents the functions of the product. DOXIA-434 Warning about old link format fails to note location of bad link

  • Major - Major loss of function.
  • Open - The issue is open and ready for the assignee to start work on it.

New Feature - A new feature of the product, which has yet to be developed. DOXIA-224 Add source name in parser

  • Major - Major loss of function.
  • Open - The issue is open and ready for the assignee to start work on it.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
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
Permalink
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
Permalink
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 ]
Assignee Lukas Theussl [ lukas ]
Resolution Won't Fix [ 2 ]
Lukas Theussl made changes -
Link This issue is superceded by DOXIA-434 [ DOXIA-434 ]

People

  • Assignee:
    Lukas Theussl
    Reporter:
    Wendy Smoak
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: