Option to choose if the feed url is required or not like in any CCK field.

The change from this:
'#required' => TRUE,

to this
'#default_value' => $form['#node']->feed->url ? $form['#node']->feed->url : $form['#post']['feedapi']['feedapi_url'],

is a ugly hack an creates useless rows int the table and some errors:

warning: file_get_contents(http:): failed to open stream: No such file or directory in /srv/www/htdocs/drupal-5.2/modules/feedapi/parser_simplepie/simplepie.inc on line 6716.
warning: Invalid argument supplied for foreach() in /srv/www/htdocs/drupal-5.2/modules/feedapi/feedapi.module on line 805.
warning: Wrong parameter count for array_merge() in /srv/www/htdocs/drupal-5.2/modules/feedapi/feedapi_item/feedapi_item.module on line 40.
warning: in_array(): Wrong datatype for second argument in /srv/www/htdocs/drupal-5.2/modules/feedapi/feedapi_item/feedapi_item.module on line 40.

Comments

budda’s picture

budda
Primary language English
Location Cheshire
commented

If you plan to aggregate a feed, surely the feed url is a required core element of the operation?
I don't understand why you would not want the feed url to be specified??

xpete’s picture

xpete commented

Quoting my own text from this: http://drupal.org/node/172131

"I'm using the content type created by the project module to create the feed, so i can have 1 feed per each project.
the content type for feed items is "story"."

xpete’s picture

xpete commented

I still got one error:

"warning: file_get_contents(http:): failed to open stream: No such file or directory in /srv/www/htdocs/drupal-5.2/modules/feedapi/parser_simplepie/simplepie.inc on line 6716."

xpete’s picture

xpete commented
Status: Active » Closed (fixed)