You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Michael Osipov (Jira)" <ji...@apache.org> on 2022/07/06 09:25:00 UTC

[jira] [Commented] (DOXIA-662) Non unique IDs generated by IndexingSink

    [ https://issues.apache.org/jira/browse/DOXIA-662?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17563072#comment-17563072 ] 

Michael Osipov commented on DOXIA-662:
--------------------------------------

Would it make sense to provide an id, see similar to MSHARED-390?

> Non unique IDs generated by IndexingSink
> ----------------------------------------
>
>                 Key: DOXIA-662
>                 URL: https://issues.apache.org/jira/browse/DOXIA-662
>             Project: Maven Doxia
>          Issue Type: Bug
>          Components: Core
>    Affects Versions: 2.0.0-M3
>            Reporter: Konrad Windszus
>            Priority: Major
>
> If a HTML source contains non-unique headlines (i.e. two or more headlines with the same text) in a single HTML page the IDs being generated out of those are not unique.
> That leads to issues with e.g. the TOC macro (https://github.com/apache/maven-doxia/blob/2aea4500ba6559b91fe3a4b83a5d3aafc10116b4/doxia-core/src/main/java/org/apache/maven/doxia/macro/toc/TocMacro.java#L155) which writes an anchor link leveraging that id.
> The problematic line is https://github.com/apache/maven-doxia/blob/2aea4500ba6559b91fe3a4b83a5d3aafc10116b4/doxia-core/src/main/java/org/apache/maven/doxia/index/IndexingSink.java#L314 which just uses the (sanitized) title as id. In case of conflicts the title needs to be suffixed with an index to be unique.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)