[stable10] Update backported code from 33302 33859 now we have PHP7

[phil-davis]

Description

PR #33302 used some PHP7 specific code, e.g. "null coalescing operator" https://www.php.net/manual/en/migration70.new-features.php

So that had to be backported differently in stable10 PR #33859

This PR applies the "null coalescing operator" changes to stable10 to make these files the same in stable10 and master

Related Issue

Aligning core master and stable10

Motivation and Context

How Has This Been Tested?

CI

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]

Brrr, I remember undoing these when backporting said PRs...

👍

[phil-davis]

Yes, life would have been much easier if PHP5.6 security support had not been extended an extra year - we could have dropped it a long time ago and not had these backport diffs.