When not using a multivalued date field. The following notices are thrown.

"Notice: Undefined index: multiple_value in _addtocalendar_preprocess_field() (line 38 of [..]/addtocalendar/includes/addtocalendar.build.inc)"

and

"Notice: Undefined index: multiple_value in _addtocalendar_preprocess_field() (line 172 of
[..]/addtocalendar/includes/addtocalendar.build.inc)"

This is due to the build functions referencing the "multiple_value" setting that is not set because the date field is not a multiple value date field.

CommentFileSizeAuthor
#2 addtocalendar-multivalued_date_field_notices-2878558-2.patch1.63 KBchrissnyder

Comments

chrissnyder created an issue. See original summary.

chrissnyder’s picture

chrissnyder
he/him
commented
Status: Active » Needs review
StatusFileSize
new addtocalendar-multivalued_date_field_notices-2878558-2.patch1.63 KB

This patch should correct the notices that are being thrown.

chrissnyder’s picture

chrissnyder
he/him
commented

Note: The patches in this issue and #2878572: AddToCalendar rendering is inconsistent and throws notices overlap. So depending on which one gets applied first the other may have to be re-rolled.

gg24’s picture

gg24 commented
Assigned: Unassigned » gg24

  • gg24 committed 97841de on 8.x-2.x authored by chrissnyder 
    Issue #2878558 by chrissnyder, gg24: Notice thrown when not using a...
gg24’s picture

gg24 commented
Assigned: gg24 » Unassigned
Status: Needs review » Reviewed & tested by the community

I have re-rolled above patch attached. This patch works as intended. Issuing RTBC.

Thanks!

gg24’s picture

gg24 commented
Status: Reviewed & tested by the community » Fixed

Fixing this issue.

Thanks

Status: Fixed » Closed (fixed)

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