Re: cvs: php-src(PHP_5_1) /ext/xmlreader php_xmlreader.c
| From: | Rob Richards | Date: | Fri, 31 Mar 2006 11:09:23 +0000 |
| Subject: | Re: cvs: php-src(PHP_5_1) /ext/xmlreader php_xmlreader.c | ||
| References: | 1 2 3 4 5 6 7 8 | Groups: | php.cvs |
| Request: | Send a blank email to php-cvs+get-37612@lists.php.net to get a copy of this message | ||
>> As I see it, E_NOTICE should be thrown since accessing uninitialized
>> variable triggers E_NOTICE, why not XML attribute?
>> (yeah, massive BC break so no chance in hell I'd recommend
>> implementing it though)
>
> That's insane :-)
>
> Null is the way to go, the question was only when is the right time to
> make the change.
My (a bit late) input:
XMLReader is not DOM. As Pierre said it follows C# xmlTextReader behavior
and not DOM specs.
This is really a bug fix and should go in now. The behavior is changed
only slightly with minimal to no impact on existing scripts, but now will
at least produce correct behavior for those who need it.