The way form is declared/returned prevents the "Show in event calendar" options from appearing in the respective content-type settings pages.

The attached patch fixes this and reorders the options to match the description.

The long, single line array format makes it a real joy to try to notice the difference ;)

CommentFileSizeAuthor
event.module_18.patch1.85 KBdrewish

Comments

drewish’s picture

drewish commented
Title: "Show in event calendar" isn't appearing in admin/settings/content-types/* » "Show in event calendar" form isn't appearing in admin/settings/content-types/*
Category: feature » bug

Boy, having a tough time getting the issue right the first time today.

gerhard killesreiter’s picture

gerhard killesreiter commented
Status: Needs review » Fixed

Now, that is really strange. i knew this form was broken, but did scratch my head with no result. any idea why this was failing? Anyway: Thanks, I committed it.

scroogie’s picture

scroogie commented

From looking at the patch I guess because a['foo'] = b returns b alone, not a.

gerhard killesreiter’s picture

gerhard killesreiter commented

this is probably the case, but at least a little non-obvious.

Anonymous’s picture

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