Bug 1104920

Summary: Renderer includes pressgangeditorlinenumber attributes in some CDATA
Product: [Community] PressGang CCMS Reporter: Lee Newson <lnewson>
Component: Web-UIAssignee: Lee Newson <lnewson>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 1.6CC: mcaspers
Target Milestone: ---   
Target Release: 1.7   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
Test Case none

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