I've getting these notices whenever I output an iCal feed using Date iCal:

Notice: Undefined index: add_microdata in theme_date_display_single() (line 264 of /drupal_testing/sites/all/modules/date/date.theme).
Notice: Undefined index: add_microdata in template_preprocess_date_display_single() (line 222 of /drupal_testing/sites/all/modules/date/date.theme).

They only appeared when I updated Date to the newest dev build. I'm guessing that this is something to do with new settings added with this new version, but which aren't saved in my existing View.

Fortunately, the fix I found is fairly simple, and I'm pretty sure it doesn't undermine any existing functionality. I'll attach my patch to comment #1.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

coredumperror’s picture

Here's the patch. It just wraps the uses of $variables['add_microdata'] with !empty() checks.

coredumperror’s picture

Status: Active » Needs review
cafuego’s picture

Status: Needs review » Fixed

Very straightforward fix, committed.

Status: Fixed » Closed (fixed)

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

dmsmidt’s picture

Issue summary: View changes
Status: Closed (fixed) » Reviewed & tested by the community

Looks like this never was committed.
Patch works.

cafuego’s picture

Status: Reviewed & tested by the community » Fixed

Ooopsie! Definitely committed now :-)

  • Commit 0b13b1b on 7.x-2.x by cafuego:
    Issue #2115377 by coredumperror: Notice: Undefined index: add_microdata...

Status: Fixed » Closed (fixed)

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