Sync синхронизирует,но удаляет другие записи в бд,что нужно чтобы новые записи синхронизировались только для текущего пользователя?

Question

[Артем Кашапов]

Получаю пользователя через checkbox,также получаю массив с товарами которые отмечены в checkbox

Дело в том что все работает,например $data[$user->id] отправляет массив товаров и id пользователя в метод saveProducts.
Все прекрасно записывается,но дело в то что удаляются записи для другого пользователя.В чем может быть ошибка?
Далее

$data = $request->except('_token');
        $users = User::all();
        foreach ($users as $user){
            if(isset($data[$user->id])){
                $user->saveProducts($data[$user->id]);
            }
            else {
                $user->saveProducts([]);
            }
        }

Метод для сохранения товаров

public function saveProducts($products)
    {
        if(!empty($products)){
            $this->products()->sync($products);
        }
        else{
            $this->products()->detach();
        }
    }

Связь в модели User

public function products() {
        return $this->belongsToMany(Product::class,'product_user');
    }

Comments

[Артем Кашапов]

Понял ошибку,я перебираю всех пользователей.
Как можно коротко написать код для одного только автора?

foreach ($users as $user){
            if(isset($data[$user->id])){
                $user->saveProducts($data[$user->id]);
            }
            else {
                //
                $user->saveProducts([]);
            }
        }

Answer 1

[Сергей delphinpro]

spoiler

Текущего пользователя можно получить

\Auth::user()
// или
$request->user()

UPD

Если админ может редактировать одновременно только одного пользователя, то необязательно передавать массив данных как вы показывали выше
Можно ID пользователя передать в отдельном скрытом поле.
То есть примерно так

<input type=hidden name=user_id value={{ ID редактируемого юзера }}>
<!-- список товаров -->
<input type=checkbox name="products[]" value={{ ID товара }}>
<input type=checkbox name="products[]" value={{ ID товара }}>
<input type=checkbox name="products[]" value={{ ID товара }}>

И тогда при получении

$userId = $request->get('user_id');
$user = User::findOrFail($userId);
$products = $request->get('products', []);
$user->saveProducts($products);

Comments

[Сергей delphinpro]

Вот это вообще не понятно isset($data[$user->id]
В каком виде приходят данные и зачем там ID пользователя?
Может лучше просто получать данные, валидировать их и сохранять

$data = $request->validate([
 // Validation rules
]);
$user->products()->sync($data);

А detach по умолчанию будет сделан, если массив пуст.

[Артем Кашапов]

данные приходят в виде массива

array:1 [▼
  139 => array:2 [▼
    0 => "4"
    1 => "5"
  ]
]

139 это id пользователя,а 4 и 5 это id товаров

[Артем Кашапов]

Вопрос,как мне получить id пользователя 139?

Типо так,$request->id

[Артем Кашапов]

Сделал что-то типо этого,

$data = $request->except('_token');
        $user = User::where('id',129)->first();
        $user->products()->sync($data);

Но получаю ошибку
SQLSTATE[42S22]: Column not found: 1054 Unknown column '0' in 'field list

[Сергей delphinpro]

Mass1veDit, так я и спрашиваю – зачем передавать ID пользователя?
Это же действие выполняет сам пользователь? Значит он залогинен. Значит вы можете получить его, как я показал выше. И все его данные

$user = Auth::user();
$user->id;

И значит передавать ID пользователя не нужно, а просто массив данных

array:2 [▼
    0 => "4"
    1 => "5"
]

а дальше как писал выше.

[Артем Кашапов]

Сергей delphinpro, "так я и спрашиваю – зачем передавать ID пользователя?"
Админ выбирает автора любого и может изменить его товары, по которым он может работать

[Артем Кашапов]

Мне необходимо вывести категорию,в категориях товары. При выборе чек бокса нужно оставить все также,для данного выбранного пользователя.
и при detach убрать только его записи которые в на данной странице отображены.

[Артем Кашапов]

Убираются все записи при detach)

[Сергей delphinpro]

Mass1veDit, а, ну то есть всё-так админ, а не сам пользователь. Вы там написали про текущего пользователя, вот я и подумал не то.

[Сергей delphinpro]

Mass1veDit, Если админ может редактировать одновременно только одного пользователя, то необязательно передавать массив данных как вы показывали выше
Можно ID пользователя передать в отдельном скрытом поле.
То есть примерно так

<input type=hidden name=user_id value={{ ID редактируемого юзера }}>
<!-- список товаров -->
<input type=checkbox name="products[]" value={{ ID товара }}>
<input type=checkbox name="products[]" value={{ ID товара }}>
<input type=checkbox name="products[]" value={{ ID товара }}>

И тогда при получении

$userId = $request->get('user_id');
$user = User::findOrFail($userId);
$products = $request->get('products', []);
$user->saveProducts($products);

[Артем Кашапов]

Сергей delphinpro, а при detach как поступить?у меня же пустой массив отправляется если чекбоксы не выбраны,
и тогда удаляются и те товары для выбранного пользователя, который не входит в категорию

[Сергей delphinpro]

Mass1veDit, тут проще всего вместе с текущей категорией в скрытых инпутах отправлять отмеченные товары из других категорий.

<input type=hidden name=user_id value={{ ID редактируемого юзера }}>
<!-- список товаров текущей категории -->
<input type=checkbox name="products[]" value={{ ID товара }}>
<input type=checkbox name="products[]" value={{ ID товара }}>
<input type=checkbox name="products[]" value={{ ID товара }}>
<!-- список уже отмеченных товаров других категории -->
<input type=hidden name="products[]" value={{ ID товара }}>
<input type=hidden name="products[]" value={{ ID товара }}>

Тогда у вас всегда будет прилетать актуальный список товаров.

Или дополнительно отправлять ID категории и уже в контроллере к полученному масcиву products добавлять подключенные товары из других категорий.

[Артем Кашапов]

Сергей delphinpro, Понял,благодарю)