CakePHP4のsaveメソッドはプライマリキー違反を検出しないのか

実現したいこと

MySQLのテーブルに登録時に、既に登録されているプライマリキーの場合、上書きされないようにしたい。

前提

CakePHP4.4.15
Amazon Linux2
Apache
Amaon RDS (MySQL community8.0.33)

以上の環境でWEBアプリを稼働させようとしています。

発生している問題・エラーメッセージ

Customersというテーブルを作成し、
プライマリキーにidというカラムを指定。

id 1000000001
が既にDBに存在する場合に、
newEmptyEntityを使用したcustomerエンティティーに、
同じID1000000001
を入れてsaveすると、
後から入力した情報で上書きされてしまう。

これが正しい動きなんでしょうか。
プライマリキー違反でエラーになって欲しいんですが。

該当のソースコード

$customer = $this->Customers->newEmptyEntity();
$customer->id = '1000000001';
&customer->name = 'bbb';
$customer->isNew(); // この時点でtrue
$this->Customers->save(&customer);

ikedas

2023/08/25 00:43

ドキュメントに書いてある通りの動作ではないでしょうか。 https://book.cakephp.org/4/ja/orm/saving-data.html

行動規範の内容に同意します

回答1件

私はほとんどtableクラスのORMは使わないので気にならなかったのですけど
少し調べてみました。
xdebugなどで動きをみると、isNew()がtrueであっても、存在チェックを行って存在していたら更新に切り替えています

Cookbookに次のようにありますが、こういう仕様であるなら読み取りにくいように感じますね

もし isNew() が真を返し、エンティティーが主キー値を持っていれば、 'exists' クエリーが発行されます。 'exists' クエリーは $options 引数に 'checkExisting' => false を渡すことで抑制することができます。

https://book.cakephp.org/4/ja/orm/saving-data.html#saving-entities

以下のようにすると、新規の時は重複チェックをせずにそのまま実行し
（キー重複があれば）例外が発生すると思います

php
1try{
2    $this->Customers->save(
3        $customer,
4        ($customer->isNew()? ['checkExisting' => false] : [] )
5    );
6}catch(\Exeption $e){
7}