Как узнать, что статья добавлена в закладки?

Question

[Dimon3x]

Таблицы:

posts
users

posts_user - промежуточная - закладки

$table->bigIncrements('id');
$table->integer('user_id');
$table->integer('post_id');

Вывожу все статьи

$allPosts = Posts::with('users')->get();

class Posts extends Model
{ 
    //получить автора статьи
    
    public function users()
    {
        return $this->hasMany('App\User', 'id', 'user_id');
    }

Не знаю, что надо сделать, что бы к запросу ещё добавить ключ, что бы
в цикле потом выводить, что статья уже добавлена, иначе добавить в закладки.

Comments

[Verstak96]

Как вариант, в модельку Post, добавить связь (один к одному) с промежуточной таблицей. Если в ответе прилетел null, то статья не в закладках, а если запись из промежуточной таблицы, то соответственно уже в закладках.

Что-то типа того:

class Post extends Model
{ 
    /**
     * Автор статьи.
     */
    public function author()
    {
        return $this->belongsTo('App\User', 'id', 'user_id');
    }

   /**
    * Ищем в промежуточной таблице, запись о том, что текущий пользователь добавил эту статью в закладки.
    */
    public function bookmark()
    {
        return $this->belongsTo('App\Bookmark', 'post_id')->where('user_id', optional(auth())->id);
    }

    public function getIsBookmarkAttribute()
    {
        return !is_null($this->bookmark);
    }
}

// Контроллер
$allPosts = Posts::with('author', 'bookmark')->get();

// Шаблон
@foreach ($posts as $post)
  @if ($post->is_bookmark)
   В избранном
  @else
    Добавить в избранное
  @endif
@endforeach

[Евгений]

Verstak96, лучший)

Answer 1

[jazzus]

В контроллере

$id = Auth::id();
//К каждой статье добавляем users_count, 
//чтобы узнать какие статьи уже добавлены в избранное авторизованного юзера
// и не делать потом запросы к бд в цикле
$posts = Post::with('users')
           ->withCount(array('users' => function($query) use ($id) {
           $query->where('users.id', $id);
           }))->get();

В шаблоне

{{-- если количество больше нуля, значит статья уже добавлена в избранное --}}
@foreach ($posts as $post)
  @if ($post->users_count>0)
    Уже добавлено в избранное
  @else
    Добавить в избранное
  @endif
@endforeach

Исправить отношение в модели Post

public function users() {
  return $this->belongsToMany('App\User', 'posts_user', 'post_id', 'user_id');
}

Добавлять в избранное через attach. Как это делается

Comments

[Dimon3x]

Спасибо, попробую

[Dimon3x]

Выводит ошибку

Column not found: 1054 Unknown column 
'questions.questions_id' in 'where clause' 
(SQL: select `questions`.*, (select count(*) from `users` where 
`questions`.`questions_id` = `users`.`questions_user` and `users`.`id` = 1) 
as `users_count` from `questions`)

[Dimon3x]

вместо posts, использую questions

[jazzus]

Нужно, чтобы были ключи в промежуточной таблице. Если нет данных в таблице то лучше удалить и создать новую

$table->integer('post_id')->unsigned();
 $table->foreign('post_id')->references('id')->on('posts')->onDelete('cascade');
 $table->integer('user_id')->unsigned();
 $table->foreign('user_id')->references('id')->on('users')->onDelete('cascade');

Затем попробуйте. Если будет ошибка, то пришлите ошибку, код из метода полностью и код отношения из модели без переименований

[Dimon3x]

jazzus, я решил сделать по другому

class Posts extends Model
{
    //получить автора статьи
    public function users()
    {
        return $this->hasMany('App\User', 'id', 'user_id');
    }
//добавлен ли вопрос в закладки
    public function bookmark()
    {
        return $this->belongsToMany('App\User');
    }
}

<div class="bookmark">
                        @if($post->bookmark->count() != 0)
                            @foreach($post->bookmark as $aaa)
 
                                @if(Auth::id() == $aaa->id)
                                <button data-bookmark="{{$post->id}}" class="for-desctop">Удалить из закладок</button>
 
                                @else
                                <button data-bookmark="{{$post->id}}" class="for-desctop">Добавить в закладки</button>
                                @endif
                                @break
                            @endforeach
 
                        @else
 
                       <button data-bookmark="{{$post->id}}" class="for-desctop">Добавить в закладки</button>
 
                        @endif
 
 
</div>

[jazzus]

Вы что id закладки сравниваете c id пользователя? Это не будет давать корректный ответ. У вас просто Id пользователя и закладки равен 1, поэтому показывает. Добавьте еще пост и проверьте.
Там нет другого варианта.
Это
@if($post->bookmark->count() != 0)
Будет делать запрос в базу для каждого поста. 2000 постов = 2000 запросов в бд при каждом обращении.
Для таких вещей в Laravel придуман withCount, как в моем примере.
Данное отношение
return $this->belongsToMany('App\User');
если нет ключей и таблица не названа по докам (как в вашем примере сверху) или не прописана в модели – не будет давать корректный ответ.
hasMany – это не один автор, а несколько. Один автор это hasOne.
Вы пишете велосипед, который не будет правильно работать. Какие сложности с моим вариантом? Я вам весь код написал.

[Dimon3x]

jazzus, вот так сделал

$posts = Post::withCount(['bookmark' => function ($query) {
$query->where('user_id', Auth::id());
}])->get();

echo $posts[0]->bookmark_count;

dump($posts);

[jazzus]

Dimon3x, bookmark – это у вас модель User?
И вы в модели user ищете user_id?

[Dimon3x]

jazzus, bookmark, это метод для модели поста

[jazzus]

Dimon3x, это тот же manytomany к модели User c промежуточной таблицей только вы его переименовали. Тогда ничего не поменялось, ваш запрос идентичен моему.
Только переименовывать users в bookmark нельзя т.к. это не просто метод, это отношение. Если хотите bookmark нужно создать модель Bookmark и связывать с постами

[Dimon3x]

jazzus, я метод не переименовал, а добавил другой

[jazzus]

Dimon3x, другую промежуточную таблицу сделали?

[Dimon3x]

jazzus, нет, та же самая

[Dimon3x]

jazzus, отношение я изменил

//добавлен ли вопрос в закладки
    public function bookmark()
    {
        return $this->belongsToMany('App\User');
    }

[jazzus]

Dimon3x, тогда это то же самое отношение и запрос. Только вместо users bookmark и не прописаны ключи. Если таблица указана в модели или правильно называется то можно и не прописывать. Но в случае ваших наименований я советую прописывать, чтобы не ошибиться в ключах. Так у вас ошибка осталась или нет?

[Dimon3x]

jazzus, ошибок нет, а как ключи прописать?

[jazzus]

Dimon3x, все также

public function users() {
  return $this->belongsToMany('App\User', 'posts_user', 'post_id', 'user_id');
}

и желательно, чтобы они фактически были в промежуточной таблице

$table->integer('post_id')->unsigned();
 $table->foreign('post_id')->references('id')->on('posts')->onDelete('cascade');
 $table->integer('user_id')->unsigned();
 $table->foreign('user_id')->references('id')->on('users')->onDelete('cascade');

[Dimon3x]

jazzus, у меня же работает, без дополнительных ключей для belongsToMany

[jazzus]

Dimon3x, посмотрите мой комментарий выше от 9апреля