プロジェクト

全般

プロフィール

Bug(バグ) #2420

Minoru Takai さんが約13年前に更新

h3. 現象 

 sfOpenPNEFriendAction クラスに不要なメンバ定義が含まれたままになっている。具体的な記述内容は commit:0df5d28e の差分を見ると分かる。 に不要なメソッド定義などがコミットされている 

 OpenPNE3.6.x 以降 sfOpenPNEFriendAction クラスは、OpenPNE-3.5.0 時点での修正 commit:c1072092 により は 
 下位互換性を保つためのもので、 
 必要な修正は opFriendAction クラスに実装が移動しており、このクラス自体は後方互換性のために残している。 クラスで行っていくべきものです。 

 <pre> 
 <?php 

 /** 
  * This class is for keeping backward compatibility. 
  * 
  * If you want to add new feature to this class, please add this to 
  * the opFriendAction class, a parent class of this class. 
  * And of course using this class is deprecated. You should not begin to 
  * use this class, and you have to replace the code that is using this class. 
  */ 
 abstract class sfOpenPNEFriendAction extends opFriendAction 
 { 
   public function preExecute() 
   { 
 (中略) 
   } 
 } 
 </pre> 

 このクラスの実装は上記のように、クラスの継承のみで、メンバの定義は一切記述されていないべきである。 このクラスのメソッド定義は削除し、opFriendAction クラスを 
 継承しているのみの状態が正しい状態と思います 

 h3. 原因 

 * (a) 1個目のコミット 2009/12/22 (OpenPNE-3.3.2) commit:8772766f 
 * (b) 2個目のコミット 2009/12/28 (OpenPNE-3.3.2) commit:69a6e482 
 * (c) 別のコミット 2010/03/03 (OpenPNE-3.5.0) commit:c1072092 
 * (d) 3個目のコミット 2010/12/28 (OpenPNE-3.7.0) commit:70cb7314 不要なバックポートのコミットが原因 

 #368 のチケットでは上記の流れでコミットが行われている。 

 * (a), (b) では sfOpenPNEFriendAction クラスのメソッドを修正している commit:0df5d28e 
 * (c) では sfOpenPNEFriendAction クラスを opFriendAction クラスに変更し、後方互換性(※)のために sfOpenPNEFriendAction クラスを残している 
 ** ※ 新しいバージョンにおいて、古いバージョンの仕様を前提にコードが書けること(ここでは sfOpenPNEFriendAction クラスを使えるようにしておくこと)を意味する 
 * (d) では、 (c) の修正を前提に opFriendAction クラスのメソッドを修正している commit:41fbb553 

 つまり、 (a), (b) のコミットは (c) のコミットよりも前であることが前提となっている。 

 しかし #2190 では (c) のコミット後に (a), (b) をバックポートで取り込んでおり、逆に言えば (a), (b) のコミット後に (c) のコミットが無いために sfOpenPNEFriendAction クラスにメソッド定義が残っているという状況となっている。 

 h3. 修正内容 

 sfOpenPNEFriendAction クラスのメソッド定義を削除し、 opFriendAction を継承するのみの状態に戻す。おそらく #2190 の commit:0df5d28e, commit:41fbb553 を revert すればよい。 を継承するのみの状態に戻す 

 h3. 対象バージョン 

 OpenPNE-3.6RC1 ( stable-3.6.x ブランチ) OpenPNE3.6RC1 でのみ発生、3.7では問題ない

戻る