jkesselm opened a new pull request, #12:
URL: https://github.com/apache/xalan-java/pull/12
**select="dyn:evaluate('document(foo)')"** has been returning empty-set
since dyn:evaluate is only aware of XPath core functions, not XSLT additions or
extensions, and **document()** was added by XSLT. We may want to support the
additional functions in the future, but adding that would be messy.
In any case, responding to a syntax error by returning the empty set is not
good behavior for an XPath interpreter.
Better to throw an exception and ask the user to fix the stylesheet,
methinks.
(Yes, a negative test should be added for this. We don't currently have an
**exslt-fail** bucket; we probably should, and should have an appropriate
spread of tests therein. If folks want to hold up this fix until I've added
that and a suitable test, that's justifiable.)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@xalan.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@xalan.apache.org
For additional commands, e-mail: dev-h...@xalan.apache.org
