fuzionnz / nz.co.fuzion.transactional

Track delivery, open, click, and bounce for transactional email.
Other
12 stars 16 forks source link

Notice: Undefined index: contact_id #32

Closed mcollopy closed 5 years ago

mcollopy commented 5 years ago

Hi guys - getting a notice on a client site running 5.7.0, extension V1.0 Is this something you've seen before?

Notice: Undefined index: contact_id in CRM_Mailing_Transactional->getContactAndEmailIds() (line 156 of /var/www/gsvdrupal/sites/default/files/civicrm/ext/nz.co.fuzion.transactional-master/CRM/Mailing/Transactional.php). Notice: Undefined index: contact_id in CRM_Mailing_Transactional->getContactAndEmailIds() (line 156 of /var/www/gsvdrupal/sites/default/files/civicrm/ext/nz.co.fuzion.transactional-master/CRM/Mailing/Transactional.php). Notice: Undefined index: returnPath in CRM_Mailing_Transactional->delivered() (line 82 of /var/www/gsvdrupal/sites/default/files/civicrm/ext/nz.co.fuzion.transactional-master/CRM/Mailing/Transactional.php). Notice: Undefined offset: 2 in CRM_Mailing_Transactional->delivered() (line 85 of /var/www/gsvdrupal/sites/default/files/civicrm/ext/nz.co.fuzion.transactional-master/CRM/Mailing/Transactional.php).

Thanks, Mick

stesi561 commented 5 years ago

Hi Mick,

As a first set we'd recommend upgrading to 5.10.3+ or if you are running the ESR version to the version that includes the February security patch.

Can you upgrade to the 1.0.1 version https://civicrm.org/extensions/transactional-emails/101 of the extension and check and see if the Notices persist.

Luke

mcollopy commented 5 years ago

Hi Luke, yes those are things we can and will do anyway.

Was wondering why without any other change that we started noticing this error.

Hence my earlier question - is this something you've seen before?

Thanks

Mick Collopy

Director

m: +61 (0) 420 476 206

e: mailto:mick@audienceware.com

w: http://www.audienceware.com.au/

http://www.audienceware.com.au/

http://www.audienceware.com.au/

---- On Tue, 05 Mar 2019 15:49:57 +1100 Luke Stewart notifications@github.com wrote ----

Hi Mick,

As a first set we'd recommend upgrading to 5.10.3+ or if you are running the ESR version to the version that includes the February security patch.

Can you upgrade to the 1.0.1 version https://civicrm.org/extensions/transactional-emails/101 of the extension and check and see if the Notices persist.

Luke

Luke

— You are receiving this because you authored the thread. Reply to this email directly, https://github.com/fuzionnz/nz.co.fuzion.transactional/issues/32#issuecomment-469536472, or https://github.com/notifications/unsubscribe-auth/ABaEiM_9_c5VQ7OVItxovEoiNRQ7FnUdks5vTfd1gaJpZM4bdj5V.

stesi561 commented 5 years ago

Hi Mike,

I haven't noticed this elsewhere. Did upgrading version and civicrm result in a fix?

Luke

mcollopy commented 5 years ago

Not sure yet Luke, waiting for the client to report another incidence. Just seemed odd that the extension hasn't been updated since 2017, so wouldn't have thought it was version related. Will keep you posted.

stesi561 commented 5 years ago

Hi Mick,

Just checking in did you see any more incidences of this?

Luke