Fix duplicate and missing return type declarations (#433)

Author: ghostwriter

src/Generators/ControllerGenerator.php

@@ -164,7 +164,7 @@ protected function buildMethods(Controller $controller)
             }
 
             if (Blueprint::supportsReturnTypeHits()) {
-                $method = str_replace('request)', 'request): \Illuminate\Http\Response', $method);
+                $method = str_replace(')'.PHP_EOL, '): \Illuminate\Http\Response'.PHP_EOL, $method);
             }
 
             $methods .= PHP_EOL.$method;

src/Generators/ModelGenerator.php

@@ -184,8 +184,8 @@ protected function buildRelationships(Model $model)
         }
 
         foreach ($model->relationships() as $type => $references) {
-            $custom_template = $template;
             foreach ($references as $reference) {
+                $custom_template = $template;
                 $key = null;
                 $class = null;
 
@@ -230,6 +230,7 @@ protected function buildRelationships(Model $model)
                 } elseif (in_array($type, ['hasMany', 'belongsToMany', 'morphMany'])) {
                     $method_name = Str::plural($column_name);
                 }
+
                 if (Blueprint::supportsReturnTypeHits()) {
                     $custom_template = str_replace(
                         '{{ method }}()',

tests/Feature/Generators/ControllerGeneratorTest.php

@@ -189,12 +189,12 @@ public function output_using_return_types()
             ->with('app/Http/Controllers', 0755, true);
 
         $this->files->expects('put')
-            ->with('app/Http/Controllers/PostController.php', $this->fixture('controllers/return-type-declarations.php'));
+            ->with('app/Http/Controllers/TermController.php', $this->fixture('controllers/return-type-declarations.php'));
 
-        $tokens = $this->blueprint->parse($this->fixture('drafts/readme-example.yaml'));
+        $tokens = $this->blueprint->parse($this->fixture('drafts/return-type-declarations.yaml'));
         $tree = $this->blueprint->analyze($tokens);
 
-        $this->assertEquals(['created' => ['app/Http/Controllers/PostController.php']], $this->subject->output($tree));
+        $this->assertEquals(['created' => ['app/Http/Controllers/TermController.php']], $this->subject->output($tree));
     }
 
     public function controllerTreeDataProvider()

tests/Feature/Generators/ModelGeneratorTest.php

@@ -692,7 +692,7 @@ public function laravel8ModelTreeDataProvider()
             ['drafts/resource-statements.yaml', 'app/User.php', 'models/resource-statements-laravel8.php'],
             ['drafts/all-column-types.yaml', 'app/AllType.php', 'models/all-column-types-laravel8.php'],
             ['drafts/alias-relationships.yaml', 'app/Salesman.php', 'models/alias-relationships-laravel8.php'],
-            ['drafts/alias-relationships.yaml', 'app/Salesman.php', 'models/return-type-declarations.php'],
+            ['drafts/return-type-declarations.yaml', 'app/Term.php', 'models/return-type-declarations.php'],
             ['drafts/uuid-shorthand-invalid-relationship.yaml', 'app/AgeCohort.php', 'models/uuid-shorthand-invalid-relationship-laravel8.php'],
         ];
     }

tests/fixtures/controllers/return-type-declarations.php

@@ -2,43 +2,89 @@
 
 namespace App\Http\Controllers;
 
-use App\Events\NewPost;
-use App\Http\Requests\PostStoreRequest;
-use App\Jobs\SyncMedia;
-use App\Mail\ReviewPost;
-use App\Post;
+use App\Http\Requests\TermStoreRequest;
+use App\Http\Requests\TermUpdateRequest;
+use App\Term;
 use Illuminate\Http\Request;
-use Illuminate\Support\Facades\Mail;
 
