`BindingResolutionException` after upgrading to `12.0.4`

[hschimpf]

Versions:

• laravel-modules Version: 12.0.4
• Laravel Version: 12.19.3
• PHP Version: 8.4.8

Description:

Seems like this commit caused it. Reverting that commit manually solves the issue.

   Illuminate\Contracts\Container\BindingResolutionException 

  Target class [cache] does not exist.

  at vendor/laravel/framework/src/Illuminate/Container/Container.php:1019
    1015▕ 
    1016▕         try {
    1017▕             $reflector = new ReflectionClass($concrete);
    1018▕         } catch (ReflectionException $e) {
  ➜ 1019▕             throw new BindingResolutionException("Target class [$concrete] does not exist.", 0, $e);
    1020▕         }
    1021▕ 
    1022▕         // If the type is not instantiable, the developer is attempting to resolve
    1023▕         // an abstract type such as an Interface or Abstract Class and there is

Steps To Reproduce:

• Upgrade from v12.0.3 to v12.0.4
• composer up nwidart/laravel-modules

Temporal fix:

• Rolled back to v12.0.3
• composer require "nwidart/laravel-modules:^12.0 <12.0.4"

[tareq-alqadi]

Hi @hschimpf
Can you please provide a reproduction repository that shows the issue?

[Congminh0707]

Same issue here after updating laravel-modules.
Is there a known compatibility issue with newer versions of laravel-modules? What version should we use for compatibility?
Thanks!

[RealMrHex]

Same issue here - it looks like something is broken on 12.0.4

[RealMrHex]

Resolved by downgrading to 12.0.3

[bbredewold]

Looks like it cannot resolve 'cache' from the container:

vendor/nwidart/laravel-modules/src/Module.php

    /**
     * The constructor.
     */
    public function __construct(Container $app, string $name, string $path)
    {
        $this->name = $name;
        $this->path = $path;
        $this->cache = $app['cache'];
        $this->files = $app['files'];
        $this->translator = $app['translator'];
        $this->activator = $app[ActivatorInterface::class];
        $this->app = $app;
    }

[peterpeep]

+1

[jnbn]

any updates on this?