Description: ------------ The change to include Whitespace entities has broken a number of applications that rely on the fact that they did not exist before.. I think the Default behaviour should remain as before, the new behaviour should be optional.

I suggest - add: @var options parser options var $options = array( 'use_cdata_sections' => false, 'strip_whitespace' => true; 'filename' => '', 'version' => '1.0' ); function XML_Tree($options = '', $version='1.0') { if (is_array($options)) { $this->options = $options + $this->options; } else { $this->options['filename'] = $options; $this->options['version'] = $options; } // BC for any wrappers that exist.. - dont use this->filname anymore.. $this->filename = $this->options['filename']; $this->version = $this->options['version']; } It leaves the door open for future additions..

the change that cause the problem is the removal of this. if (trim($data) != '') { $this->cdata .= $data; }

Perhaps your application using XML_Tree wasn't fully defined. It might be better to write the condition for when there is whitespace, but in your case, the application would simply do nothing about it. This is easier to fix then the side-effect of turning off or on a configuration optin.

by "fix" I meant to say "work with". By the way, this advice of writing an application with XML_Tree that considers the whitespace condition, I'm applying it myself.

see: http://pear.php.net/bugs/2457

Until the package gets a new maintainer.