Bug 1098851 - Asciidoc and linkend errors
Summary: Asciidoc and linkend errors
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: PressGang CCMS
Classification: Community
Component: ImportTool
Version: 1.6
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: 1.7
Assignee: Matthew Casperson
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: 1091570
TreeView+ depends on / blocked
 
Reported: 2014-05-19 03:27 UTC by Matthew Casperson
Modified: 2014-08-04 22:28 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed:
Embargoed:


Attachments (Terms of Use)

Description Matthew Casperson 2014-05-19 03:27:54 UTC
Importing asciidoc can lead to issues with the linkend attribute:

<section id="article" remap="TID_38509">
	<title>article</title>
	<simpara>
		Used for short documents, articles and general documentation. See the AsciiDoc distribution <literal>./doc/article.txt</literal> example.
	</simpara>
	<simpara>
		AsciiDoc defines standard DocBook article frontmatter and backmatter <link linkend="opal$opal$opal2">opal$opal$opal2</link> (appendix, abstract, bibliography, glossary, index).
	</simpara>
</section>

This is almost certainly because of an issue with a regex and the find/replace of $ with opal$

Comment 1 Matthew Casperson 2014-05-19 04:05:18 UTC
Fixed in Build 1.6-SNAPSHOT 201405191353

https://github.com/asciidoctor/asciidoctor/blob/master/test/fixtures/asciidoc.txt and https://github.com/asciidoctor/asciidoctor/blob/master/test/fixtures/asciidoc_index.txt both demonstrated the original problem, and both now import correctly.

Note that https://github.com/asciidoctor/asciidoctor/blob/master/test/fixtures/asciidoc.txt has invalid linkend attributes when it is converted. See https://github.com/asciidoctor/asciidoctor/issues/959

Comment 2 Lee Newson 2014-06-16 05:29:03 UTC
Verified that the linkend attributes are correct, however these should be <xref>'s and not <link>'s.

Comment 3 Lee Newson 2014-06-16 05:30:23 UTC
Actually ignore that sorry, because they contain additional text they need to be links. Moving this to VERIFIED.


Note You need to log in before you can comment on or make changes to this bug.