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.
Merge request !30Issue#3357831: Resolve PHP Deprecation error. → (Closed) created by sakthi_dev
Open in Jenkins → Open on Drupal.org →
Core: 9.5.5 + Environment: PHP 7.4 & MySQL 5.7
last update almost 2 years ago
15 pass
Status changed to Needs review almost 2 years ago10:40am 3 May 2023
Comment almost 2 years 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 almost 2 years ago
15 pass
Status changed to RTBC almost 2 years ago9:53am 9 May 2023
Comment almost 2 years 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 almost 2 years ago
15 pass
Comment over 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 over 1 year ago →
🇺🇸United States rymcveigh
This issue fork worked for me as well.
Comment over 1 year ago →
🇺🇸United States Ariun
This changes works for me too. I would say that it is good to merge.
First commit to issue fork.
Comment 4 months ago →
System Message
japerry → closed merge request !30
Merge request !40Resolve #3357831 "Php 8.2 compatibility" → (Merged) created by japerry
Pipeline finished with Skipped
4 months ago
#370324
Comment 4 months ago →
🇺🇸United States japerry KVUO
Comment 4 months ago →
System Message

japerry → committed b2bd930b on 4.x
Issue #3357831 by sakthi_dev: PHP 8.2 compatibility
Comment 4 months ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

Problem/Motivation

Merge Requests

!40PHP 8.2 compatibility
Merged

!30PHP 8.2 compatibility
Closed

Comments & Activities

PHP 8.2 compatibility

Problem/Motivation

Merge Requests

!40PHP 8.2 compatibilityMerged

!30PHP 8.2 compatibilityClosed

Comments & Activities

!40PHP 8.2 compatibility
Merged

!30PHP 8.2 compatibility
Closed