[jira] Created: (JAXEN-200) XPath support for Jericho HTML parser

XPath support for Jericho HTML parser
-------------------------------------

                 Key: JAXEN-200
                 URL: http://jira.codehaus.org/browse/JAXEN-200
             Project: jaxen
          Issue Type: Improvement
         Environment: Any
            Reporter: Sujit Pal
         Attachments: DocumentNavigator.java, JerichoNavigatorTest.java, JerichoXPath.java

I needed to build XPath support for Jericho locally, so I created custom implementations of Navigator and XPath to do this. I would like to contribute the code to the Jaxen project. I have attached the three java files (DocumentNavigator.java, JerichoXPath.java and a test JerichoNavigatorTest.java).

The first two would go into the main directory as a sibling of jdom and dom4j packages and the last one would go into the test directory.

Additional dependencies are jericho-html-2.5.jar, commons-logging.jar and log4j.jar (although the last two can be removed if you remove/replace the log.warn messages in the catch blocks in DocumentNavigator).


--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email

     [ http://jira.codehaus.org/browse/JAXEN-200?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Elliotte Rusty Harold closed JAXEN-200.
---------------------------------------

> XPath support for Jericho HTML parser
> -------------------------------------
>
>                 Key: JAXEN-200
>                 URL: http://jira.codehaus.org/browse/JAXEN-200
>             Project: jaxen
>          Issue Type: Improvement
>         Environment: Any
>            Reporter: Sujit Pal
>            Assignee: Elliotte Rusty Harold
>         Attachments: DocumentNavigator.java, JerichoNavigatorTest.java, JerichoXPath.java
>
>
> I needed to build XPath support for Jericho locally, so I created custom implementations of Navigator and XPath to do this. I would like to contribute the code to the Jaxen project. I have attached the three java files (DocumentNavigator.java, JerichoXPath.java and a test JerichoNavigatorTest.java).
> The first two would go into the main directory as a sibling of jdom and dom4j packages and the last one would go into the test directory.
> Additional dependencies are jericho-html-2.5.jar, commons-logging.jar and log4j.jar (although the last two can be removed if you remove/replace the log.warn messages in the catch blocks in DocumentNavigator).

--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email

     [ http://jira.codehaus.org/browse/JAXEN-200?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Elliotte Rusty Harold resolved JAXEN-200.
-----------------------------------------

      Assignee: Elliotte Rusty Harold
    Resolution: Won't Fix

Adding all the navigators directly into Jaxen was a mistake, and one we are hoping to cure in the near future. It causes significant problems for other projects that depend on jaxen because they then pull in dependencies on XOM, JDOM, dom4j, and so on that they usually don't need. This is especially true for maven based projects. Your proposal would, I'm afraid, make the situation worse by adding additional dependencies to everyone who builds jaxen.

 I suspect we'll keep the DOM navigator in the core since it only really depends on the JDK these days, but we'll move the others into new projects.

Once this move is complete, perhaps we could consider hosting your navigator as a subproject of jaxen on Codehaus or some such, but it really shouldn't be bundled with the core.

> XPath support for Jericho HTML parser
> -------------------------------------
>
>                 Key: JAXEN-200
>                 URL: http://jira.codehaus.org/browse/JAXEN-200
>             Project: jaxen
>          Issue Type: Improvement
>         Environment: Any
>            Reporter: Sujit Pal
>            Assignee: Elliotte Rusty Harold
>         Attachments: DocumentNavigator.java, JerichoNavigatorTest.java, JerichoXPath.java
>
>
> I needed to build XPath support for Jericho locally, so I created custom implementations of Navigator and XPath to do this. I would like to contribute the code to the Jaxen project. I have attached the three java files (DocumentNavigator.java, JerichoXPath.java and a test JerichoNavigatorTest.java).
> The first two would go into the main directory as a sibling of jdom and dom4j packages and the last one would go into the test directory.
> Additional dependencies are jericho-html-2.5.jar, commons-logging.jar and log4j.jar (although the last two can be removed if you remove/replace the log.warn messages in the catch blocks in DocumentNavigator).

--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email

    [ http://jira.codehaus.org/browse/JAXEN-200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=174080#action_174080 ]

Sujit Pal commented on JAXEN-200:
---------------------------------

Sure, that makes sense...if you want to go that route, would it make sense to have some documentation on how to build an adapter to work with a different object model?

What helped me during building the Jericho adapter was the examples, and single stepping through the Jaxen code to see what I needed to override. The extension points are not very well defined, so its hard to figure out what to override and what not to. A programmatic way to handle this would be to make some of the overridable methods abstract rather than returning null or UnsupportedOperationException. Alternatively, a one page writeup explaining what methods are needed to be overriden would be a great help.

Please let me know if you would like me to open a separate bug for this.

> XPath support for Jericho HTML parser
> -------------------------------------
>
>                 Key: JAXEN-200
>                 URL: http://jira.codehaus.org/browse/JAXEN-200
>             Project: jaxen
>          Issue Type: Improvement
>         Environment: Any
>            Reporter: Sujit Pal
>            Assignee: Elliotte Rusty Harold
>         Attachments: DocumentNavigator.java, JerichoNavigatorTest.java, JerichoXPath.java
>
>
> I needed to build XPath support for Jericho locally, so I created custom implementations of Navigator and XPath to do this. I would like to contribute the code to the Jaxen project. I have attached the three java files (DocumentNavigator.java, JerichoXPath.java and a test JerichoNavigatorTest.java).
> The first two would go into the main directory as a sibling of jdom and dom4j packages and the last one would go into the test directory.
> Additional dependencies are jericho-html-2.5.jar, commons-logging.jar and log4j.jar (although the last two can be removed if you remove/replace the log.warn messages in the catch blocks in DocumentNavigator).

--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email

    [ http://jira.codehaus.org/browse/JAXEN-200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=174114#action_174114 ]

Elliotte Rusty Harold commented on JAXEN-200:
---------------------------------------------

We could certainly consider that. We'd have to think about whether that's likely to break any existing navigators. I.e. are there any navigators that depend on these methods being implemented in the superclass? And of course more documentation is always appreciated.

> XPath support for Jericho HTML parser
> -------------------------------------
>
>                 Key: JAXEN-200
>                 URL: http://jira.codehaus.org/browse/JAXEN-200
>             Project: jaxen
>          Issue Type: Improvement
>         Environment: Any
>            Reporter: Sujit Pal
>            Assignee: Elliotte Rusty Harold
>         Attachments: DocumentNavigator.java, JerichoNavigatorTest.java, JerichoXPath.java
>
>
> I needed to build XPath support for Jericho locally, so I created custom implementations of Navigator and XPath to do this. I would like to contribute the code to the Jaxen project. I have attached the three java files (DocumentNavigator.java, JerichoXPath.java and a test JerichoNavigatorTest.java).
> The first two would go into the main directory as a sibling of jdom and dom4j packages and the last one would go into the test directory.
> Additional dependencies are jericho-html-2.5.jar, commons-logging.jar and log4j.jar (although the last two can be removed if you remove/replace the log.warn messages in the catch blocks in DocumentNavigator).

--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email