encode jobs that have an array as job value

[patrickjahns]

Description

If the argument of a job is an array, json_encode it

Related Issue

• Fixes Error thrown when running occ background:queue:status #35222

Motivation and Context

Fix #35222

How Has This Been Tested?

• 🤖

Screenshots (if appropriate):

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Database schema changes (next release will require increase of minor version instead of patch)
• Breaking change (fix or feature that would cause existing functionality to change)
• Technical debt
• Tests only (no source changes)

Checklist:

• Code changes
• Unit tests added
• Acceptance tests added
• Documentation ticket raised:

Open tasks:

• Backport (if applicable set "backport-request" label and remove when the backport was done)

[PVince81]

👍

[individual-it]

---------- begin diff ----------
--- Original
+++ New
@@ -45,8 +45,8 @@
 	}
 
 	private function getJobArgumentAsString($argument) {
-		if (is_array($argument)) {
-			$argument = json_encode($argument);
+		if (\is_array($argument)) {
+			$argument = \json_encode($argument);
 		}
 		return $argument;
 	}

----------- end diff -----------

[codecov]

Codecov Report

Merging #35224 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master   #35224      +/-   ##
============================================
+ Coverage     65.53%   65.54%   +<.01%     
- Complexity    18645    18647       +2     
============================================
  Files          1218     1218              
  Lines         70540    70544       +4     
  Branches       1288     1288              
============================================
+ Hits          46231    46235       +4     
  Misses        23932    23932              
  Partials        377      377

Flag	Coverage Δ	Complexity Δ
#javascript	53.69% <ø> (ø)	0 <ø> (ø)	⬇️
#phpunit	66.9% <100%> (ø)	18647 <2> (+2)	⬆️

Impacted Files	Coverage Δ	Complexity Δ
core/Command/Background/Queue/Status.php	100% <100%> (ø)	6 <2> (+2)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7e920e3...c46bff8. Read the comment docs.

[codecov]

Codecov Report

Merging #35224 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master   #35224      +/-   ##
============================================
+ Coverage     65.53%   65.54%   +<.01%     
- Complexity    18645    18647       +2     
============================================
  Files          1218     1218              
  Lines         70540    70544       +4     
  Branches       1288     1288              
============================================
+ Hits          46231    46235       +4     
  Misses        23932    23932              
  Partials        377      377

Flag	Coverage Δ	Complexity Δ
#javascript	53.69% <ø> (ø)	0 <ø> (ø)	⬇️
#phpunit	66.9% <100%> (ø)	18647 <2> (+2)	⬆️

Impacted Files	Coverage Δ	Complexity Δ
core/Command/Background/Queue/Status.php	100% <100%> (ø)	6 <2> (+2)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7e920e3...c46bff8. Read the comment docs.

[phil-davis]

Backport stable10 PR #35225
Backport release-10.2.9.0 PR #35228

[individual-it]

CC @paurakhsharma please adjust the tests you are writing accordingly