Verwenden von Model Events Listener in Laravel 5

Question

Ich möchte sicherstellen, dass ich Model-Events-Listener in Laravel 5 richtig verwendet habe und nichts vermasselt habe (Listener vs Handler?). Meine Lösung funktioniert gut, aber ich frage mich, ob ich nach Konzept und Konvention von Laravel 5.

Ziel entwickelt: immer gesetzt $ themen-> status_id auf einen bestimmten Wert, wenn Modell Speichern wird.

In app \ Providers \ EventServiceProvider.php

<?php namespace App\Providers; 

... 

class EventServiceProvider extends ServiceProvider { 

    ... 

    public function boot(DispatcherContract $events) 
    { 
     parent::boot($events); 

     Issue::saving('App\Handlers\Events\SetIssueStatus'); 
    } 

} 

In app \ Handlers \ Events \ SetIssueStatus.php

<?php namespace App\Handlers\Events; 

... 

class SetIssueStatus { 

    ... 

    public function handle(Issue $issue) 
    { 
     if (something) 
     { 
      $issueStatus = IssueStatus::where(somethingElse)->firstOrFail(); 
     } 
     else 
     { 
      $issueStatus = IssueStatus::where(somethingAnother)->firstOrFail(); 
     } 

     // issue_status() is One-to-One relations with IssueType (belongsTo) 
     $issue->issue_status()->associate($issueStatus); 
    } 

} 

Vielen Dank für Ihre Zeit.

Answer 1

Wie Sie sagten, Sie haben eine funktionierende Version und es ist eine gültige Version, jetzt liegt es an Ihnen, herauszufinden, ob es für Sie in Ordnung ist.

Nur um zu verdeutlichen, sage ich nicht, dass dies bessere Lösungen sind, sie sind nur ein anderer Weg.

Da das, was Sie tun, ist spezifisch für das Emissionsmodell oder zumindest scheint es nicht ein generisches Ereignis zu sein, könnten Sie es auf Ihrem Modell einrichten direkt

<?php namespace App; 

use Illuminate\Database\Eloquent\Model; 
use IssueStatus; 

class Issue extends Model { 


    protected static function boot() 
    { 
     parent::boot(); 

     static::saving(function($issue){ 
      if (something) 
      { 
       $issueStatus = IssueStatus::where(somethingElse)->firstOrFail(); 
      } 
      else 
      { 
       $issueStatus = IssueStatus::where(somethingAnother)->firstOrFail(); 
      } 

      // issue_status() is One-to-One relations with IssueType (belongsTo) 
      $issue->issue_status()->associate($issueStatus); 

     }); 
    } 
} 

aber wenn Ihre Veranstaltung in der Tat eine generische und Sie möchten es über mehrere Modelle hinweg verwenden, könnten Sie das gleiche erreichen. Sie müssen nur den Code aus dem Modell extrahieren und Eigenschaften (wie Sie mit weichen löschen) verwenden

Zuerst erstellen wir unser Merkmal (in diesem Fall haben wir auf der Wurzel unserer App erstellt) und extrahieren Sie den Code, schrieb ich vor, aus dem Modell:

<?php namespace App 

use IssueStatus; 

trait IssueStatusSetter 
{ 
    protected static function boot() 
    { 
     parent::boot(); 

     static::saving(function($model){ 
      if (something) 
      { 
       $issueStatus = IssueStatus::where(somethingElse)->firstOrFail(); 
      } 
      else 
      { 
       $issueStatus = IssueStatus::where(somethingAnother)->firstOrFail(); 
      } 

      // issue_status() is One-to-One relations with IssueType (belongsTo) 
      $model->issue_status()->associate($issueStatus); 

     }); 
    } 
} 

nun auf den Modellen, wo Sie es verwenden möchten, importieren Sie einfach den Zug und erklären, es ist Gebrauch:

<?php namespace App; 

use Illuminate\Database\Eloquent\Model; 
use IssueStatusSetter; 

class Issue extends Model { 

    use IssueStatusSetter; 

} 

Nun ist diese letzte Option, die ich Ihnen gezeigt, es ist ein Sie haben die generische Option, die Sie auf jedes Modell anwenden können, indem Sie es einfach deklarieren auf der Oberseite Ihres Modells.