Почему возникает ошибка Duplicate entry for key user_id_unique?

Question

[jazzus]

Laravel 5.8, php 7.3, Mysql, Ubuntu
Ошибка
Integrity constraint violation: 1062 Duplicate entry for key 'user_locations_user_id_unique' (SQL: insert into `user_locations` поля).
Ошибка только на продакшене. Тесты не воспроизводят. Возникает иногда у некоторых пользователей по непонятным причинам. Эти пользователи ничем не отличаются от других. Смотрел запись метрики – пользователь добавляет данные без каких-либо проблем. Делает только ошибки Form Request валидатора (не заполняет обязательное поле), затем заполняет - запись сохраняется и в логах ошибка. Думал, что исправил через $request->except('id', 'user_id') Но в логах опять появилось (20 записей от одного пользователя, хотя щелкал сохранить 2 раза). Код

public function store(ContactRequest $request)
 {
       $user = Auth::user();
       $country = Country::find($request->input('country_id'));

       if (!$id = $request->input('id')) {
           $location = new UserLocation;
           $request['user_id'] = $user->id;
           $location=$location->create($request->all());
       }else{
           $location = UserLocation::findOrFail($id);
       }

       if (!$country->isOtherCountry()) {
            $request['other_country'] = null;
            $request['other_code'] = null;
       }

       if ($location->update($request->except('id', 'user_id'))) {
           return response()->json(['success_msg'=>'Адрес сохранен']);
       }
 }

Это может быть связано с продакшн сервером? Что проверить?
Update
Переписал создание через отношение пользователя
$user->location()->create($request->all());

Comments

[Alex Wells]

Прочитать ошибку не судьба?

[jazzus]

Alex Wells, в смысле "прочитать ошибку"? Вы про уникальный user_id или что-то другое?

[Alex Wells]

jazzus, да, я про уникальный user_id.

[jazzus]

Alex Wells, как я уже писал - ошибка не воспроизводится тестами. Возникает только на продакшене "за кадром". Т.е. для пользователя ошибки нет - запись создается. Только в логах. 2 недели нет ошибки, потом вдруг - от 10 записей для пользователя. Полностью повторял действия пользователя (по записям с метрики) с регистрацией, не заполнением поля и т.д. - ошибки нет. Возможно связано с сервером т.к. на локалхосте ни разу не было ее.

[Nikita Dergachov]

Вероятно пользователь несколько раз клацает на кнопку сохранить и отправляется несколько запросов и получается ошибка дублирования уникального поля.

[Alex Wells]

jazzus, ну так твой говнокод позволяет "ломать систему". Не передаешь id - и создается новый, вот тебе и ошибка. Вообще не понимаю нахрена там ID, если у юзера может быть только один location. Огромная дыра в коде и идиотский код.

[jazzus]

Alex Wells, у тебя все хорошо? Не нужно так нервничать. Id там передается, когда редактирование. Проверку по id делал, когда location было hasMany. Один метод т.к. не знал тогда, как на фронте менять роуты в одном компоненте. Писал давно, трогать не хотел. В любом случае мой вопрос не об этом.

[jazzus]

V M, пробовал несколько раз щелкать по кнопке сохранить – ошибки нет, все создается/перезаписывается без ошибок.

[Alex Wells]

jazzus, в смысле "вопрос не об этом"? Вопрос об этом. Юзер отправляет запрос без ID - вот и ломается твой мега-код.

Все вот это вообще делать нельзя:

$request['user_id']
$request['other_country']
$request['other_code']
$location->update($request->except('id', 'user_id'))

Все с нуля напиши с правильной валидацией и логикой работы, тут работы на две минуты максимум - и уйдет твоя проблема.