When viewing a civiCRM contact at a URL like this:

contact/view?reset=1&cid=50786

The following PHP Notice is thrown:

Notice: Undefined index: id in commerce_civicrm_civicrm_links() (line 105 of commerce_civicrm.module).

thanks!

CommentFileSizeAuthor
#5 1897654-5-undefined-index-cid.patch561 bytestanmayk
#1 notice-1897654-1.patch633 bytesdavidwbarratt

Comments

davidwbarratt’s picture

davidwbarratt commented
Status: Active » Needs review
StatusFileSize
new notice-1897654-1.patch633 bytes

The attached patch first checks to make sure that the 'id' index is not empty.

thanks!

muhleder’s picture

muhleder commented
Issue summary: View changes
Status: Needs review » Fixed

Yup, looks good.

Status: Fixed » Closed (fixed)

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

bryanhidalgo’s picture

bryanhidalgo commented

I applied the patch but still having the same problem. when going to admin/commerce/config/civicrm i get a white screen, the error persists: Notice : Undefined index: cid in commerce_civicrm_civicrm_links()

tanmayk’s picture

tanmayk
he/him
Primary language English
Location Maharashtra
commented
Status: Closed (fixed) » Needs review
StatusFileSize
new 1897654-5-undefined-index-cid.patch561 bytes

Patch for Notice: undefined index cid against 7.x branch.

tanmayk’s picture

tanmayk
he/him
Primary language English
Location Maharashtra
commented
Title: Notice: Undefined index: id in commerce_civicrm_civicrm_links() (line 105 of commerce_civicrm.module). » Undefined index warnings in commerce_civicrm_civicrm_links().

Changing the name just to make it not limited for one index only.

kebz’s picture

kebz commented

Do I use both patches for this? Or just this one? > 1897654-5-undefined-index-cid.patch

Notice: Undefined index: cid in commerce_civicrm_civicrm_links() (line 100 of /homepages/30/d513154960/htdocs/OVLVX_com/BLove/sites/all/modules/commerce_civicrm/commerce_civicrm.module).

millionleaves’s picture

millionleaves commented

I applied #5 and the error has stopped appearing. Didn't apply #1.

kurund’s picture

kurund commented
Status: Needs review » Closed (outdated)