Model::updateOrCreate почему не считывает в первом параметре, второй элемент массива?

Question

[Hfnas]

Model::updateOrCreate(['amoID' => 43222233, 'crmCompanyID'=>1]);

Выдает ошибку:

Illuminate\Database\QueryException : SQLSTATE[HY000]: General error: 1364 Field 'crmCompanyID' doesn't have a default value (SQL: insert into `Model` (`amoID`, `updated`, `created`) values (43222233, 2025-01-05 20:41:11, 2025-01-05 20:41:11))

Значение по умолчанию нет, но задан параметр.
В бд есть уникальный ключ (amoID, crmCompanyID)

Comments

[Алексей Уколов]

Покажите как у вас модель выглядит. Подозреваю, что дело в fillable/guarded.

[Hfnas]

Алексей Уколов,

class Model extends BaseCRM
{
    protected $table = 'crmAmoContact';

    protected $casts = [
        'created' => 'datetime',
        'updated' => 'datetime',
        'name' => 'string',
        'phone' => 'string',
        'phone1' => 'string',
        'phoneNorm' => 'string',
        'email'=>'string',
        'amoID' => 'int',
        'crmAmoUserID' => 'int',
        'crmAmoUser2ID' => 'int',
        'crmAmoContactTypeID' => 'int',
        'crmCompanyID' => 'int',
    ];

    protected $fillable = [
        'name',
        'amoID',
        // 'crmCompanyID'
    ];
...
}

в бд crmCompanyID'обязательно к заполнению, не может быть нулем.

[Алексей Уколов]

Ну вот - у вас это поле в fillable не указано, поэтому фреймворк его и не заполняет.

[Hfnas]

Алексей Уколов,

AmoContact::firstOrNew(['amoID' => $contacts[0]->getId(),'crmCompanyID'=>$companyID]);

тоже не учитывает второй элемент массива первого параметра,несмотря на то, что

protected $fillable = [
        'name',
        'amoID',
        'crmCompanyID'
    ];

заполнено

[Алексей Уколов]

Да не может оно не учитывать, вы посмотрите код фреймворка - все эти методы литерали из трёх строк.
У вас либо опечатка где-то; либо в переменной, которую вы подставляете, содержится null; либо вы fillable раскомментировали, а какой-нибудь supervisor не перезапустили, если код в очереди исполняется, и там ещё старая версия. Ещё, может, вы сеттер этого атрибута переопределили и он кривой.

[Hfnas]

Алексей Уколов, версия ларавела 5.7 , склоняюсь к тому, что плохо работает версия.

[Алексей Уколов]

Во-первых, о том, что у вас версия из прошлого десятилетия, нужно говорить сразу. Во-вторых, отличия от актуальной версии там, конечно, есть, но код всё равно элементарный и должен работать.
Сейчас ещё окажется, что у вас PHP древний тоже.

[Алексей Уколов]

И вообще такие вопросы решаются дебагом. Версия минимум - вставить дампы прямо в код фреймворка и смотреть, в каком месте ломается. Версия медиум - поставить Telescope (если он заведётся) и смотреть, какие запросы исполняются и что дампится. Версия топ - поставить XDebug.
В 99.9999% случаев проблема в вашем коде, а не в "системном", который прекрасно работает у миллионов других людей.

[ThunderCat]

Model::updateOrCreate(['amoID' => 43222233, 'crmCompanyID'=>1]);

Это реальный код, или вы подставили возможные значения, которые берете динамически (нарпимер в цикле)?

Алсо, у вас там не повторяется каша как в предыдущем вопросе, типа TestLead/CrmLead?

[Hfnas]

ThunderCat, благодарю за помощь!
Я подставила значения, которые получаю в цикле....
Model - абстрактное понятие, так как ломается у меня при любых моделях, , взять тот же

AmoContact::firstOrNew(['amoID' => $contacts[0]->getId(),'crmCompanyID'=>$companyID]);

//тоже не учитывает второй элемент массива первого параметра,несмотря на то, что
protected $fillable = [
        'name',
        'amoID',
        'crmCompanyID'
    ];

//заполнено

Integrity constraint violation: 1048 Column 'crmCompanyID' cannot be null (SQL: insert into `crmAmoContact` (`amoID`, `crmCompanyID`, `updated`, `created`) values (42397607, , 2025-01-06 17:31:54, 2025-01-06 17:31:54))

[ThunderCat]

Hfnas,

AmoContact::firstOrNew(['amoID' => $contacts[0]->getId(),'crmCompanyID'=>$companyID]);

Магия вне Хогвардса хреново работает, так что остается полагаться на логику. Так как мы здесь видим переменную, а не значение, можно предположить что там не то значение, которое вы ожидаете. Поможет dd() или var_dump().

Кроме тогого, емнип второй аргумент должен быть отдельным массивом, а не еще одной парой, то есть

AmoContact::firstOrNew(['amoID' => $contacts[0]->getId()],['crmCompanyID'=>$companyID]);

[Hfnas]

ThunderCat, проще говоря я ищу запись удовлетворяющее условию ['amoID' => $contacts[0]->getId(), ['crmCompanyID'=>$companyID] если нет , то создаем запись, а у меня говорит , что crmCompanyID не может быть равен нулю. Перепишу

AmoContact::firstOrNew(['amoID' => $contacts[0]->getId(),'crmCompanyID'=>$companyID]);

на

$amoContactEntity = AmoContact::where('amoID',$contacts[0]->getId())->where('crmCompanyID',$companyID)->first();
                    if ($amoContactEntity===null){
                        $amoContactEntity=new AmoContact();
                        $amoContactEntity->crmCompanyID=(int)$companyID;
                        $amoContactEntity->amoID=$contacts[0]->getId();
                        $amoContactEntity->name="fromLead";
                        $amoContactEntity->save();
                    }

                    $entity->crmAmoContact2ID = $amoContactEntity->id;

а то выдает ошибку

Illuminate\Database\QueryException  : SQLSTATE[23000]: Integrity constraint violation: 1048 Column 'crmCompanyID' cannot be null (SQL: insert into `crmAmoUser` (`crmCompanyID`, `amoID`, `updated`, `created`) values (, 11198626, 2025-01-07 19:25:50, 2025-01-07 19:25:50)

У меня такая запись в бд есть, просто не учитывает почему то crmCompanyID,

В модели написано

class AmoContact extends BaseCRM
{
    protected $table = 'crmAmoContact';

    protected $casts = [
        'created' => 'datetime',
        'updated' => 'datetime',
        'name' => 'string',
        'phone' => 'string',
        'phone1' => 'string',
        'phoneNorm' => 'string',
        'email'=>'string',
        'amoID' => 'int',
        'crmAmoUserID' => 'int',
        'crmAmoUser2ID' => 'int',
        'crmAmoContactTypeID' => 'int',
        'crmCompanyID' => 'int',
    ];

    protected $fillable = [
        'name',
        'amoID',
        'crmCompanyID' //не учитывает в AmoContact::firstOrNew(['amoID' => $mainUserField,'crmCompanyID'=>$companyID]), AmoContact::updateOrCreate(['amoID' => $mainUserField,'crmCompanyID'=>$companyID])
    ];
.....
}

Метод AmoContact::firstOrNew то работает, то не работает.

По времени исполнения firstOrNew, where работают одинаково. Тогда смысл использовать сырые запросы?
у меня стоит уникальный индекс на пару amoID, crmCompanyID

[Hfnas]

Алексей Уколов, @ThunderCat
В общем замена не помогла на where

$amoContactEntity = AmoContact::where('amoID',$contacts[0]->getId())->where('crmCompanyID',$companyID)->first();
                    if ($amoContactEntity===null){
                        $amoContactEntity=new AmoContact();
                        $amoContactEntity->crmCompanyID=(int)$companyID;
                        $amoContactEntity->amoID=$contacts[0]->getId();
                        $amoContactEntity->name="fromLead";
                        $amoContactEntity->save();
                    }

                    $entity->crmAmoContact2ID = $amoContactEntity->id;

почему пропадает в конструкторе

foreach ($companies as $company){

                $amoCRMIntegration=new AmoCRMIntegration($company->id);
                $amoCRMIntegration->getContacts($all); // пробовала передать $amoCRMIntegration->getLeadsWithContacts($all,(int)$company->id);  .. все равно никак
}

Выдает ошибку:

Integrity constraint violation: 1048 Column 'crmCompanyID' cannot be null (SQL: insert into `crmAmoContact` (`amoID`, `crmCompanyID`, `name`, `updated`, `created`) values (142, , FromLead, 2025-01-07 22:36:29, 2025-01-07 22:36:29))

AmoLeadModel::import($leads,$this->crmCompanyID,$constants);

Далее манпуляции с firstOrNew, where,- обе выдают ошибку...

У меня то работает, то не работает, когда включаю дебаг , тогда работает код, отловить ошибку не получается

$amoStatusEntity = AmoStatus::where('amoID', $row->getStatusId())->where('crmCompanyID',$companyID)->first();
                    if ($amoStatusEntity===null){
//статус существует, но все равно попадает сюда, значит второй where не обрабатывает почему то....
                        var_dump($companyID); //int(1)
                        echo $companyID; //1
                        $amoStatusEntity=new AmoStatus();
                        $amoStatusEntity->amoID=$row->getStatusId();
                        $amoStatusEntity->crmCompanyID=$companyID;
                        $amoStatusEntity->name="fromLead";
                        $amoStatusEntity->save();
                    }

все равно выводит ошибку

int(1)
1
Illuminate\Database\QueryException : SQLSTATE[23000]: Integrity constraint violation: 1048 Column 'crmCompanyID' cannot be null (SQL: insert into `crmAmoStatus` (`amoID`, `crmCompanyID`, `name`, `updated`, `created`) values (142, , FromLead, 2025-01-07 23:01:44, 2025-01-07 23:01:44))

class AmoStatus extends BaseCRM
{
protected $table = 'crmAmoStatus';

protected $casts = [
'created' => 'datetime',
'updated' => 'datetime',
'name' => 'string',
'amoID' => 'int',
'crmCompanyID' => 'int',
];

protected $fillable = [
'name',
'amoID',
'crmCompanyID'
];

[ThunderCat]

Hfnas, Я же говорил - проверьте переменные, у вас в crmCompanyID попадает пустое значение.

[Алексей Уколов]

Когда код «то работает, то нет», это значит, что проблема в данных — это единственное, что меняется.

[Hfnas]

Алексей Уколов,
ThunderCat,

Закомментировала строку модели

protected $fillable = [
        'name',
        'amoID',
       // 'crmCompanyID'
    ];

и все заработало, хотя неправильно, но так получилось. Я никак не могу найти ошибку в переменнных. Спасибо за внимание!!!!!

Answer 1

[zordec]

Метод updateOrCreate() работает следующим образом:

1. Поиск существующей записи: Использует первый массив (условия поиска).
   
2. Обновление или создание записи:
   
   • Если запись найдена, значения из второго массива используются для обновления.
     
   • Если запись не найдена, создаётся новая запись с использованием первого массива для условий и второго массива для заполнения полей.
     
   
   
   

В вашем коде отсутствует указание поля crmCompanyID в параметрах для создания записи. Следовательно, Laravel пытается вставить NULL (или пропускает поле), что вызывает ошибку, так как это поле не допускает NULL и не имеет значения по умолчанию.

Model::updateOrCreate(
    ['amoID' => 43222233], // Условия для поиска
    ['crmCompanyID' => 1, 'updated' => now(), 'created' => now()] // Данные для обновления или создания
);

Comments

[Hfnas]

Model::updateOrCreate(
    ['amoID' => 43222233, 'crmCompanyID' => 1], // Условия для поиска
    ['updated' => now(), 'created' => now()] // Данные для обновления или создания
);

В первом параметре первой функции не учитывает второй элемент массива, если запись не существует, ругается

Column 'crmCompanyID' cannot be null (SQL: insert into `crmAmoContact` (`amoID`, `crmCompanyID`, `updated`, `created`) values (42397607, , 2025-01-06 17:31:54, 2025-01-06 17:31:54))

Этот код стирает поля "updated" , "created", и ставит текущее время, если задашь свои значения для полей updated, "created"

'updated' => "2024-01-01 22:00:24", 'created' => now()

, что мне не подходит.

[zordec]

Hfnas, это пример ты можешь свои поля подставить для обновления