Как обновлять данные в БД после нажатия на кнопку, Laravel?

Question

[DenKG]

Есть у меня две таблицы: Purses и Transactions. В таблице Purses есть поле purse_balance. После выполнения транзакции оно должно обновляться, т.е. к предыдущему значению что-то прибавляется. Я написал код, но толку от него мало, потому что вылетают ошибки:

public function trans(Request $request){

   $transactions = new Transaction;

   $transactions->trans_pay_sum = $request->trans_pay_sum;
   $transactions->trans_pay_appoint = $request->trans_pay_appoint;
   $transactions->trans_pay_purse_id = $request->trans_pay_purse_id;
   $transactions->trans_pay_purse_client_id = Auth::id();
   $transactions->save();

   $purses = new Purse;
   $purses->purse_name = $request->trans_pay_purse_id;
   $purses->purse_balance = $this->sum($request);
   $purses->save();

    return view('home');
}
  public function sum( Request $request){
    $result = '';
    $result= $request->trans_pay_sum;
    return $result;
  }

Теперь ошибка:
QueryException in Connection.php line 769:
SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry '3774817312069704' for key 'purse_name' (SQL: insert into `purses` (`purse_name`, `purse_balance`, `updated_at`, `created_at`) values (3774817312069704, 1000, 2016-12-01 04:00:20, 2016-12-01 04:00:20))

Comments

[DevMan]

DenKG подсветка кода? не, не слышал.

[DenKG]

Я тут понимаю, что не указано поле, в которое должно записываться значение, но я не знаю, как это сделать:(
Строчка $purses->purse_name = $request->trans_pay_purse_id; просто перезаписывает предыдущее имя кошелька, но ведь не указывает, в какой нужно записывать.
А LeftJoin вообще отказывается работать:
$transactions = DB::table( 'purses' )

->leftJoin( 'transactions', 'purses.purse_balance', '=', 'transactions.trans_pay_sum' )
->where('purses.purse_name', '=', 'transactions.trans_pay_purse_id')
->get();

[DenKG]

DevMan: я впервые тут. Не умею, правда

[DevMan]

DenKG: в редакторе выдели код и нажми на кнопку с тремя точками.

Answer 1

[Антон Сашнин]

purse_name должно быть уникальным, поэтому и ошибка.

Comments

[DenKG]

Оно и так уникальное

[Антон Сашнин]

DenKG: в БД уже существует запись с таким purse_name, проверьте.

[DenKG]

antonsash: да, это я уже понял и написал об этом ниже, но не знаю, как исправить :(

[Антон Сашнин]

DenKG: как убрать уникальность? Просто уберите unique() для поля purse_name в миграции purses.
После php artisan migrate:rollback, php artisan migrate.

[DenKG]

antonsash: так поле и должно быть уникальным. Просто получается так, что в коде не указано, в каком кошельке должна происходить это операция.

[DenKG]

И вот я не знаю, как указать, в каком кошельке должна происходить эта операция.

[Антон Сашнин]

DenKG: тогда нужно смотреть миграции, модели :-) Но смысл вашей в не уникальности значения поля.

[DenKG]

antonsash: А что там смотреть? Там пусто :)

[Антон Сашнин]

DenKG: покажите Transaction, Purse

[DenKG]

antonsash: <?php

namespace App;

use Illuminate\Database\Eloquent\Model;

class Transaction extends Model
{
//
}

<?php

namespace App;

use Illuminate\Database\Eloquent\Model;

class Purse extends Model
{
//
}

[Антон Сашнин]

DenKG: поля указываются в свойстве fillable, например:
protected $fillable = ['trans_pay_sum', 'trans_pay_appoint'];

[DenKG]

antonsash: это в контроллере?

[Антон Сашнин]

DenKG: это в моделях

[DenKG]

antonsash: ну хорошо, а как мне заполнить конкретное поле-то? У меня он просто создает новую запись:
40
NULL (номер кошелька)
NULL (валюта)
2016-12-01
2016-12-01
1000.00 (сумма)

[Антон Сашнин]

Долго объяснять, читайте -- https://laravel.com/docs/5.3/eloquent

[DenKG]

antonsash: добрался я уже до такой ошибки:
QueryException in Connection.php line 769:
SQLSTATE[42S22]: Column not found: 1054 Unknown column 'purses.id' in 'where clause' (SQL: select * from `purses` where `purses`.`id` = 3774817312069704 limit 1)

<?php

namespace App;

use Illuminate\Database\Eloquent\Model;

class Purse extends Model
{
protected $fillable = ['purse_balance'];
public function transaction()
{
return $this->hasOne('Purse', 'purse_balance');
}
}

Почему он ищет purses_id?

[Антон Сашнин]

DenKG: в миграции purses нет поля id, укажите.

[DenKG]

antonsash: ну так его там и не должно быть

[Антон Сашнин]

DenKG: должен быть, вы же указали связь transaction в purse.

[DenKG]

antonsash: https://laravel.ru/docs/v4/eloquent#oo
Здесь написано:
Если вы хотите назвать внешний ключ по другому, передайте это имя вторым параметром к методу belongsTo()

Я так и сделал:
public function transactions()
{
return $this->belongsTo('Transaction', 'purse_name', 'trans_pay_purse_id');
}

[DenKG]

Но ошибка та же

[Антон Сашнин]

DenKG: напиши в скайп - antonsash, через 3 часа смогу помочь.

Answer 2

[Антон Перевощиков]

Duplicate entry '3774817312069704'

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

Comments

[DenKG]

я сделал по-другому:

public function transactions()
{
return $this->belongsTo('Transaction', 'purse_name', 'trans_pay_purse_id');
}

В контроллере прописал так:

$purse = Purse::find($request->trans_pay_purse_id)->transactions;
$purse->purse_balance = $this->sum($request);
$purse->save();

Но теперь он почему-то пытается записать это всё в поле purses.id, которое я нигде не указывал.

SQLSTATE[42S22]: Column not found: 1054 Unknown column 'purses.id' in 'where clause' (SQL: select * from `purses` where `purses`.`id` = 3774817312069704 limit 1)

Говорят, что в миграциях что-то делать нужно, но я чайник в этом деле, даже не знаю, как эту миграцию создать и что в ней прописывать:(