doPaymentsFailed() refers to wrong variable

artfulrobot / uk.artfulrobot.civicrm.gocardless

A CiviCRM extension providing GoCardless integration to handle UK Direct Debits.

GNU Affero General Public License v3.0

5 stars 18 forks source link

doPaymentsFailed() refers to wrong variable #25

Closed wmortada closed 6 years ago

wmortada commented 6 years ago

This looks like a typo to me. The function doPaymentsFailed() refers to a variable $update but this isn't defined. I think it should refer to $contribution.

I haven't tested this myself, but spotted it while reading through the code.

artfulrobot commented 6 years ago

Thanks @wmortada and happy New Year!

wmortada commented 6 years ago

It's a pleasure! Happy new year!