Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Payment (Mutation) Cron Error #11

Closed
HugotonHosting opened this issue Nov 3, 2019 · 4 comments
Closed

Payment (Mutation) Cron Error #11

HugotonHosting opened this issue Nov 3, 2019 · 4 comments
Labels
enhancement New feature or request

Comments

@HugotonHosting
Copy link

Hello,

We have set up the WHMCS integration but receive the below error on cron.php - no payments/mutations are moved from WHMCS to Moneybird. Please can you assist?

[WHMCS Application] ERROR: TypeError: Argument 1 passed to Picqer\Financials\Moneybird\Entities\FinancialMutation::getVersions() must be of the type array, null given, called in hugotonhosting.co.uk/public_html/members/modules/addons/moneybird/cron.php on line 131 and defined in hugotonhosting.co.uk/public_html/members/modules/addons/moneybird/vendor/picqer/moneybird-php-client/src/Picqer/Financials/Moneybird/Actions/Synchronizable.php:43 Stack trace: #0 hugotonhosting.co.uk/public_html/members/modules/addons/moneybird/cron.php(131): Picqer\Financials\Moneybird\Entities\FinancialMutation->getVersions(NULL) #1 hugotonhosting.co.uk/public_html/members/modules/addons/moneybird/cron.php(214): processFinancialMutations(Array, true) #2 {main} {"exception":"[object] (TypeError(code: 0): Argument 1 passed to Picqer\Financials\Moneybird\Entities\FinancialMutation::getVersions() must be of the type array, null given, called in hugotonhosting.co.uk/public_html/members/modules/addons/moneybird/cron.php on line 131 at hugotonhosting.co.uk/public_html/members/modules/addons/moneybird/vendor/picqer/moneybird-php-client/src/Picqer/Financials/Moneybird/Actions/Synchronizable.php:43)"} []

@ju5t
Copy link
Contributor

ju5t commented Nov 4, 2019

It works the other way round. It uses MoneyBird as a single source of truth for financial mutations. It will not sync payments from WHMCS into MoneyBird. You won't be able to get your ledgers synced if you would.

I think this can happen if you have no mutations in MoneyBird yet. Can you confirm this?

@HugotonHosting
Copy link
Author

Ah okay, thanks for confirming. I added a test mitation in Moneybird and the error went away.

We won't be using Moneybird though unfortunately as we cannot import our PayPal transactions in automatically.

Thanks for your help!

@ju5t
Copy link
Contributor

ju5t commented Nov 4, 2019

PayPal support would have been a good feature for us too. You could raise a feature request but they often take time, unfortunately.

I will leave this issue open as we could make this behaviour more obvious. Thanks for raising this issue!

@ju5t ju5t added the enhancement New feature or request label Nov 4, 2019
@ju5t
Copy link
Contributor

ju5t commented Nov 14, 2019

This has been solved by PR #12.

@ju5t ju5t closed this as completed Nov 14, 2019
@ghost ghost mentioned this issue Apr 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants