Generates and handles Modules for Laravel

Hi, thank you for providing this awesome package. Our team have been using this for years now. As good as it is, it can be troublesome to generate controllers,models, etc. using Laravel's scaffolding commands. It would be nice if we could just run php artisan make:controller SomeController --module=SomeModule and the generated controller will be inside the assigned module.

I have forked this repo and tried implementing this, you can see the branch here. I've only applied what I think is sensible to be placed in their respective module (e,g, events, jobs, listeners, etc.). I can exclude or include others if you so desire, just let me know.

I've done some light testing on it but I'd like to know how you properly test this so I could too before I make a PR for this.

When using this module I noticed that route caching don't work on the routes.php files in the different modules.

I would suggest this solution in the ModuelServiceProvider file:

// Allow routes to be cached
if(is_dir(app_path().'/Modules/')) {
$modules = config("modules.enable") ?: array_map('class_basename', $this->files->directories(app_path().'/Modules/'));
foreach($modules as $module)  {
// Allow routes to be cached
if (!$this->app->routesAreCached()) {
    $routes = app_path() . '/Modules/' . $module . '/routes.php';
    if($this->files->exists($routes)) include $routes;
}
$helper = app_path().'/Modules/'.$module.'/helper.php';
$views  = app_path().'/Modules/'.$module.'/Views';
$trans  = app_path().'/Modules/'.$module.'/Translations';

if($this->files->exists($helper)) include $helper;
if($this->files->isDirectory($views)) $this->loadViewsFrom($views, $module);
if($this->files->isDirectory($trans)) $this->loadTranslationsFrom($trans, $module);
}
}

I write on model and contlloer it is now wokring

        $filght = Blogs::all();
    print_r($filght);

"Model"

Hello, first of all thank you for your works!

I'm using your module, but i think to have find a problem. Default routes of module (web.php) is:

Route::group(array('module' => 'Test', 'middleware' => ['web'], 'namespace' => 'App\Modules\Test\Controllers'), function() {
 //list of my routes, for example 
 Route::get('/test', 'TestController@testIndex');
});

If i would like to protect my routes by middleware, i should change Route::group like that:

Route::group(array('module' => 'Test', 'middleware' => ['**auth**'], 'namespace' => 'App\Modules\Test\Controllers'), function() {
 //list of my routes, for example
 Route::get('/test', 'TestController@testIndex');
});

If i change Route::group like example above when i try to go to "/test", i will be redirect to "/". My suspicion is that auth middleware are not loaded in right way inside module, maybe for different namespace?

The only way i finded to workaround, is to edit routes.php of project (not of module!) adding inside Route::group(['middleware' => ['auth']], function() { [...] Route::get('/test', '\App\Modules\Test\Controllers\TestController@testIndex');[...]

This is no a really useful way to solve the problem. What i can do to solve that problem in web.php of module, istead of edit main Route file?

Best Regards, Simone

artisan route:cache command fails due to more than one inclusion of the helper.php files.

I have fixed the issue with updating "include $helper;" to "include_once $helper;" in boot() method of ModuleServiceProvider class.

I have been using this package for about an year now and am very pleased with how easily it manages modules in a project.

But, I am still using my project with Laravel 5.1.1 and recently noticed (when doing some composer updates) that it was throwing the following error. http://imgur.com/iAUlGSC

So, is there anyway I can get over this issue or it no longer supports Laravel less than v5.1.1?

Any help is much appreciated.

Thanks in advance.

Hello.

I've updated L5 Modular in my project from V1 to V2.

I've set the config file to load the V1 modules, but now I'm getting this message:

Target class [App\Modules\MyModule\Controllers\App\Modules\MyModule\Controllers\MyController] does not exist.

I think that this is happening because some of the project developers have used the action function with the full path to the controller method [Ex: action('\App\Modules\MyModule\Controllers\MyController@index');] and L5 Modular makes the concatenation beetween the module path and with the action parameter.

Can someone help me with this?

Thank's!

Is your feature request related to a problem? Please describe. many of the time a module need a configuration that can be published to the main project this feature in so important to the modules structures and make it easy to reuse the module in other projects with different configurations

Describe the solution you'd like It would be awesome if I can add a config file for my module for example: app/Modules/FooBar/config/config.php

Describe alternatives you've considered and if you add an artisan command like (php artisan make:module:config) that would be awesome too

Additional context

Call to undefined function ArtemSchander\L5Modular\Console\studly_case()

hi, I always get this error when making a module, I tried to remove the studly_case and other function indicated on the error, and it works, I hope you'll fix this for laravel 6.*

Hi,

I would like to report this issue when I tried to run the command on generating module error was occurred. Please check below.

https://www.screencast.com/t/lRosinIVIJ6

Note: My php version: 7.1+

Thank you.

Ronard

Hi there,

Could you please help me out with a problem with this package. Actually the problem is that when I try to protect my routes with auth middleware in the web.php file, then the auth middleware doesn't work, but however if I use the middleware in the api.php file then the routes in the web.php is also protected.

It seems that the package is prioritizing api.php file before web.php file. I hope I have made the situation clear.

Thanks

How to use namespace Admin/Module_name and User/Module_name ?

folder routes not auto use ... :-(

Admin/routes/web.php Work (Module auto use only this path)

Admin/Test/routes/web.php Not Work

Describe the bug Installed this package on clean laravel 8.12 and jsut make a new module call Faq.

After this I just go to https://l5modular.com/usage/#facade

and using first one

L5Modular::exists('HelloWorld');

and it give me following error

Error Class 'L5Modular' not found http://localhost:8000/test/

To Reproduce Steps to reproduce the behavior:

1. Install new laravel and this package.
2. Make any module after setup the package.
3. Make a test route and use any facade from docs or the exists() one.
4. You will get class not found error.

Expected behavior It should return true or false as per docs.

Screenshots

Environment (please complete the following information):

• OS & version: Windows 10 Build 19042
• Laravel version: 8.12
• L5Modular version: 2.1

Hello, guys.

It would be great if we could have Livewire support in this package, so we could create Livewire components and views inside each module.

Is it possible?

Thank you.

Hey guys

How can I generate migrations in the standard way like in the previous versions ? I've played around with config files but no way

UPDATE :

I had to edit this method here in MigrateMakeCommand Class

 protected function getMigrationPath()
    {
       
        $migrationPath = $this->laravel['path'] .'/' . $this->getConfiguredFolder('migrations');
   
        if (! is_null($targetPath = $this->input->getOption('path'))) {
            return $migrationPath . '/' . $targetPath;
        }

        return $migrationPath;
    }

is there a cleaner way to make this change ?

I liked the arrilot/laravel-widgets functionality. Roughly add widgets through the module:

class ModuleServiceProvider

use Illuminate\Support\Facades\View;

function boot()

$widgets  = app_path().'/Modules/'.$module.'/Widgets';

if ($this->files->isDirectory($widgets)){
	$directories = $this->files->directories($widgets);
	foreach($directories as $Widjet){ 
		$WidjetViews = $Widjet."/Views";
		if ($this->files->isDirectory($WidjetViews)) {
			View::addNamespace('Module.'.$module.'.Widget', $WidjetViews);
		}
		$controllerList = $this->files->files($Widjet);
		foreach($controllerList as $controller){
			include_once($controller->getPathname());
		}
	}
} 

laravel-project/
    app/
    └── Modules/
        └── FooBar/
            └── Widgets/
                └── Widget/
                     └── Views/
                          └── bar.blade.php
                     Bar.php

view('Module.FooBar.Widget::bar')