All date fields where display type is not 'date_default' or 'date_interval' are set to 'date_default' by the database update 7000 (even those that have been hidden). This causes the fields to be displayed as well as the following notice:

Notice: Undefined index: hidden in date_default_formatter_settings_summary() (line 114 of sites/all/modules/date/date_admin.inc).

CommentFileSizeAuthor
#7 date_hidden_fix_patch-1404494-7.patch.gz547 bytesthinkyhead
#1 date-hidden-fields-1404494.patch765 bytesbyrond

Comments

byrond’s picture

byrond commented
StatusFileSize
new date-hidden-fields-1404494.patch765 bytes

This patch adds 'hidden' as an exception of fields that are changed to the 'date_default' display.

karens’s picture

karens commented
Status: Active » Fixed

Committed. Thanks!

Status: Fixed » Closed (fixed)

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

tamerzg’s picture

tamerzg commented
Status: Closed (fixed) » Active

I am getting this same error with latest dev. Problem is with 'date_default' which exists after D6 migration but is not returned by date_format_type_options().

karens’s picture

karens commented
Status: Active » Postponed (maintainer needs more info)

I have no idea what you are saying.

thinkyhead’s picture

thinkyhead commented
Status: Needs work » Postponed (maintainer needs more info)

Nevermind this comment. See the next one for my solution.

thinkyhead’s picture

thinkyhead commented
Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new date_hidden_fix_patch-1404494-7.patch.gz547 bytes

Here is a patch that will fix any hidden date fields that were improperly made 'default' during the period when date_update_7000 didn't have 'hidden' as an exception. This is a helpful alternative to having admins figure out that they need to edit the fields manually at admin/structure/types/manage/[type]/display.

Status: Needs review » Needs work

The last submitted patch, date-hidden-fields-1404494.patch, failed testing.

thinkyhead’s picture

thinkyhead commented
Status: Postponed (maintainer needs more info) » Needs work

Darn. Well, that's the general idea. It's not catching some cases, so I'm still working on a definitive patch anyway.