проверки таким образом (если данные с поля не пришли то ничего не делай)? Если да то где грамотно подобный код размещать? Спасибо!
if (!empty($postData['tags'])) { // attach } будет достаточно, не нужны там доп переменные, и unset-ы. тем более ты проверил только наличие ключа в массиве, а не значение.
далее, обрати внимание что тебе приходит в сервис: $postData - ты уверен что это массив? у тебя нет типизации, это может быть строка или что угодно. далее. если ты пропишешь array, то будешь ли ты уверен в ключах, которые содердит массив, ты напрямую передаешь в create() (спасибо fillable что спасает немного, если указаны правильно там поля) лучше в такие сервисы передавать не массив, а DTO, или массив придется валидировать в сервисе.
на 19ой строке по хорошему необходимо передавать не какой-то абстрактный массив данных, а указать какие поля ты пишешь. Да, это может показаться лишней работой, но в таком случае будет хотя бы видно что и куда ты пишешь, а не просто какой-то $postData и ищи где он формируется, скорей всего ты так до формы дойдешь, где еще будет if-ы и в итоге без дебага не разберешься.
Мне пока сложновато но я разберусь над всем что написали. Спасибо!
А кстати я не могу туда сразу атач впихнуть потому что пивот требует айди поста которого на тот момент нету.
Сразу сохраняешь модель, потом атачишь
Тогда в таблицу с постами пойдут данные для пивота и она не пропустит. Или я вас не понял.
А чего в таблицу пойдут данные от пивота, для пивота отдельная же таблица
так этот if после create()
не должно. у модели есть fillable - он удалит все ключи, которые там не указаны
Обсуждают сегодня