Page MenuHomePhorge

Fold WalletCharge and WalletCheck into one
AbandonedPublic

Authored by mollekopf on Jun 3 2024, 11:31 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Oct 1, 2:05 AM
Unknown Object (File)
Wed, Sep 11, 6:31 PM
Unknown Object (File)
Sun, Sep 8, 2:16 AM
Unknown Object (File)
Sun, Sep 8, 1:30 AM
Unknown Object (File)
Aug 25 2024, 9:07 PM
Unknown Object (File)
Aug 21 2024, 12:15 PM
Unknown Object (File)
Aug 18 2024, 10:21 AM
Unknown Object (File)
Aug 16 2024, 3:26 AM
Subscribers

Details

Reviewers
None
Group Reviewers
Restricted Project

Diff Detail

Repository
rK kolab
Branch
dev/mollekopf
Lint
Lint Skipped
Unit
No Test Coverage
Build Status
Buildable 47544
Build 18018: arc lint + arc unit

Event Timeline

mollekopf created this revision.
mollekopf added a reviewer: Restricted Project.Jun 3 2024, 11:32 AM

Will require at least some more cleanup (remove WalletCharge job and command).

Maybe it makes sense, if huge number of jobs is not an issue. Also we should probably do dispatch($wallet->id) instead of dispatch($wallet) (and do find() instead of refresh()).

One more note. Currently if topUpWallet() throws exception (for any reason, e.g. Mollie connection issues) the WalletCharge job will try to top up the wallet again. Here it won't happen, so it will be more fragile to temporary issues with the payment provider. Catching the exception and invoking a separate WalletCharge job might make sense.

That means at least I wouldn't remove job:walletcharge command (and the job), it might come in handy.