[PHP-DEV] simplexml inconsistencies in the last few releases (the same issue already discussed)
Hi It's about the two tests in http://svn.php.net/viewvc/php/php-src/branches/PHP_5_3/ext/simplexml/tests/008.phpt and http://svn.php.net/viewvc/php/php-src/branches/PHP_5_3/ext/simplexml/tests/bug48601.phpt which contractited themself and we had back and forth commits due to this. So we had the following history 5.3.0 old behaviour ... 5.3.3 new behaviour ... 5.3.7 old behaviour ... 5.4.0 new behaviour The main question now is, should we introduce back the new behavior in the 5.3 line? Since the new behaviour was already there since more than a year, but now isn't, i'm not sure, what to do. The good point about the new behaviour is that it's almost BC, so going from the old to the new shouldn't cause much problems, but vice versa it does. The reason is that if you do an valid xpath query which returns zero results, the old behaviour returned false, the new one returns an empty array, so before 5.3.3 you had to write *** $res = $sx->query("/some/xpath"); if ($res) { foreach($res as $node) {} } *** Which still will work with the new behaviour, but that "if" is not really needed with the new behaviour, so going back to the old one may break if nothing is found (there may be problems when you asked for "zero-results" with checking if $res is false) Anyway, I'd appreciate some opinions about introducing back the new beehaviour in 5.3 or not (I guess 5.3.8 wasn't the last 5.3 release :)) chregu -- Liip AG // Feldstrasse 133 // CH-8004 Zurich Tel +41 43 500 39 81 // Mobile +41 76 561 88 60 www.liip.ch // blog.liip.ch // GnuPG 0x0748D5FE -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
Re: [PHP-DEV] simplexml inconsistencies in the last few releases (the same issue already discussed)
imo new behavior in 5.3 Rob On Sep 1, 2011, at 1:35 AM, Christian Stocker wrote: > Hi > > It's about the two tests in > > http://svn.php.net/viewvc/php/php-src/branches/PHP_5_3/ext/simplexml/tests/008.phpt > and > http://svn.php.net/viewvc/php/php-src/branches/PHP_5_3/ext/simplexml/tests/bug48601.phpt > > which contractited themself and we had back and forth commits due to this. > > So we had the following history > > 5.3.0 old behaviour > ... > 5.3.3 new behaviour > ... > 5.3.7 old behaviour > ... > 5.4.0 new behaviour > > > The main question now is, should we introduce back the new behavior in > the 5.3 line? Since the new behaviour was already there since more than > a year, but now isn't, i'm not sure, what to do. > > The good point about the new behaviour is that it's almost BC, so going > from the old to the new shouldn't cause much problems, but vice versa it > does. The reason is that if you do an valid xpath query which returns > zero results, the old behaviour returned false, the new one returns an > empty array, so before 5.3.3 you had to write > > *** > $res = $sx->query("/some/xpath"); > > if ($res) { > foreach($res as $node) {} > } > *** > > Which still will work with the new behaviour, but that "if" is not > really needed with the new behaviour, so going back to the old one may > break if nothing is found (there may be problems when you asked for > "zero-results" with checking if $res is false) > > Anyway, I'd appreciate some opinions about introducing back the new > beehaviour in 5.3 or not (I guess 5.3.8 wasn't the last 5.3 release :)) > > chregu > > > -- > Liip AG // Feldstrasse 133 // CH-8004 Zurich > Tel +41 43 500 39 81 // Mobile +41 76 561 88 60 > www.liip.ch // blog.liip.ch // GnuPG 0x0748D5FE > > > -- > PHP Internals - PHP Runtime Development Mailing List > To unsubscribe, visit: http://www.php.net/unsub.php -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
Re: [PHP-DEV] simplexml inconsistencies in the last few releases (the same issue already discussed)
Hi Ok, it's committed chregu On 01.09.11 13:10, Rob Richards wrote: > imo new behavior in 5.3 > > Rob > > On Sep 1, 2011, at 1:35 AM, Christian Stocker > wrote: > >> Hi >> >> It's about the two tests in >> >> http://svn.php.net/viewvc/php/php-src/branches/PHP_5_3/ext/simplexml/tests/008.phpt >> and >> http://svn.php.net/viewvc/php/php-src/branches/PHP_5_3/ext/simplexml/tests/bug48601.phpt >> >> which contractited themself and we had back and forth commits due to this. >> >> So we had the following history >> >> 5.3.0 old behaviour >> ... >> 5.3.3 new behaviour >> ... >> 5.3.7 old behaviour >> ... >> 5.4.0 new behaviour >> >> >> The main question now is, should we introduce back the new behavior in >> the 5.3 line? Since the new behaviour was already there since more than >> a year, but now isn't, i'm not sure, what to do. >> >> The good point about the new behaviour is that it's almost BC, so going >> from the old to the new shouldn't cause much problems, but vice versa it >> does. The reason is that if you do an valid xpath query which returns >> zero results, the old behaviour returned false, the new one returns an >> empty array, so before 5.3.3 you had to write >> >> *** >> $res = $sx->query("/some/xpath"); >> >> if ($res) { >> foreach($res as $node) {} >> } >> *** >> >> Which still will work with the new behaviour, but that "if" is not >> really needed with the new behaviour, so going back to the old one may >> break if nothing is found (there may be problems when you asked for >> "zero-results" with checking if $res is false) >> >> Anyway, I'd appreciate some opinions about introducing back the new >> beehaviour in 5.3 or not (I guess 5.3.8 wasn't the last 5.3 release :)) >> >> chregu >> >> >> -- >> Liip AG // Feldstrasse 133 // CH-8004 Zurich >> Tel +41 43 500 39 81 // Mobile +41 76 561 88 60 >> www.liip.ch // blog.liip.ch // GnuPG 0x0748D5FE >> >> >> -- >> PHP Internals - PHP Runtime Development Mailing List >> To unsubscribe, visit: http://www.php.net/unsub.php -- Liip AG // Feldstrasse 133 // CH-8004 Zurich Tel +41 43 500 39 81 // Mobile +41 76 561 88 60 www.liip.ch // blog.liip.ch // GnuPG 0x0748D5FE -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php