Fixed controller creator to use controller.model.stub

readme.md

@@ -1,6 +1,6 @@
 # Laravel Simple CRUD Generator
 
-This is a package to create simple CRUD on your Laravel 5.4 application.
+An artisan `make:crud` command to create a simple CRUD feature on your Laravel 5.4 application.
 
 > **Development in progress**
 

src/CrudMake.php

@@ -44,7 +44,13 @@ class CrudMake extends Command
 
         $this->callSilent('make:model', ['name' => $model]);
         $this->info($model.' model generated.');
-        $this->callSilent('make:controller', ['name' => $pluralModel.'Controller']);
+
+        if (! $this->files->isDirectory(app_path('Http/Controllers'))) {
+            $this->files->makeDirectory(app_path('Http/Controllers'), 0777, true, true);
+        }
+
+        $controllerPath = app_path('Http/Controllers/'.$pluralModel.'Controller.php');
+        $this->files->put($controllerPath, $this->files->get(__DIR__.'/stubs/controller.model.stub'));
         $this->info($pluralModel.'Controller generated.');
 
         $migrationFilePath = database_path('migrations/'.date('Y_m_d_His').'_create_'.$lowerCasePluralModel.'_table.php');

src/stubs/controller.model.stub

@@ -0,0 +1,65 @@
+<?php
+
+namespace DummyNamespace;
+
+use DummyFullModelClass;
+use Illuminate\Http\Request;
+use DummyRootNamespaceHttp\Controllers\Controller;
+
+class DummyClass extends Controller
+{
+    /**
+     * Display a listing of the resource.
+     *
+     * @return \Illuminate\Http\Response
+     */
+    public function index()
+    {
+        //
+    }
+
+    /**
+     * Store a newly created resource in storage.
+     *
+     * @param  \Illuminate\Http\Request  $request
+     * @return \Illuminate\Http\Response
+     */
+    public function store(Request $request)
+    {
+        //
+    }
+
+    /**
+     * Display the specified resource.
+     *
+     * @param  \DummyFullModelClass  $DummyModelVariable
+     * @return \Illuminate\Http\Response
+     */
+    public function show(DummyModelClass $DummyModelVariable)
+    {
+        //
+    }
+
+    /**
+     * Update the specified resource in storage.
+     *
+     * @param  \Illuminate\Http\Request  $request
+     * @param  \DummyFullModelClass  $DummyModelVariable
+     * @return \Illuminate\Http\Response
+     */
+    public function update(Request $request, DummyModelClass $DummyModelVariable)
+    {
+        //
+    }
+
+    /**
+     * Remove the specified resource from storage.
+     *
+     * @param  \DummyFullModelClass  $DummyModelVariable
+     * @return \Illuminate\Http\Response
+     */
+    public function destroy(DummyModelClass $DummyModelVariable)
+    {
+        //
+    }
+}

tests/CrudMakeCommandTest.php

@@ -58,4 +58,85 @@ class Item extends Model
         exec('rm -r '.base_path('tests/Feature'));
         exec('rm -r '.base_path('tests/Unit'));
     }
+
+    /** @test */
+    public function it_creates_correct_controller_class_content()
+    {
+        $this->artisan('make:crud', ['name' => 'Item', '--no-interaction' => true]);
+
+        $this->assertFileExists(app_path('Item.php'));
+        $ctrlClassContent = "<?php
+
+namespace DummyNamespace;
+
+use DummyFullModelClass;
+use Illuminate\Http\Request;
+use DummyRootNamespaceHttp\Controllers\Controller;
+
+class DummyClass extends Controller
+{
+    /**
+     * Display a listing of the resource.
+     *
+     * @return \Illuminate\Http\Response
+     */
+    public function index()
+    {
+        //
+    }
+
+    /**
+     * Store a newly created resource in storage.
+     *
+     * @param  \Illuminate\Http\Request  \$request
+     * @return \Illuminate\Http\Response
+     */
+    public function store(Request \$request)
+    {
+        //
+    }
+
+    /**
+     * Display the specified resource.
+     *
+     * @param  \DummyFullModelClass  \$DummyModelVariable
+     * @return \Illuminate\Http\Response
+     */
+    public function show(DummyModelClass \$DummyModelVariable)
+    {
+        //
+    }
+
+    /**
+     * Update the specified resource in storage.
+     *
+     * @param  \Illuminate\Http\Request  \$request
+     * @param  \DummyFullModelClass  \$DummyModelVariable
+     * @return \Illuminate\Http\Response
+     */
+    public function update(Request \$request, DummyModelClass \$DummyModelVariable)
+    {
+        //
+    }
+
+    /**
+     * Remove the specified resource from storage.
+     *
+     * @param  \DummyFullModelClass  \$DummyModelVariable
+     * @return \Illuminate\Http\Response
+     */
+    public function destroy(DummyModelClass \$DummyModelVariable)
+    {
+        //
+    }
+}
+";
+        $this->assertEquals($ctrlClassContent, file_get_contents(app_path('Http/Controllers/ItemsController.php')));
+        exec('rm '.app_path('Item.php'));
+        exec('rm -r '.app_path('Http'));
+        exec('rm '.database_path('migrations/*'));
+        exec('rm -r '.resource_path('views/items'));
+        exec('rm -r '.base_path('tests/Feature'));
+        exec('rm -r '.base_path('tests/Unit'));
+    }
 }