[ 
http://jira.codehaus.org/browse/DOXIA-91?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_92311
 ] 

Henning Schmiedehausen commented on DOXIA-91:
---------------------------------------------

Ok, you got me confused. This is only possible when a PathDescriptor contains 
null for both the PathUrl and the relativePath. Which can only happen when the 
PathDescriptor has been constructed using a 'null' path. And the only place 
where this is possible is in PathUtils line 47 and there is an explicit null 
test in front of it. 

Can you give some pointers on what project exactly you tried to build the sites 
for? Or is that some non-public site? 

If possible, could you put a breakpoint at PathUtils::getRelativePath(String, 
String) and tell me what exactly goes in there. I'd guess that oldPath is null.


I'll add another patch shortly which has another minor fix and I will add some 
explicit testing on that. Also some unit tests.

> Update Doxia Decoration model to actually work in reactor build.
> ----------------------------------------------------------------
>
>                 Key: DOXIA-91
>                 URL: http://jira.codehaus.org/browse/DOXIA-91
>             Project: doxia
>          Issue Type: Improvement
>          Components: Decoration Model
>            Reporter: Henning Schmiedehausen
>             Fix For: 1.0
>
>         Attachments: doxia-decoration-3.patch, doxia-decoration.patch, 
> doxia-decoration.patch, 
> org.apache.maven.doxia.site.decoration.inheritance.DecorationModelInheritenceAssemblerTest.txt
>
>
> This is a major patch to the decoration model to bring some sanity in the 
> link resolution when it is used under the reactor build. It generates 
> breadcrumbs and item links correctly in all scenarios that I have tested (the 
> current HEAD code does not) and passes all the unit tests that are already 
> there.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to