チケット #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');

チケットの履歴 (9 件中 3 件表示)

2009-08-27 05:01 更新者: sotarok
  • 新しいチケット "プラグインを読み込み,変数にあらかじめセットしておく" が作成されました
2009-09-19 04:13 更新者: sotarok
コメント

Wikiにも追記しました.

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

2009-10-06 06:57 更新者: 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 更新者: sotarok
コメント

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

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

2009-10-07 13:07 更新者: mumumu-org
コメント

sotarok への返信

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

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

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

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

2009-10-08 17:13 更新者: sotarok
コメント

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

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

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

2009-10-10 14:32 更新者: sotarok
  • マイルストーン2.5.0p6 (削除済み) から 2.6.0 に更新されました
2011-10-23 01:45 更新者: sotarok
  • 解決法なし から 修正済み に更新されました
  • 状況オープン から 完了 に更新されました
  • チケット完了時刻2011-10-23 01:45 に更新されました

添付ファイルリスト

添付ファイルはありません

編集

このチケットにコメントを追加するには、ログインが必要です » ログインする