Yuya Terajima
terra****@e2eso*****
2011年 8月 29日 (月) 14:34:48 JST
寺嶋です。 しばらくMLを離れておりましたが、kenjiさんから私宛に連絡が来ているとお知 らせいただいたので戻ってきました。 >shoさん これまでのissueの扱いから、 ・バグであっても数日経つと放置される可能性があったこと ・ちょうどProfiler周りの個人拡張を書いていたので早めに直って欲しかった ・Profilerのskin変更がreview段階にあったので、同時にReactorエンジニアが チェックした方が取込みが早いだろうと考えた(https://github.com/EllisLab /CodeIgniter/pull/279) でpull requestした次第です。 さて、"$enable > 0"追加の件です。 個人的にはその判定は無くても問題ないと考えています。 query_toggel_countの値は、以下の部分でのみ使用されています。 Profiler.phpの202行目付近: $hide_queries = (count($db->queries) > $this->_query_toggle_count) ? 'display:none' : ''; 単純に、走ったクエリの数が query_toggle_countより大きいかの判定のみで す。なので、例えば-1に設定していても問題はないと考えています。 また、個人的な趣味ですが、以下のように考えている部分もあります。 ・-の値を設定したのに、突然デフォルト値の25が設定されて表示されるのが気 持ち悪い ・-1を設定することで、常に折畳み表示ということで扱えると個人的に嬉しい 気になるようであれば、あまり日本のコミュニティだけで考える問題ではないと 思いますので、githubの方にコメントか修正のコードを送っていただいた方がい いと思います。 特にこだわりはないので、"$enable > 0"への反対意見はありません。 よろしくお願いします。 寺嶋 ------------------------------------------------------------------------ //元のメッセージ sho です。 On 8/29/11 12:37 PM, sho wrote: >>> それと，「アプリケーションのプロファイリング」の検証中にバグを見つけたの >>> で本家に報告しておきました。 >>> >>> https://github.com/EllisLab/CodeIgniter/issues/318 >> >> 修正の pull request が投稿されてますね。 > > 本当ですね。yterajima さんありがとうございます。 > > ただ system/core/Output.php の 288 行目は，is_int($enable) が FALSE のと > きは $this->_profiler_sections[$section] に FALSE を入れるより何もしない > (デフォルト値を維持) 方が良くないですか？ 失礼しました。既にこのような動作をするようになっていました。 ただ $enable > 0 という条件を入れておいた方が良いかも知れませんね。 -- sho <sho****@ohkun*****>
