OpenPNE 3

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

問題ないように見えますが、パフォーマンスの観点から以下二点指摘させてください（細かいですが……）

opProjectConfiguration に正規表現でマッチングをおこなっている箇所がありますが、sf_task_name をセットしたい状況では $subject にはタスクのクラスインスタンスが格納されているのですから、 instanceof 演算子で sfTask を継承しているかどうかをチェックすればよいはずです。

$subject = $event->getSubject();
if (preg_match('/Task$/', get_class($subject)))
{
  sfConfig::set('sf_task_name', get_class($subject));
}

また、 opApplicationConfiguration にも正規表現でマッチングをおこなっている箇所がありますが、前述のようなコードでは sf_task_name にタスクのクラス名以外の文字列が渡ることは考えにくいため、 Task で終わっている文字列かどうかをチェックする必要はないと思われます。
Task で終了しているかどうかのチェックが不要であるという前提だと、 sfDoctrineBuild で開始されているかどうかをチェックするだけになるので、正規表現よりも strpos() でのチェックの方がお手軽かつパフォーマンス的に有利ということになると思いますがどうでしょうか。

if (!preg_match('/^sfDoctrineBuild.*Task$/', sfConfig::get('sf_task_name')))

確かにその通りです。修正します。

Kousuke Ebihara は書きました:

問題ないように見えますが、パフォーマンスの観点から以下二点指摘させてください（細かいですが……）

opProjectConfiguration に正規表現でマッチングをおこなっている箇所がありますが、sf_task_name をセットしたい状況では $subject にはタスクのクラスインスタンスが格納されているのですから、 instanceof 演算子で sfTask を継承しているかどうかをチェックすればよいはずです。
[...]

また、 opApplicationConfiguration にも正規表現でマッチングをおこなっている箇所がありますが、前述のようなコードでは sf_task_name にタスクのクラス名以外の文字列が渡ることは考えにくいため、 Task で終わっている文字列かどうかをチェックする必要はないと思われます。
Task で終了しているかどうかのチェックが不要であるという前提だと、 sfDoctrineBuild で開始されているかどうかをチェックするだけになるので、正規表現よりも strpos() でのチェックの方がお手軽かつパフォーマンス的に有利ということになると思いますがどうでしょうか。
[...]

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