-
Notifications
You must be signed in to change notification settings - Fork 188
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[WIP] [E4 Xpath] Replace apache.commons.jxpath by javax.xml.xpath #2387
base: master
Are you sure you want to change the base?
Conversation
Instead of using a queue, one can simply use eAllContents() to get a tree iterator over all elements. One one is created for each node, which is then attached to its container.
@HannesWell As the tests will hopefully show, the current state should support generic XPaths such as |
Test Results 1 818 files ±0 1 818 suites ±0 1h 30m 3s ⏱️ - 3m 3s For more details on these failures, see this check. Results for commit dc9d2d4. ± Comparison against base commit 4e0740d. This pull request removes 3 and adds 6 tests. Note that renamed tests count towards both.
|
This is a continuation of #2290 and attempts to perform an XPath evaluation of the EObjects via the native javax.xml.xpath API rather than relying on JXPath. The Java model is converted to an XML structure where each EObject is mapped to a DOM node. The XPath is applied on the root node and the resulting nodes are then converted back to EObjects.