-class PostController extends Controller
+class TermController extends Controller
 {
     /**
      * @param \Illuminate\Http\Request $request
      * @return \Illuminate\Http\Response
      */
     public function index(Request $request): \Illuminate\Http\Response
     {
-        $posts = Post::all();
+        $terms = Term::all();
 
-        return view('post.index', compact('posts'));
+        return view('term.index', compact('terms'));
     }
 
     /**
-     * @param \App\Http\Requests\PostStoreRequest $request
+     * @param \Illuminate\Http\Request $request
+     * @return \Illuminate\Http\Response
+     */
+    public function create(Request $request): \Illuminate\Http\Response
+    {
+        return view('term.create');
+    }
+
+    /**
+     * @param \App\Http\Requests\TermStoreRequest $request
+     * @return \Illuminate\Http\Response
+     */
+    public function store(TermStoreRequest $request): \Illuminate\Http\Response
+    {
+        $term = Term::create($request->validated());
+
+        $request->session()->flash('term.id', $term->id);
+
+        return redirect()->route('term.index');
+    }
+
+    /**
+     * @param \Illuminate\Http\Request $request
+     * @param \App\Term $term
+     * @return \Illuminate\Http\Response
+     */
+    public function show(Request $request, Term $term): \Illuminate\Http\Response
+    {
+        return view('term.show', compact('term'));
+    }
+
+    /**
+     * @param \Illuminate\Http\Request $request
+     * @param \App\Term $term
      * @return \Illuminate\Http\Response
      */
-    public function store(PostStoreRequest $request): \Illuminate\Http\Response
+    public function edit(Request $request, Term $term): \Illuminate\Http\Response
     {
-        $post = Post::create($request->validated());
+        return view('term.edit', compact('term'));
+    }
 
-        Mail::to($post->author->email)->send(new ReviewPost($post));
+    /**
+     * @param \App\Http\Requests\TermUpdateRequest $request
+     * @param \App\Term $term
+     * @return \Illuminate\Http\Response
+     */
+    public function update(TermUpdateRequest $request, Term $term): \Illuminate\Http\Response
+    {
+        $term->update($request->validated());
 
-        SyncMedia::dispatch($post);
+        $request->session()->flash('term.id', $term->id);
 
-        event(new NewPost($post));
+        return redirect()->route('term.index');
+    }
 
-        $request->session()->flash('post.title', $post->title);
+    /**
+     * @param \Illuminate\Http\Request $request
+     * @param \App\Term $term
+     * @return \Illuminate\Http\Response
+     */
+    public function destroy(Request $request, Term $term): \Illuminate\Http\Response
+    {
+        $term->delete();
 
-        return redirect()->route('post.index');
+        return redirect()->route('term.index');
     }
 }

tests/fixtures/drafts/return-type-declarations.yaml

@@ -0,0 +1,17 @@
+models:
+  Term:
+    uuid: string unique
+    name: json
+    description: json nullable
+    meta: json nullable
+    user_id: id nullable foreign onDelete:null onUpdate:cascade
+    team_id: id foreign onDelete:cascade onUpdate:cascade
+    published: boolean default:1
+    type: string nullable default:booking
+    softDeletes
+    relationships:
+      belongsToMany: Organizer, Event
+
+controllers:
+  Term:
+    resource

tests/fixtures/models/return-type-declarations.php

@@ -4,18 +4,26 @@
 
 use Illuminate\Database\Eloquent\Factories\HasFactory;
 use Illuminate\Database\Eloquent\Model;
+use Illuminate\Database\Eloquent\SoftDeletes;
 
-class Salesman extends Model
+class Term extends Model
 {
-    use HasFactory;
+    use HasFactory, SoftDeletes;
 
     /**
      * The attributes that are mass assignable.
      *
      * @var array
      */
     protected $fillable = [
+        'uuid',
         'name',
+        'description',
+        'meta',
+        'user_id',
+        'team_id',
+        'published',
+        'type',
     ];
 
     /**
@@ -25,21 +33,32 @@ class Salesman extends Model
      */
     protected $casts = [
         'id' => 'integer',
+        'name' => 'array',
+        'description' => 'array',
+        'meta' => 'array',
+        'user_id' => 'integer',
+        'team_id' => 'integer',
+        'published' => 'boolean',
     ];
 
 
-    public function lead(): \Illuminate\Database\Eloquent\Relations\HasOne
+    public function organizers(): \Illuminate\Database\Eloquent\Relations\BelongsToMany
     {
-        return $this->hasOne(\App\User::class);
+        return $this->belongsToMany(\App\Organizer::class);
     }
 
-    public function methodNames(): \Illuminate\Database\Eloquent\Relations\HasMany
+    public function events(): \Illuminate\Database\Eloquent\Relations\BelongsToMany
     {
-        return $this->hasMany(\App\ClassName::class);
+        return $this->belongsToMany(\App\Event::class);
     }
 
-    public function methodName(): \Illuminate\Database\Eloquent\Relations\BelongsTo
+    public function user(): \Illuminate\Database\Eloquent\Relations\BelongsTo
     {
-        return $this->belongsTo(\App\ClassName::class);
+        return $this->belongsTo(\App\User::class);
+    }
+
+    public function team(): \Illuminate\Database\Eloquent\Relations\BelongsTo
+    {
+        return $this->belongsTo(\App\Team::class);
     }
 }