待辦事項 #18417

プラグインを読み込み,変数にあらかじめセットしておく

啟用日期: 2009-08-27 05:01 最後更新: 2011-10-23 01:45

回報者:
負責人:
類型:
狀態:
關閉
元件:
里程碑:
優先權:
5 - 中
嚴重程度:
5 - 中
處理結果:
修正
檔案:

細節

 var $plugins = array('Cachemanager_Localfile");

などとしておくと

 $this->Cachemanager_Localfile->someMethod();
などとして使える.

変数名にエイリアスを決めることもできるようにする(ドラフト) ex:

 var $plugins = array('c' => 'Cachemanager_Localfile", 'm' => 'Cachemanager_Memcache');

Ticket History (3/9 Histories)

2009-08-27 05:01 Updated by: sotarok
  • New Ticket "プラグインを読み込み,変数にあらかじめセットしておく" created
2009-09-19 04:13 Updated by: sotarok
評語

Wikiにも追記しました.

>mumumuさん ブランチのコードレビューしていただけますか.問題なければ master にマージします.

2009-10-06 06:57 Updated by: mumumu-org
評語

コードを一通り見て、テストを実行してみました。 以下にいくつかコメントを示しますが、最初の二つは感想なのでスルーしてもらって構いません。

三つめの点について合意がとれるのであれば master にマージして問題ありません。

- アイディアレベルでは問題ありません。特に、エイリアス機能は優れていると思います。
- これまでの機能と衝突しないかを見ましたが、僕が見た限りでは問題ないと思います。
- test/Ethna_Plugin_Test.php 及び test/Ethna_ViewClass_Test.php に (instanceof や method chain等の) PHP5 でしか通じないコードが含まれています。これをmasterにマージすると「以後 PHP5 でしか Ethna はテストできない(しない)」ことを意味しますが、それを意図して書いていますか? 
2009-10-06 23:01 Updated by: sotarok
評語

>PHP5 なんどかIRCで呼びかけてこの件は相談しようとは思っていたのですが,つかまらなかったのでMLにしますね.

特にそういう意図があったわけではありませんが,根底にPHP4はサポートしなくていいのでは,という考えがあったからそういうコードを平気で書いていたのかもしれません.現状の方針のままでは問題アリですが,方針そのものを見直すべきだとは思っています.>PHP4でテストできない(しない)

2009-10-07 13:07 Updated by: mumumu-org
評語

sotarok への返信

>PHP5 なんどかIRCで呼びかけてこの件は相談しようとは思っていたのですが,つかまらなかったのでMLにしますね.

すみません。最近IRCに出られる状態じゃないのです(汗

特にそういう意図があったわけではありませんが,根底にPHP4はサポートしなくていいのでは,という考えがあったからそういうコードを平気で書いていたのかもしれません.現状の方針のままでは問題アリですが,方針そのものを見直すべきだとは思っています.>PHP4でテストできない(しない)

PHP5に関して見直す議論は別件と看做して、該当するPHP5専用テストをとりあえずコメントアウトしてマージしてしまうのもアリだと思います。 レビュアーとしての僕はそれ以外のテストやコード自体は問題ないと判断していますので。

2009-10-08 17:13 Updated by: sotarok
評語

すみません。最近IRCに出られる状態じゃないのです(汗

問題ないですー,そのためのMLですので.

とりあえずMLの議論の流れ次第ですが,もしかしたらコメントアウトせずにマージする形になるかもしれませんね.

2009-10-10 14:32 Updated by: sotarok
  • 里程碑 Update from 2.5.0p6 (deleted) to 2.6.0
2011-10-23 01:45 Updated by: sotarok
  • 處理結果 Update from to 修正
  • 狀態 Update from 開啟 to 關閉
  • Ticket Close date is changed to 2011-10-23 01:45

Attachment File List

No attachments

編輯

Please login to add comment to this ticket » 登入