WPIntell

Source evidence

Fatal Error with Roots/Acorn 3

WP Umbrella: Update Backup Restore & Monitoring 路 support 路 2023-11-01T17:27:00+00:00

mixedsentiment
highseverity
0.83relevance
1replies
Evidence linked to opportunitycommercial context

Proof Health

Open evidence

Commercial opportunities need traceable source links before they are treated as build-worthy.

2 / 31 rows with source links

6.5% of this page's analysis has direct source links.

0 build-decision rows missing links

0 rows here require auditable proof before promotion.

29 rows with no attached evidence

0 rows have source counts but still need direct links.

Conversation

support
charlesdupoiron resolved
Hi, We get following error in v2.11.6 of the plugin. Our starter theme (Sage 10) also depends on Illuminate\Container (through roots/acorn). Fatal error: Declaration of Illuminate\Container\Container::get(string $id) must be compatible with Psr\Container\ContainerInterface::get($id) in /home/charles/dev/project/app/public/wp-content/themes/test/vendor/illuminate/container/Container.php on line 714 I think plugin requires a old version of psr/container. A patch to fix it : Index: vendor/psr/container/src/ContainerInterface.php IDEA additional info: Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP <+>UTF-8 =================================================================== diff --git a/vendor/psr/container/src/ContainerInterface.php b/vendor/psr/container/src/ContainerInterface.php --- a/vendor/psr/container/src/ContainerInterface.php +++ b/vendor/psr/container/src/ContainerInterface.php (date 1698857506884) @@ -20,7 +20,7 @@ * * @return mixed Entry. */ - public function get($id); + public function get(string $id); /** * Returns true if the container can return an entry for the given identifier. @@ -33,5 +33,5 @@ * * @return bool */ - public function has($id); + public function has(string $id): bool; } Can you fix the compatibility problems using a namespace? Similary issue : https://wordpress.org/support/topic/fatal-error-after-on-7-3-8-and-7-3-9-with-roots-sage/ Kind regards, Charles Say hello to Florian 馃檪 This topic was modified 2 years, 6 months ago by charlesdupoiron . Hi, Thank you very much for this information. This is because we both load the psr/container module and they load before we do. As they don鈥檛 have the same versions, they have priority. We plan to work on this by the end of the month and prefix our vendors to avoid this kind of problem. Have a nice day, Thomas

Comments

1 shown
Thomas Deneulin 2023-11-07T23:29:00+00:00

Hi, Thank you very much for this information. This is because we both load the psr/container module and they load before we do. As they don鈥檛 have the same versions, they have priority. We plan to work on this by the end of the month and prefix our vendors to avoid this kind of problem. Have a nice day, Thomas