Skip to content

Conversation

@ChristophWurst
Copy link

PHP is adding more types to its core classes/interface and the new
attribute is necessary to prevent a deprecation notice in the logs.

Return type of Horde_Mail_Rfc822_List::rewind() should either be compatible with Iterator::rewind(): void, or the #[\\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice at Horde_Mail/Horde/Mail/Rfc822/List.php#440

PHP is adding more types to its core classes/interface and the new
attribute is necessary to prevent a deprecation notice in the logs.

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
@ChristophWurst
Copy link
Author

@bytestream I think you're not subscribed to this repo so I'm kindly asking you to approve the CI run and review my fixes :)

@bytestream bytestream merged commit b7a672a into bytestream:master Apr 28, 2022
@ChristophWurst ChristophWurst deleted the fix/php81-return-type-will-change branch April 28, 2022 10:55
@bytestream
Copy link
Owner

@ChristophWurst yeah please @ me. I used a script to clone all 150+ horde repos and I guess it didn't subscribe me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants