Модель Laravel с двумя первичными ключами update [дубликат]

этот вопрос уже есть ответ здесь:

Я пытаюсь обновить модель, которая имеет два первичных ключа.

модель 

namespace App;

use IlluminateDatabaseEloquentModel;

class Inventory extends Model
{
    /**
     * The table associated with the model.
     */
    protected $table = 'inventories';

    /**
     * Indicates model primary keys.
     */
    protected $primaryKey = ['user_id', 'stock_id'];
...

миграция

Schema::create('inventories', function (Blueprint $table) {
    $table->integer('user_id')->unsigned();
    $table->integer('stock_id')->unsigned();
    $table->bigInteger('quantity');

    $table->primary(['user_id', 'stock_id']);

    $table->foreign('user_id')->references('id')->on('users')
        ->onUpdate('restrict')
        ->onDelete('cascade');
    $table->foreign('stock_id')->references('id')->on('stocks')
        ->onUpdate('restrict')
        ->onDelete('cascade');
});

это код, который должен обновить инвентарь модель, но это не так.

$inventory = Inventory::where('user_id', $user->id)->where('stock_id', $order->stock->id)->first();
$inventory->quantity += $order->quantity;
$inventory->save();

Я получаю эту ошибку:

Illegal offset type

Я также попытался использовать метод updateOrCreate (). Он не работает (я получаю ту же ошибку).

может ли кто-нибудь сказать, как модель с двумя первичными ключами должна быть обновлена?

12
composite-primary-key eloquent laravel laravel-5 php

2 ответов

я столкнулся с этой проблемой пару раз. Вам нужно переопределить некоторые свойства:

protected $primaryKey = ['user_id', 'stock_id'];
public $incrementing = false;

и методы (кредит):

/**
 * Set the keys for a save update query.
 *
 * @param  \Illuminate\Database\Eloquent\Builder  $query
 * @return \Illuminate\Database\Eloquent\Builder
 */
protected function setKeysForSaveQuery(Builder $query)
{
    $keys = $this->getKeyName();
    if(!is_array($keys)){
        return parent::setKeysForSaveQuery($query);
    }

    foreach($keys as $keyName){
        $query->where($keyName, '=', $this->getKeyForSaveQuery($keyName));
    }

    return $query;
}

/**
 * Get the primary key value for a save query.
 *
 * @param mixed $keyName
 * @return mixed
 */
protected function getKeyForSaveQuery($keyName = null)
{
    if(is_null($keyName)){
        $keyName = $this->getKeyName();
    }

    if (isset($this->original[$keyName])) {
        return $this->original[$keyName];
    }

    return $this->getAttribute($keyName);
}

Я предлагаю поместить эти методы в HasCompositePrimaryKey черта, так что вы можете просто use это в любом из ваших моделей, которые нуждаются в этом.

40

Я решил это, добавив incrementing id и изменив праймериз на uniques.

Schema::create('inventories', function (Blueprint $table) {
    $table->increments('id');
    $table->integer('user_id')->unsigned();
    $table->integer('stock_id')->unsigned();
    $table->bigInteger('quantity');

    $table->unique(['user_id', 'stock_id']);

    $table->foreign('user_id')->references('id')->on('users')
        ->onUpdate('restrict')
        ->onDelete('cascade');
    $table->foreign('stock_id')->references('id')->on('stocks')
        ->onUpdate('restrict')
        ->onDelete('cascade');
});

кроме того, я удалил из модели

protected $primaryKey = ['user_id', 'stock_id'];

на мой взгляд, это не лучшее решение.

0