• Resolved charlesdupoiron

    (@charlesdupoiron)


    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://www.ads-software.com/support/topic/fatal-error-after-on-7-3-8-and-7-3-9-with-roots-sage/

    Kind regards,
    Charles

    Say hello to Florian ??

Viewing 1 replies (of 1 total)
  • Plugin Contributor Thomas Deneulin

    (@gmulti)

    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’t 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

Viewing 1 replies (of 1 total)
  • The topic ‘Fatal Error with Roots/Acorn 3’ is closed to new replies.