web-dev-qa-db-ja.com

Laravel：通知のカスタマイズまたは拡張-データベースモデル

IMHO、Laravelで通知を保存するための現在のデータベースチャネルは、本当に悪い設計です。

  • たとえば、削除されたアイテムの通知をクリーンアップするために、アイテムに外部キーカスケードを使用することはできません。
  • （配列にキャストされた）data列のカスタム属性の検索は最適ではありません

ベンダーパッケージでDatabaseNotificationモデルを拡張するにはどうしますか？

列を追加したいevent_id、question_id、user_id（通知を作成したユーザー）など...デフォルトlaravel notifications table

より多くの列を含めるためにsend関数をどのようにオーバーライドしますか？

に：

vendor/laravel/framework/src/Illuminate/Notifications/Channels/DatabaseChannel.php

コード：

class DatabaseChannel
{
 /**
  * Send the given notification.
  *
  * @param  mixed  $notifiable
  * @param  \Illuminate\Notifications\Notification  $notification
  * @return \Illuminate\Database\Eloquent\Model
  */
 public function send($notifiable, Notification $notification)
 {
    return $notifiable->routeNotificationFor('database')->create([
        'id' => $notification->id,
        'type' => get_class($notification),

      \\I want to add these
        'user_id' => \Auth::user()->id,
        'event_id' => $notification->type =='event' ? $notification->id : null, 
        'question_id' => $notification->type =='question' ? $notification->id : null,
      \\End adding new columns

        'data' => $this->getData($notifiable, $notification),
        'read_at' => null,
    ]);
 }
}
laravellaravel-5laravel-6
10
BassMHL

カスタム通知チャネルを作成するには：

まず、たとえばApp\Notificationsにクラスを作成します。

<?php

namespace App\Notifications;

use Illuminate\Notifications\Notification;

class CustomDbChannel 
{

  public function send($notifiable, Notification $notification)
  {
    $data = $notification->toDatabase($notifiable);

    return $notifiable->routeNotificationFor('database')->create([
        'id' => $notification->id,

        //customize here
        'answer_id' => $data['answer_id'], //<-- comes from toDatabase() Method below
        'user_id'=> \Auth::user()->id,

        'type' => get_class($notification),
        'data' => $data,
        'read_at' => null,
    ]);
  }

}

次に、Notificationクラスのviaメソッドでこのチャネルを使用します。

<?php

namespace App\Notifications;

use Illuminate\Notifications\Notification;

use App\Notifications\CustomDbChannel;

class NewAnswerPosted extends Notification
{
  private $answer;

  public function __construct($answer)
  {
    $this->answer = $answer;
  }

  public function via($notifiable)
  {
    return [CustomDbChannel::class]; //<-- important custom Channel defined here
  }

  public function toDatabase($notifiable)
  {
    return [
      'type' => 'some data',
      'title' => 'other data',
      'url' => 'other data',
      'answer_id' => $this->answer->id //<-- send the id here
    ];
  }
}
25
BassMHL

独自のNotificationモデルとNotifiableトレイトを作成して使用し、（ユーザー）モデルで独自のNotifiableトレイトを使用します。

App\Notifiable.php：

namespace App;

use Illuminate\Notifications\Notifiable as BaseNotifiable;

trait Notifiable
{
    use BaseNotifiable;

    /**
     * Get the entity's notifications.
     */
    public function notifications()
    {
        return $this->morphMany(Notification::class, 'notifiable')
                            ->orderBy('created_at', 'desc');
    }
}

App\Notification.php：

namespace App;

use Illuminate\Notifications\DatabaseNotification;

class Notification extends DatabaseNotification
{
    // ...
}

App\User.php：

namespace App;

use Illuminate\Foundation\Auth\User as Authenticatable;

class User extends Authenticatable
{
    use Notifiable;

    // ...
}
5
Kamal Khan

"Bassem El Hachem"とは異なり、via()メソッドにdatabaseキーワードを保持したかったのです。

したがって、カスタムDatabaseChannelに加えて、createDatabaseDriver()メソッドで独自のChannelManagerを返す独自のDatabaseChannelも作成しました。

アプリのServiceProvider::register()メソッドで、元のChannelManagerクラスのシングルトンを上書きして、カスタムマネージャーを返しました。

3
cweiske

@ cweiske response の例。

本当に必要な場合は、Illuminate\Notifications\Channels\DatabaseChannel新しいチャンネルを作成しない：

チャネルを拡張します。

<?php

namespace App\Notifications;

use Illuminate\Notifications\Channels\DatabaseChannel as BaseDatabaseChannel;
use Illuminate\Notifications\Notification;

class MyDatabaseChannel extends BaseDatabaseChannel
{
    /**
     * Send the given notification.
     *
     * @param  mixed  $notifiable
     * @param  \Illuminate\Notifications\Notification  $notification
     * @return \Illuminate\Database\Eloquent\Model
     */
    public function send($notifiable, Notification $notification)
    {
        $adminNotificationId = null;
        if (method_exists($notification, 'getAdminNotificationId')) {
            $adminNotificationId = $notification->getAdminNotificationId();
        }

        return $notifiable->routeNotificationFor('database')->create([
            'id' => $notification->id,
            'type' => get_class($notification),
            'data' => $this->getData($notifiable, $notification),

            // ** New custom field **
            'admin_notification_id' => $adminNotificationId,

            'read_at' => null,
        ]);
    }
}

そして、Illuminate\Notifications\Channels\DatabaseChannel再びアプリケーションコンテナで：

app\Providers\AppServiceProvider.php

class AppServiceProvider extends ServiceProvider
{
    /**
     * Bootstrap any application services.
     *
     * @return void
     */
    public function boot()
    {
        //
    }

    /**
     * Register any application services.
     *
     * @return void
     */
    public function register()
    {
        $this->app->bind(
            Illuminate\Notifications\Channels\DatabaseChannel::class,
            App\Notifications\MyDatabaseChannel::class
        );
    }
}

今度はIlluminate\Notifications\ChannelManager try createDatabaseDriverは、登録されているデータベースドライバを返します。

この問題を解決するためのもう1つのオプション！

2
Samuel Martins

私の意見では、laravel通知はまだ準備ができておらず、機能が非常に限られています。

とにかく、私は通知クラスをカスタマイズすることで同様の問題を解決しました：

このアクションのクラスを作成します。

artisan make:notification NewQuestion

その中：

public function __construct($user,$question)
    {
        $this->user=$user;
        $this->question=$question;
    }


...

    public function toDatabase($notifiable){
        $data=[
            'question'=>$this->(array)$this->question->getAttributes(),
            'user'=>$this->(array)$this->user->getAttributes()
        ];

        return $data;
    }

その後、次のようにビューまたはコントローラーで適切なデータにアクセスできます。

@if($notification->type=='App\Notifications\UserRegistered')
<a href="{!!route('question.show',$notification->data['question']['id'])!!}">New question from {{$notification->data['user']['name']}}</a>
@endif
0
Luca C.