PHP 8.1 deprecations

Deprecated: Return type of Drupal\feeds_ex\File\LineIterator::rewind() should either be compatible with SplFileObject::rewind(): void, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in /app/web/modules/contrib/feeds_ex/src/File/LineIterator.php on line 36 Deprecated: Return type of Drupal\feeds_ex\File\LineIterator::valid() should either be compatible with SplFileObject::valid(): bool, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in /app/web/modules/contrib/feeds_ex/src/File/LineIterator.php on line 55 Deprecated: Return type of Drupal\feeds_ex\File\LineIterator::next() should either be compatible with SplFileObject::next(): void, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in /app/web/modules/contrib/feeds_ex/src/File/LineIterator.php on line 47

Comments & Activities

Issue created by @hlopez
Status changed to Needs review over 1 year ago10:03pm 1 June 2023
Open in Jenkins → Open on Drupal.org →
Core: 9.5.5 + Environment: PHP 7.4 & MySQL 5.7
last update over 1 year ago
113 pass
Comment over 1 year ago →
🇺🇸United States hlopez
Open in Jenkins → Open on Drupal.org →
Core: 10.2.x + Environment: PHP 8.1 & MySQL 8
last update 8 months ago
113 pass
Open in Jenkins → Open on Drupal.org →
Core: 9.5.x + Environment: PHP 8.1 & MySQL 5.7
last update 8 months ago
113 pass
Comment 8 months ago →
🇯🇵Japan ptmkenny
📌 Move from arthurkushman/querypath to GravityPDF/querypath Needs review is also necessary to fix PHP 8.1 deprecations since the querypath library currently used breaks on PHP 8.1.
Status changed to RTBC 8 months ago12:21pm 18 April 2024
Comment 8 months ago →
🇯🇵Japan ptmkenny
Tests pass on 8.1 and the code looks good to me.
Comment 8 months ago →
System Message

MegaChriz → committed 4ce8361a on 8.x-1.x
Issue #3364313 by hlopez, ptmkenny: Fixed a few PHP 8.1 deprecations.
Status changed to Fixed 8 months ago10:29am 20 April 2024
Comment 8 months ago →
🇳🇱Netherlands megachriz
Committed #2. Thanks!
Comment 8 months ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

Problem/Motivation

Proposed resolution

Comments & Activities