Bug 819420

Summary: Processing instructions should not be stripped during build
Product: [Community] Publican Reporter: Raphaël Hertzog <raphael>
Component: publicanAssignee: Jeff Fearn <jfearn>
Status: CLOSED CURRENTRELEASE QA Contact: Ruediger Landmann <rlandman>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 3.0CC: anross, rlandman
Target Milestone: 3.0   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 3.0.0 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-10-30 23:11:45 EDT Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:

Description Raphaël Hertzog 2012-05-07 03:44:24 EDT
Description: I have put some processing instructions in the XML files (for example <?dbhtml filename="intro.html" ?> or <?latex \pagebreak ?>).

When I build the document, those processing instructions are stripped, i.e. they are no longer present in the tmp/<lang>/xml/* files.

Expected result: The processing instructions should be kept.
Comment 1 Jeff Fearn 2012-08-17 02:18:27 EDT
Added call to store_pis in XmlClean::process_file.

To ssh://git.fedorahosted.org/git/publican.git
   aa33a03..d608434  master -> master
Comment 2 Andrew Ross 2012-10-03 00:49:21 EDT
Built from upstream source on a RHEL6.3 workstation: publican-3.0-0.el6.t226.noarch

Here's the steps I took to test the fix:

1. Create a new book:

publican create --name Test_Book

2. Insert processessing tags into en-US/Chapter.xml (used both the tags listed in comment #0)

<title>Test Section 1</title>
 <para>
	<?dbhtml filename="intro.html" ?>
	This is a test paragraph in a section <?latex \pagebreak ?>
</para>

3. Build 

publican build --formats html-single --langs en-US

4. Check the tmp/en-US/xml/Chapter.xml in an editor (gedit in my case). Processing commands are still present.

<title>Test Section 1</title>
 <para>
	<?dbhtml filename="intro.html" ?> This is a test paragraph in a section <?latex \pagebreak ?>
</para>


---

During the build I got an unvalidated tag warning [1]. I couldn't find any tags in either en-US or tmp/xml with pi :-/

I removed the 2 processing instruction tags, cleaned and rebuilt. That error didn't appear.

Any ideas if this is PEBKAC or something we should be concerned about?



[1] *WARNING: Unvalidated tag: '~pi'. This tag may not be displayed correctly, may generate invalid xhtml, or may breach Section 508 Accessibility standards.
Comment 3 Jeff Fearn 2012-10-08 01:05:34 EDT
(In reply to comment #2)
> [1] *WARNING: Unvalidated tag: '~pi'. This tag may not be displayed
> correctly, may generate invalid xhtml, or may breach Section 508
> Accessibility standards.

This is a minor issue and can be ignored. In all likelihood we will be completely removing that warning in the future anyway.