tDiary

ただただしです。

リリース後は一週間くらい頭がうまく働かないので、ちゃんとチェックできない
かも知れません(笑)。

Kenta MURATA <muraken2@...> wrote:
>TDiaryComment#eval_rhtml が TDiaryBase#do_eval_rhtml で起こっ
>た TDiary::PluginError，SyntaxError, ArgumentError を無視して
>しまう状態を修正する patch を作ったので添付します．
>
>patch のサマリを述べると，TDiaryComment#do_eval_rhtml を新たに
>上書き定義して，TDiaryComment#eval_rhtml の定義をそちらに移し
>ました．なので，プラグインのロードで発生した例外は 
>TDiaryBase#eval_rhtml によって通常通り処理されると思います．

従来、プラグインのロード時にはメソッド定義だけが行われて、実際の処理はそ
の後のERbで実行されるというモデルだったのが、最近はロード時に処理を行う
プラグインが増えたので(犯人:ただ)、こういう問題が出てきたわけですね。

パッチを見た限りでは、問題はないように思えます。kitajのチェックも通った
ら:-)、適用しても良いと思います。

>あ，それから，skel/day.rhtml の update_proc の呼出しの部分は必
>要無いと思うので，添付した patch が上手く動いたら消しても良い
>と思います．

kotakさんのところで「プレビューがあれば更新完了ページはいらないのでは」
という指摘があり<http://kotak.dyndns.org/~kotak/diary/?date=20030515#p01>、
「でもupdate_procが呼ばれないのは困るしな」と考えていたところでした。

このパッチを適用したら、更新完了ページをなくしてしまう改造も一緒にしてし
まうセンもありですね。

━━━━━━━━━━━━━━━━━━━
ただただし <http://www.spc.gr.jp/sho/>
    ♪ツッコミは、短く鋭く愛を込めて。
━━━━━━━━━━━━━━━━━━━

ただただしです。

On Fri, 16 May 2003 18:28:24 +0900 TADA Tadashi <sho@...> wrote:
>パッチを見た限りでは、問題はないように思えます。kitajのチェックも通った
>ら:-)、適用しても良いと思います。

とりあえずこのパッチを適用し、commitしました。

その他の部分はまたいずれ。

━━━━━━━━━━━━━━━━━━━
ただただし <http://www.spc.gr.jp/sho/>
    ♪ツッコミは、短く鋭く、愛をこめて
━━━━━━━━━━━━━━━━━━━