Issue Details (XML | Word | Printable)

Key: MPDF-11
Type: Improvement Improvement
Status: Open Open
Priority: Minor Minor
Assignee: Unassigned
Reporter: Wendy Smoak
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Maven 2.x PDF Plugin

Improve warning message for invalid links

Created: 06/May/09 08:52 AM   Updated: 06/May/09 04:24 PM
Return to search
Component/s: None
Affects Version/s: 1.0
Fix Version/s: None

Time Tracking:
Not Specified

Environment: 1.0-SNAPSHOT r772248


 Description  « Hide

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 ./ ).



Wendy Smoak added a comment - 06/May/09 09:38 AM

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


Lukas Theussl added a comment - 06/May/09 04:24 PM

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.