- Issue created by @Alina Basarabeanu
- ๐ฎ๐ณIndia chetan 11
chetan 11 โ made their first commit to this issueโs fork.
- ๐จ๐ฆCanada danrod Ottawa
I am getting this error when installing the errorlog module:
Fatal error: Declaration of Drupal\errorlog\Logger\ErrorLogMessageFormatter::log($level, $message, array $context = []) must be compatible with Drupal\Core\Logger\RfcLoggerTrait::log($level, Stringable|string $message, array $context = []): void in /var/www/html/web/modules/contrib/logging_alerts/errorlog/src/Logger/ErrorLogMessageFormatter.php on line 46
Attached a patch to fix this.
- Status changed to Needs review
about 1 year ago 6:53am 28 September 2023 - ๐ฎ๐ณIndia Madhu Kumar M E
The above patch #4 applied cleanly is compatible with d10 , it can install the module without any issue, attached screenshot for the same
- ๐ฌ๐งUnited Kingdom Alina Basarabeanu
The error reported by #4 is fixed with the issue 3382684.
The current pull request is fixing D10 compatibility only.
We need those changes in a new stable version to use the module on Drupal 10. - First commit to issue fork.
- Status changed to Needs work
9 months ago 12:15am 28 March 2024 - ๐ธ๐ฐSlovakia poker10
I merged two other issues, so moving this to Needs Work.
We can finish the D10 compatibility issues from what is left here. If possible, we should keep D9 compatibility as well. Thanks!
I plan to publish the release once this issue is fixed.
- Status changed to RTBC
9 months ago 9:31pm 5 April 2024 - ๐ธ๐ฐSlovakia poker10
Seems like that all that is left from the original changes in this MR is to change the
core_version_requirement
and some other minor adjustments. So let's fix this. I have removed the newly added composer.json - we can still add it later, but I do not think we need it now (the file is not needed in certain circumstances: https://www.drupal.org/docs/develop/using-composer/add-a-composerjson-file โ ).I have also changed
core_version_requirement
from ^8 to ^8.8, as per: https://www.drupal.org/node/3070687 โ . I do not see any other D10 incompatibilities in the issue queue now, so going to commit this change. -
poker10 โ
committed 0fe1f13f on 2.0.x authored by
chetan 11 โ
Issue #3390107 by poker10, zoiosilva, Alina Basarabeanu, danrod: Drupal...
-
poker10 โ
committed 0fe1f13f on 2.0.x authored by
chetan 11 โ
- Status changed to Fixed
9 months ago 9:34pm 5 April 2024 Automatically closed - issue fixed for 2 weeks with no activity.