CakePHPでHasOneアソシエーションを保存する方法

Question

User モデルが Profile モデルを HasOne で保持しています。Profileモデルの値を更新して保存するときに UsersTable::save() を使いたいんですが patchEntity() を使わないと保存されないのは割り切って使うしかないんでしょうか？
具体的にはこんなコードです。

class User extends Entity {
  // ユーザの属性とプロフィールを $data の値で更新する
  public function updateProfile(array $data)
  {
    // いろいろUser自身の属性を更新
    // 住所情報を更新
    $this->profile->changeAddress($data['address']);
    // $this->setDirty('profile', true); これを実行すれば保存される
  }
}

class Profile extends Entity
{
  // 住所変更メソッド
  public function changeAddress($newAddress)
  {
    // 離島に住んでいる人向けの処理があるのでエンティティにロジックを持たせる
    // 住所変更時に属性を更新する
    if ($this->isIsland($newAddress)) {
      $this->_properties['lives_island'] = true;
      $this->_properties['default_transport_fee'] = $this->getDefaultTransportFee($newAddress);
    }
  }
}

class UserProfileController {
  public function update()
  {
    // (いろいろ省略)
    $user->updateProfile($this->request->data);
    $this->Users->save($user);
    // $user->profileも更新されてるけど dirty フラグが立っていないので保存されない
    // $this->Prifile->save($user->profile); を実行してもいいけどそれよりも $this->Users->save($user) で一括保存したい

   // $this->Users->patchEntity($user, $this->request->data) を実行すればdirtyフラグが立つのは知ってるけど、Profile::changeAddress() 内のロジックが実行されない。
  }
}

Answer 1

Entity 内でプロパティに値がセットされたときに、何らかの処理をする場合は
ミューテーターを使うと外部から意識しなくて良くなるので、良いかもしれません。

https://book.cakephp.org/3.0/ja/orm/entities.html#Cake\ORM\Entity::set

これを使用すれば、単純に $this->Users->patchEntity() だけで済むと思いますがいかがでしょうか？

例：

    protected function _setAddress($address)
    {
        if ($this->isIsland($newAddress)) {
            $this->lives_island = true;
            $this->default_transport_fee = $this->getDefaultTransportFee($newAddress);
        }

        return $address;
    }

一応、質問内容のそのものの答えとしては
https://book.cakephp.org/3.0/ja/orm/entities.html#id7
ここにあるように $article->setDirty('address', true); をしてあげれば
Dirty をたてることが出来ます。

Comments

[tomcat0090]

というか、書いてて思ったんですが、

$this->_properties['lives_island'] = true;
$this->_properties['default_transport_fee'] = $this->getDefaultTransportFee($newAddress);

ここは

$this->lives_island = true;
$this->default_transport_fee = $this->getDefaultTransportFee($newAddress);

のほうがお行儀が良い書き方だと思います。

[kodai]

アドバイスありがとうございました！setDirty() を使っていたんですが _setAddress() のミューテータもいいですね。$_propertiesへの設定方法もコメントありがとうございます。