If the API is not available anymore, even if the gateway is disabled, the module queue worker continue to try to update payments status and an error is fired.
We should : 1, handle this error; 2, handle a way to continue OR stop updating payments status independently of the gateway status.

CommentFileSizeAuthor
#5 interdiff_4-5.txt1.01 KBguillaume-samtmann
#5 3464618-5-Call-to-a-member-function-fetch-on-null-during-cron-job.patch1.01 KBguillaume-samtmann
#4 3464618-4-Call-to-a-member-function-fetch-on-null-during-cron-job.patch4.25 KBnicolasgraph
#2 3464618-2-Call-to-a-member-function-fetch-on-null-during-cron-job.patch3.39 KBnicolasgraph

Issue fork commerce_alma-3464618

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

NicolasGraph created an issue. See original summary.

nicolasgraph’s picture

nicolasgraph
Primary language French
Location Strasbourg
commented
StatusFileSize
new 3464618-2-Call-to-a-member-function-fetch-on-null-during-cron-job.patch3.39 KB

Here's a patch.

nicolasgraph’s picture

nicolasgraph
Primary language French
Location Strasbourg
commented
Assigned: nicolasgraph » Unassigned
Status: Active » Needs review
nicolasgraph’s picture

nicolasgraph
Primary language French
Location Strasbourg
commented
StatusFileSize
new 3464618-4-Call-to-a-member-function-fetch-on-null-during-cron-job.patch4.25 KB

Fixing call to getPlugin() on string.

guillaume-samtmann’s picture

guillaume-samtmann commented
StatusFileSize
new 3464618-5-Call-to-a-member-function-fetch-on-null-during-cron-job.patch1.01 KB
new interdiff_4-5.txt1.01 KB

nicolasgraph opened merge request !4

  • nicolasgraph committed edaa74d4 on 1.x 
    Issue #3464618 by nicolasgraph, guillaume-samtmann: Call to a member...
nicolasgraph’s picture

nicolasgraph
Primary language French
Location Strasbourg
commented
Status: Needs review » Fixed

Patch #5 won't apply, it is an interdiff.
I created a MR from patch #4 including the suggested change of the interdiff.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.