Closed (fixed)
Project:
Feeds
Version:
7.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
16 Jan 2012 at 09:53 UTC
Updated:
25 May 2012 at 17:40 UTC
Jump to comment: Most recent file
Custom fetchers that doesn't need a source configuration, fails per default. The problem is that if sourceForm doesn't return a form, the $node->feeds value ends up not being an array. That causes problems later when feeds does a += defaultConfig. It can be worked around by adding a sourceForm that does:
return array(
'#type' => 'hidden',
'#value' => 'dummy',
);
But this still creates an empty Feeds fieldset.
Attached patch fixes the issue, and doesn't add the fieldset if it has no content.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | no_source_form-1406260-2.patch | 1.36 KB | logaritmisk |
| no_source_form.patch | 1.9 KB | xen |
Comments
Comment #1
franzThis patch seems to fix the issue only on node forms, what about standalone? I get the same error, using the workaround.
Comment #2
logaritmisk commentedHad the same issue, but with a standalone. After I found, and fixed, the issue I found this issue. My patch do almost the same thing, but should work for both standalone and for importers attached to nodes.
Comment #3
franzThis is highly more appropriate, thanks logaritmisk, I'll test this and report back.
Comment #4
franzTested, fixes the issue. Committed.