Posted by neclimdul on August 14, 2009 at 6:56pm
| Project: | FeedAPI Extensible Parser |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | critical |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
CSV would be pretty handy but I'm sure there are other formats that are not xml that would be pretty handy to pull into FeedAPI. Can you say killer app? ;)
Comments
#1
I think this is actually possible now! Wow that happened a lot faster than I thought.
#2
Hi!
Here is a patch that allow other modules to add their specific form in the eparser fieldset (in the node edit form), through a new hook: hook_eparser_settings_form. The settings are stored (serialized) in the database, in a new field: settings.
I needed that for a CSV parser I created within eparser, in an other module, to specify the CSV delimiter. If you agree with my patch and is interested with CSV parser, I can also include the CSV parser in epaser core.
#3
Awesome! I'd come across a similar sort of need while trying to implement a JSON parser.
2 comments. 1) I'd kinda like the form function to be part of the plugin object similar to the way views does it so things are a little more encapsulated. 2) I'd kind of like to do #560162: Move UI to node tab first so this is a little easier to manage.
#4
We can do this now...reopen and clarify if there was anything else you wanted done with this.
#5
Hi!
New way to store settings via multistep form is a great idea. But there is an omission in the code: the settings are not loaded when instancing parser.
Correct this is very quick and simple, the patch only modify 1 line and add another one.
#6
commited
#7
Automatically closed -- issue fixed for 2 weeks with no activity.