OpenPNE 3

更新履歴 5bd27b44b7212ec5f80fafc650c34b941e79ba1c で適用されました。

更新履歴 bfc258b3df1716ffa3dde0f6c7aa2326b9825714 で適用されました。

note-4 の指摘について。

まず僕がチケットに記載した仕様はsymfonyのadminモジュールで使われている doctrine.admin.save_object 等の既存のイベントに似せて書いたものです。
パラメータの object というキーも既存のものに合わせた名前となっています。これについては record なり適切な名称に変更することは問題ないと思います。

しかし、修正コミット (bfc258b3) に含まれている opDoctrineEvent を用いた方法は適切ではないと思います。これは、 opDoctrineEvent というクラスを作成する設計が悪いという意味ではなく、「他のイベントがそうではない」ためです。現状の、パラメータに連想配列を使うスタイルが綺麗でないとして別の方法を使ったとしても、以前のスタイルに慣れ親しんでいる側としては $event->getHogehoge() という書き方はいささか不自然に感じます。それに、イベントによって $event["hogehoge"] と $event->getHogehoge() が混在することは決して良いとは思いません。

要約: 下記の修正で十分ではないでしょうか。

@@ -25,7 +25,7 @@ class opDoctrineEventNotifier extends Doctrine_Record_Listener
     }

     $dispatcher = sfContext::getInstance()->getEventDispatcher();
-    $dispatcher->notify(new sfEvent(null, sprintf('op_doctrine.%s_%s_%s', $when, $action, get_class($doctrineEvent->getInvoker())), array('object' => $doctrineEvent->getInvoke
+    $dispatcher->notify(new sfEvent(null, sprintf('op_doctrine.%s_%s_%s', $when, $action, get_class($doctrineEvent->getInvoker())), array('record' => $doctrineEvent->getInvoke
   }

   public function preSave(Doctrine_Event $event)

Youichi Kimura は書きました:

note-4 の指摘について。

まず僕がチケットに記載した仕様はsymfonyのadminモジュールで使われている doctrine.admin.save_object 等の既存のイベントに似せて書いたものです。
パラメータの object というキーも既存のものに合わせた名前となっています。これについては record なり適切な名称に変更することは問題ないと思います。

しかし、修正コミット (bfc258b3) に含まれている opDoctrineEvent を用いた方法は適切ではないと思います。これは、 opDoctrineEvent というクラスを作成する設計が悪いという意味ではなく、「他のイベントがそうではない」ためです。現状の、パラメータに連想配列を使うスタイルが綺麗でないとして別の方法を使ったとしても、以前のスタイルに慣れ親しんでいる側としては $event->getHogehoge() という書き方はいささか不自然に感じます。それに、イベントによって $event["hogehoge"] と $event->getHogehoge() が混在することは決して良いとは思いません。

要約: 下記の修正で十分ではないでしょうか。
[...]

以前のスタイルとして連想配列から値を取ってくるという使い方を提供すべきという点については理解しました．これについては note-9 の方法もいいと思います．ただ今回の修正が悪いというわけでもないため，今回の修正に以前のスタイルの値の取得方法を提供するという案でどうでしょうか？

note-11は opDoctrineEvent にArrayAccessによるパラメータの取得とgetterメソッドを両方備える折衷案という解釈でよろしいでしょうか？ もしそうであれば僕は問題ないと思います。

note-12 の通りです．懸念点としては getSubject() の値と ArrayAccess で取得された内容に差異が出る可能性があるため， getSubject() をオーバーライドして ArrayAccess で得られる値を返すようにすべきという点でしょうか．

修正案

diff --git a/lib/event/opDoctrineEvent.class.php b/lib/event/opDoctrineEvent.class.php
index 912c28e..1fc78a9 100644
--- a/lib/event/opDoctrineEvent.class.php
+++ b/lib/event/opDoctrineEvent.class.php
@@ -21,7 +21,10 @@ class opDoctrineEvent extends sfEvent

   public function __construct(Doctrine_Event $event, $when, $action, $parameters = array())
   {
-    parent::__construct($event->getInvoker(), sprintf('op_doctrine.%s_%s_%s', $when, $action, get_class($event->getInvoker())), $parameters);
+    $invoker = $event->getInvoker();
+    $parameters['record'] = $invoker;
+
+    parent::__construct($invoker, sprintf('op_doctrine.%s_%s_%s', $when, $action, get_class($invoker)), $parameters);
     $this->doctrineEvent = $event;
   }

@@ -29,4 +32,9 @@ class opDoctrineEvent extends sfEvent
   {
     return $this->doctrineEvent;
   }
+
+  public function getSubject()
+  {
+    return $this['record'];
+  }
 }

sfEvent では subject を変更する手段がコンストラクタでのみ提供されていますが， opDoctrineRecord の note-14 の実装だと $parameters の値を変更することで subject を変更で来てしまうというハックができてしまいますが，これは想定されない使い方として扱うものとします．

更新履歴 4ad01d6ca9ed72bc3d4fe304ed77d18a527a710c で適用されました。

  static function tetete(opDoctrineEvent $event)
  {
    var_dump($event->getSubject()->getId());
    var_dump($event['record']->getId());
 }

↑のような使い方をすることによって，期待する値を取得することができます．