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

Zero contribution amount since the new upgrade #105

Closed tapashdatta closed 3 years ago

tapashdatta commented 3 years ago

Since the upgraded to latest release a last week, many of my direct debit payment registering as zero value, but nothing gets registered to GC.

All I see in the log is following:

Warning: Invalid argument supplied for foreach() in CRM_Price_BAO_LineItem::format() (line 304 of /home/webadmin/public_html/cmm/sites/all/modules/civicrm/CRM/Price/BAO/LineItem.php).

Warning: Invalid argument supplied for foreach() in CRM_Price_BAO_PriceSet::getLine() (line 1718 of /home/webadmin/public_html/cmm/sites/all/modules/civicrm/CRM/Price/BAO/PriceSet.php).

artfulrobot commented 3 years ago

@tapashdatta "latest release" Do you mean 1.10.1? What Civi version?

And when does this occur? Setting up new DD, or in the webhook/IPN code?

tapashdatta commented 3 years ago

Hi @artfulrobot yes 1.10.1 with 5.33 During the first signup. Actually no signup goes to GC. Only created a 0.00£ Recurring contribution in civi as completed.

artfulrobot commented 3 years ago

I've just tested a simple other-amount monthly donation form on my buildkit and all is fine on 5.35. I also tested on a client's site that's using 5.31.

Can you double check your contribution page's config?

It's complaining about line items not being correct. What version did you upgrade from?

I've done one more commit since 1.10.1 which fixes the error I saw at https://chat.civicrm.org/civicrm/pl/5oc8esmyd3yktn5duphwgyfyny

But nobody else has complained about that and the symptoms don't match this issue.

If you have more of a stack trace - i.e the lines within the GC extension that would be helpful.

artfulrobot commented 3 years ago

Note: the HEAD of the dev branch includes extra logging.

tapashdatta commented 3 years ago

Seems not happening anymore..