php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Doc Bug #74492 XMLWriter adds indent between mixed child nodes.
Submitted: 2017-04-23 10:58 UTC Modified: 2018-04-02 22:05 UTC
From: subjective@php.net Assigned:
Status: Analyzed Package: XML Writer
PHP Version: 7.1.4 OS:
Private report: No CVE-ID: None
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes.
If this is not your bug, you can add a comment by following this link.
If this is your bug, but you forgot your password, you can retrieve your password here.
Password:
Status:
Package:
Bug Type:
Summary:
From: subjective@php.net
New email:
PHP Version: OS:

 

 [2017-04-23 10:58 UTC] subjective@php.net
Description:
------------
XMLWriter adds the defined indent between mixed child nodes. This will change the meaning if the text nodes do not contain leading/trailing whitespaces.

This does not happen if the indent is set to zero.



Test script:
---------------
<?php
$_ = new XMLWriter();
$_->openMemory();
$_->setIndent(2);

$_->startDocument();
$_->startElement('p');
$_->text('before');
$_->writeElement('a', 'element');
$_->text('after');
$_->endElement();
$_->endDocument();

echo $_->outputMemory();

Expected result:
----------------
<?xml version="1.0"?>
<p>before<a>element</a>after</p>

Actual result:
--------------
<?xml version="1.0"?>
<p>before <a>element</a>
after</p>

Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2018-04-02 22:05 UTC] cmb@php.net
-Status: Open +Status: Analyzed
 [2018-04-02 22:05 UTC] cmb@php.net
ext/xmlwriter is a rather thin layer over libxml2's xmlwriter
module[1], and this is the way that xmlTextWriterSetIndent() works
(currently). It might be worthwhile to report this issue
upstream[2], but pretty printing might better be left to
specialized tools, anyway.

In my opinion, it is best to make this a documentation issue, and
to convey that ::setIndent() is not necessarily suitable for mixed
content.

By the way, XMLWriter::setIndent() expects a boolean argument, not
an integer.

[1] <http://xmlsoft.org/html/libxml-xmlwriter.html>
[2] <https://bugzilla.gnome.org/buglist.cgi?product=libxml2>
 [2018-04-02 22:05 UTC] cmb@php.net
-Type: Bug +Type: Documentation Problem
 
PHP Copyright © 2001-2019 The PHP Group
All rights reserved.
Last updated: Fri Nov 22 05:01:23 2019 UTC