Posted by nonsie on October 27, 2010 at 12:03am
4 followers
| Project: | XML parser |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | task |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (won't fix) |
Issue Summary
Here's a straightforward port from D6 to D7 without any modifications mentioned in http://drupal.org/node/757250#comment-3031868
| Attachment | Size |
|---|---|
| xml_parser_d7.patch | 2.48 KB |
Comments
#1
Awesome, thanks for kicking this off! =)
#2
Unfortunately 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.
#3
subscribe
#4
I'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.
#5
I 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.
#6
Ok, 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.
#7
Hi 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.