When 'refresh feed on creation' is selected, the $feed->settings variable is not present in _feedapi_invoke_refresh(). Using the $settings variable instead solves the problem. See patch.

CommentFileSizeAuthor
feedapi_settings.patch1.46 KBalex_b

Comments

alex_b’s picture

alex_b commented

BTW: This bug can have funky symptoms: e. g. in my case all feed items where the present date although i had set to use the feed item publish date. Basically, all settings fall back to code-level default (mind: NOT content type default, as there are NO settings at all).

aside:

It's best practice to call
feedapi_get_settings() with $node_type parameter, as this allows the fall-back-to-content-type-setting mechanism in the function to kick in.

aron novak’s picture

aron novak
Primary language Hungarian
Location Hungary, Budapest
commented
Status: Active » Fixed

Thanks, it's fixed for both 5.x and 6.x

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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