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

neclimdul’s picture

Status: Active » Fixed

I think this is actually possible now! Wow that happened a lot faster than I thought.

b-prod’s picture

Status: Fixed » Patch (to be ported)
StatusFileSize
new3.43 KB

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.

neclimdul’s picture

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.

andrewlevine’s picture

Status: Patch (to be ported) » Fixed

We can do this now...reopen and clarify if there was anything else you wanted done with this.

b-prod’s picture

Category: task » bug
Priority: Normal » Critical
Status: Fixed » Needs review
StatusFileSize
new939 bytes

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.

neclimdul’s picture

Status: Needs review » Fixed

commited

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.