プラグインを読み込み,変数にあらかじめセットしておく
Wikiにも追記しました.
>mumumuさん ブランチのコードレビューしていただけますか.問題なければ master にマージします.
コードを一通り見て、テストを実行してみました。 以下にいくつかコメントを示しますが、最初の二つは感想なのでスルーしてもらって構いません。
三つめの点について合意がとれるのであれば master にマージして問題ありません。
- アイディアレベルでは問題ありません。特に、エイリアス機能は優れていると思います。 - これまでの機能と衝突しないかを見ましたが、僕が見た限りでは問題ないと思います。 - test/Ethna_Plugin_Test.php 及び test/Ethna_ViewClass_Test.php に (instanceof や method chain等の) PHP5 でしか通じないコードが含まれています。これをmasterにマージすると「以後 PHP5 でしか Ethna はテストできない(しない)」ことを意味しますが、それを意図して書いていますか?
>PHP5 なんどかIRCで呼びかけてこの件は相談しようとは思っていたのですが,つかまらなかったのでMLにしますね.
特にそういう意図があったわけではありませんが,根底にPHP4はサポートしなくていいのでは,という考えがあったからそういうコードを平気で書いていたのかもしれません.現状の方針のままでは問題アリですが,方針そのものを見直すべきだとは思っています.>PHP4でテストできない(しない)
sotarok への返信
>PHP5 なんどかIRCで呼びかけてこの件は相談しようとは思っていたのですが,つかまらなかったのでMLにしますね.
すみません。最近IRCに出られる状態じゃないのです(汗
特にそういう意図があったわけではありませんが,根底にPHP4はサポートしなくていいのでは,という考えがあったからそういうコードを平気で書いていたのかもしれません.現状の方針のままでは問題アリですが,方針そのものを見直すべきだとは思っています.>PHP4でテストできない(しない)
PHP5に関して見直す議論は別件と看做して、該当するPHP5専用テストをとりあえずコメントアウトしてマージしてしまうのもアリだと思います。 レビュアーとしての僕はそれ以外のテストやコード自体は問題ないと判断していますので。
すみません。最近IRCに出られる状態じゃないのです(汗
問題ないですー,そのためのMLですので.
とりあえずMLの議論の流れ次第ですが,もしかしたらコメントアウトせずにマージする形になるかもしれませんね.
などとしておくと
などとして使える.変数名にエイリアスを決めることもできるようにする(ドラフト) ex: