14

我正在使用 Laravel 构建一个 API,并希望使用 Laravel 通知系统发送推送通知。我有一个匹配模型(基本上是一个帖子),另一个用户可以喜欢这个匹配。当比赛被点赞时,帖子的创建者将收到推送通知。就像 Instagram、Facebook 等一样。

通常推送通知不会发送给用户。我安装了 Laravel Horizo​​n 看看是否有错误。有时通知已发送,有时未发送。使用完全相同的数据:

Laravel 地平线列表

通知有时会因完全相同的数据(相同的用户,相同的匹配)而失败。

错误如下:

Illuminate\Database\Eloquent\ModelNotFoundException:在 /home/forge/owowgolf.com/vendor/laravel/framework/src/Illuminate/Database/Eloquent/Builder.php:312 中没有模型 [App\Models\Match] 118 的查询结果

我确定匹配并且用户存在于数据库中,我在发送通知之前已经验证了这一点。有人知道出了什么问题吗?我在网上能找到的一切都是人们在将通知发送到队列之前没有保存他们的模型。但是,如果模型不存在,甚至不会到达代码将通知发送到队列的那一行。由于路由/控制器中的隐式绑定。

控制器方法:

/**
 * Like a match.
 *
 * @param  \App\Models\Match  $match
 * @return \Illuminate\Http\JsonResponse
 */
public function show(Match $match)
{
    $match->like();

    $players = $match->players()->where('user_id', '!=', currentUser()->id)->get();

    foreach ($players as $user) {
        $user->notify(new NewLikeOnPost($match, currentUser()));
    }

    return ok();
}

通知:

<?php

namespace App\Notifications;

use App\Models\Match;
use App\Models\User;
use Illuminate\Bus\Queueable;
use NotificationChannels\Apn\ApnChannel;
use NotificationChannels\Apn\ApnMessage;
use Illuminate\Notifications\Notification;
use Illuminate\Contracts\Queue\ShouldQueue;

class NewLikeOnPost extends Notification implements ShouldQueue
{
    use Queueable;

    /**
     * The match instance.
     *
     * @var \App\Models\Match
     */
    private $match;

    /**
     * The user instance.
     *
     * @var \App\Models\User
     */
    private $user;

    /**
     * Create a new notification instance.
     *
     * @param  \App\Models\Match  $match
     * @param  \App\Models\User  $user
     */
    public function __construct(Match $match, User $user)
    {
        $this->user = $user;
        $this->match = $match;

        $this->onQueue('high');
    }

    /**
     * Get the notification's delivery channels.
     *
     * @param  \App\Models\User  $notifiable
     * @return array
     */
    public function via($notifiable)
    {
        if ($notifiable->wantsPushNotification($this)) {
            return ['database', ApnChannel::class];
        }

        return ['database'];
    }

    /**
     * Get the mail representation of the notification.
     *
     * @param  \App\Models\User  $notifiable
     * @return \NotificationChannels\Apn\ApnMessage
     */
    public function toApn($notifiable)
    {
        return ApnMessage::create()
            ->badge($notifiable->unreadNotifications()->count())
            ->sound('success')
            ->body($this->user->username . ' flagged your match.');
    }

    /**
     * Get the array representation of the notification.
     *
     * @param  mixed  $notifiable
     * @return array
     */
    public function toArray($notifiable)
    {
        return [
            'user_id' => $this->user->id,
            'body' => "<flag>Flagged</flag> your match.",
            'link' => route('matches.show', $this->match),
            'match_id' => $this->match->id,
        ];
    }

    /**
     * Get the match attribute.
     *
     * @return \App\Models\Match
     */
    public function getMatch()
    {
        return $this->match;
    }
}
4

3 回答 3

1

这不是一个完整的解决方案,但它会降低您将来遇到此错误的机会。

而不是将整个Match模型传递到作业中,只传递id模型的。然后,您可以在构造函数中获取该模型。

/**
 * Like a match.
 *
 * @param  \App\Models\Match  $match
 * @return \Illuminate\Http\JsonResponse
 */
public function show(Match $match)
{
    $match->like();

    $players = $match->players()->where('user_id', '!=', currentUser()->id)->get();

    foreach ($players as $user) {
        $user->notify(new NewLikeOnPost($match->id, currentUser()->id));
    }

    return ok();
}

通知:

class NewLikeOnPost extends Notification implements ShouldQueue
{
    use Queueable;

    private const QUEUE_NAME = 'high';

    /**
     * The match instance.
     *
     * @var \App\Models\Match
     */
    private $match;

    /**
     * The user instance.
     *
     * @var \App\Models\User
     */
    private $user;

    /**
     * Create a new notification instance.
     *
     * @param  int $match
     * @param  int $user
     */
    public function __construct(int $matchId, int $userId)
    {
        $this->user = User::query()->where('id', $userId)->firstOrFail();
        $this->match = Match::query()->where('id', $matchId)->firstOrFail();

        $this->onQueue(self::QUEUE_NAME);
    }

    // Rest of the class is still the same...
}

您可以使用该SerializesModelstrait,但当您为排队的作业添加延迟时,它就无法正常工作。这是因为它会尝试重新加载模型__wakeup(),有时它找不到类。

希望这会有所帮助:)

于 2019-03-24T16:13:51.770 回答
0

可能是因为 $user 不是 User 模型的对象,它是 Match 模型的对象。您需要执行 User::findorfail 或 User::firstOrFail 然后通知用户。

public function show(Match $match)
{
$match->like();

$players = $match->players()->where('user_id', '!=', currentUser()->id)->get();

foreach ($players as $user) {
    $someUser = User::findOrFail($user->user_id);
    $someUser->notify(new NewLikeOnPost($match, currentUser()));
}

return ok();

}

除非在 Match 模型中使用了 notify 特征。或者您可以使用预先加载,这将花费更少的查询!

于 2019-03-10T12:20:07.490 回答
-2

检查您的 .env 以确保您确实使用了 REDIS

BROADCAST_DRIVER=redis
CACHE_DRIVER=redis
SESSION_DRIVER=redis
SESSION_LIFETIME=120
QUEUE_DRIVER=redis

然后清除缓存(php artisan cache:clear , php artisan view:clear),这应该可以解决问题

编辑

我有类似的问题,但现在我只使用 Docker,在我必须检查缓存的配置文件、错误的文件/文件夹权限等之前(REDIS 仅用于广播,其他是标准的)。我开始只使用 redis - 这对我来说更容易、更快、更易于调试!与 Docker 一起使用确实有助于不使用混乱的 nginx/apache/php/redis/ ...

于 2018-02-22T07:41:21.290 回答