search

eileenmcnaughton / nz.co.fuzion.civixero

Synchronisation between CiviCRM & Xero
7 stars 14 forks source link

Getting an error when running the main Civi cron job #123

Open Upperholme opened 1 year ago

Upperholme commented 1 year ago

Over the weekend I updated to the latest master version of this extension, which looks to have fixed issues with my sync to Xero. However my main cron job has stopped working since.

I'm seeing the following error reported:

TypeError: Argument 2 passed to CRM_Civixero_Invoice::mapCancelled() must be of the type int, null given, called in /home/inca/sites/default/sites/all/extensions/nz.co.fuzion.civixero-master/CRM/Civixero/Invoice.php on line 426 in CRM_Civixero_Invoice->mapCancelled() (line 273 of /home/inca/sites/default/sites/all/extensions/nz.co.fuzion.civixero-master/CRM/Civixero/Invoice.php).

Any pointers on this issue gratefully received.

eileenmcnaughton commented 1 year ago

I pushed a fix

Upperholme commented 1 year ago

Great, thanks. I'll give it a spin.

eileenmcnaughton commented 1 year ago

I think we can close this now? 2.1.4 working afaik