Closed (won't fix)
Project:
XML parser
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
27 Oct 2010 at 00:03 UTC
Updated:
23 Oct 2013 at 02:41 UTC
Jump to comment: Most recent file
Comments
Comment #1
greg.harveyAwesome, thanks for kicking this off! =)
Comment #2
ss81 commentedUnfortunately this patch doesn't work to me: I can't implement it because of patch process issues (6 out of 6 hunks FAILED). So, I moved changes manually and added some more changes.
Result is in attach.
Comment #3
paulgemini commentedsubscribe
Comment #4
greg.harveyI'm not sure if this is still a valid project. Feeds is so much more flexible these days, this module used to provide a set of custom tools for complex and non-standard XML formats that the old Feeds API didn't handle, but I think the Feeds module can probably do that anyway now.
Not sure if it's worth making a D7 XML Parser version. Will investigate.
Comment #5
paulgemini commentedI think I kind of agree. I'm using Feeds + XPath for XML parsing. It works fairly well and while there are a few features that need to be fixed (most relate to the shaky versions of Date/Location/Ctools that are ever evolving), it pretty much does the job.
Comment #6
greg.harveyOk, so for now I'm going to close this - I've updated the project page. If someone discovers a compelling reason - something this module did in D6 that D7 equivalents cannot handle - I'm open to re-considering.
Comment #7
ss81 commentedHi Greg,
The main problem which I found in D7 is that I can't parse file/string into array with one function. It's much easier to use this module then write almost the same code again. Therefore I prefer to use functions of this module when I work with SOAP.
Version for D7 works well for me so far. I guess it could be good to commit version for D7 into repository to have it not only in attached ZIP file.
Comment #8
uberhacker commentedHere is the tarball version of xml_parser.zip. In my case, I didn't have unzip installed on the server and the sysadmin wasn't available to install the package.