ZF-6280: Wrong return type for Zend_Dom_Query_Css2XPath::transform()

Issue Type: Docs: Problem Created: 2009-04-12T21:10:26.000+0000 Last Updated: 2009-07-29T19:35:03.000+0000 Status: Resolved Fix version(s): - 1.9.0 (31/Jul/09)

Reporter: Matthew Turland (elazar) Assignee: Satoru Yoshida (satoruyoshida) Tags: - Zend_Dom_Query

Related issues: - ZF-6281

Attachments: - patch.diff


The docblock for Zend_Dom_Query_Css2XPath::transform() indicates that the only return type should be a string. However, when multiple expressions are specified, it is possible for it to return an array. It appears from its test suite that this is expected behavior. See the test suite method testTransformShouldReturnMultiplePathsWhenExpressionContainsCommas(). The docblock should be corrected to reflect this.


Posted by Matthew Turland (elazar) on 2009-04-12T21:11:01.000+0000

Patch to correct the docblock issue.

Posted by Matthew Turland (elazar) on 2009-04-12T21:14:57.000+0000

Zend_Dom_Query::query() passes the return value of Zend_Dom_Query_Css2XPath::transform() as the first parameter ($xpathQuery) to Zend_Dom_Query::queryXPath(). The docblock for Zend_Dom_Query::queryXPath() indicates that $xpathQuery may only be a string, but can obviously also be an array.

Posted by Matthew Turland (elazar) on 2009-04-12T21:16:16.000+0000

Updated patch to include fix for Zend_Dom_Query::queryXPath().

Posted by Satoru Yoshida (satoruyoshida) on 2009-07-29T19:35:02.000+0000

Solve in SVN r17292

Have you found an issue?

See the Overview section for more details.


© 2006-2018 by Zend, a Rogue Wave Company. Made with by awesome contributors.

This website is built using zend-expressive and it runs on PHP 7.