Bug 1104920 - Renderer includes pressgangeditorlinenumber attributes in some CDATA
Summary: Renderer includes pressgangeditorlinenumber attributes in some CDATA
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: PressGang CCMS
Classification: Community
Component: Web-UI
Version: 1.6
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: 1.7
Assignee: Lee Newson
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-06-05 03:43 UTC by Lee Newson
Modified: 2014-06-22 21:32 UTC (History)
1 user (show)

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

Attachments (Terms of Use)
Test Case (380 bytes, text/xml)
2014-06-05 05:01 UTC, Lee Newson 		no flags Details
View All

Description Lee Newson 2014-06-05 03:43:48 UTC 
The following XML when rendered includes the pressgang line numbers:

<section>
	<title>A test topic</title>
 <programlisting language="XML" role='XML'><![CDATA[<figure>
 	<title>title</title>
 	<mediaobject>
 		<imageobject>
 			<imagedata align='center' fileref='images/imageid.png'/>
 		</imageobject>
 		<textobject>
 			<phrase>description</phrase>
 		</textobject>
 	</mediaobject>
 </figure>]]></programlisting>
</section>
Comment 1 Lee Newson 2014-06-05 04:48:27 UTC 
Fixed in 1.7-SNAPSHOT build 201406051424

Tracked this down to the XML having CRLF line endings and the function that added the pressgangeditorlinenumber attribute wasn't taking this into account.
Comment 2 Lee Newson 2014-06-05 05:01:43 UTC 
Created attachment 902401 [details]
Test Case

Adding a test case since the XML in the initial comment doesn't retain the CRLF line endings.
Comment 3 Lee Newson 2014-06-05 05:03:28 UTC 
Note: This version is now live on the test/development server.
Comment 4 Matthew Casperson 2014-06-16 20:55:40 UTC 
Verified
Note You need to log in before you can comment on or make changes to this bug.