PHP 8.2 compatibility

PHP Deprecated: a:5:{s:11:"deprecation";s:78:"explode(): Passing null to parameter #2 ($string) of type string is deprecated";s:5:"class";s:65:"Drupal\Tests\google_analytics\Functional\GoogleAnalyticsBasicTest";s:6:"method";s:31:"testGoogleAnalyticsTrackingCode";s:15:"triggering_file";s:130:"/home/runner/work/acquia_cms/orca-build/docroot/modules/contrib/google_analytics/tests/src/Functional/GoogleAnalyticsBasicTest.php";s:11:"files_stack";a:7:{i:0;s:130:"/home/runner/work/acquia_cms/orca-build/docroot/modules/contrib/google_analytics/tests/src/Functional/GoogleAnalyticsBasicTest.php";i:1;s:77:"/home/runner/work/acquia_cms/orca-build/docroot/sites/simpletest/TestCase.php";i:2;s:77:"/home/runner/work/acquia_cms/orca-build/docroot/sites/simpletest/TestCase.php";i:3;s:91:"/home/runner/work/acquia_cms/orca-build/vendor/phpunit/phpunit/src/Framework/TestResult.php";i:4;s:77:"/home/runner/work/acquia_cms/orca-build/docroot/sites/simpletest/TestCase.php";i:5;s:19:"Standard input code";i:6;s:19:"Standard input code";}} in /home/runner/work/acquia_cms/orca-build/vendor/symfony/phpunit-bridge/Legacy/SymfonyTestsListenerTrait.php on line 284

Comments & Activities

Issue created by @vishalkhode
First commit to issue fork.
Open in Jenkins → Open on Drupal.org →
Core: 9.5.5 + Environment: PHP 7.4 & MySQL 5.7
last update over 1 year ago
15 pass
@sakthi_dev opened merge request.
Status changed to Needs review over 1 year ago10:40am 3 May 2023
Comment over 1 year ago →
🇮🇳India sakthi_dev
Please review.
Open in Jenkins → Open on Drupal.org →
Core: 10.1.x + Environment: PHP 8.2 & MySQL 8
last update over 1 year ago
15 pass
Status changed to RTBC over 1 year ago9:53am 9 May 2023
Comment over 1 year ago →
🇮🇳India rajeshreeputra Pune
Changes looks good, tests passing, moving ahead.
Open in Jenkins → Open on Drupal.org →
Core: 9.5.5 + Environment: PHP 8.1 & MySQL 5.7
last update over 1 year ago
15 pass
Comment about 1 year ago →
🇺🇸United States MegaKeegMan
I can confirm that the issue fork also worked for me. If any maintainers are around, this appears safe to merge
Comment about 1 year ago →
🇺🇸United States rymcveigh
This issue fork worked for me as well.
Comment about 1 year ago →
🇺🇸United States Ariun
This changes works for me too. I would say that it is good to merge.

Problem/Motivation

Comments & Activities