BugTrack/2291

convert_html()で無駄なdigest生成をしている†

ページ: BugTrack2
投稿者: ぃぉぃぉ
優先順位: 低
状態: 提案
カテゴリー: 本体バグ
投稿日: 2007-11-28 (水) 02:30:07
バージョン: 1.4.7

メッセージ†

lib/convert_html.phpのfunction convert_html()で

	// Set digest
	$digest = md5(join('', get_source($vars['page'])));

とglobal変数の$digestの生成をしていますが、この値は他から使用されていません。

digest値の主な使用目的と思われるplugin/edit.inc.phpのplugin_edit_write()では、postされたvars['digest']を使用しているのみです。このvars['digest']は一発目のedit_form表示時に計算され、convert_html時の値は使用されていません。

ということで、convert_html()の

	// Set digest
	$digest = md5(join('', get_source($vars['page'])));

は削除して良いと思います。通常表示では、これが2度呼ばれているので、無駄な負荷を減らせます。
skinから呼ばれるdo_menu_convertによるmenubar表示時のconvert_htmlでも、menubarではなくそのとき表示しようとするページのdigestを計算しています。($vars['page']はmenubarではなく、表示するページが入っているため)

PukiWiki/1.4/ちょっと便利に/任意のページごとの閲覧・編集制限/未対応プラグイン対策実装時に発見。 -- ぃぉぃぉ 2007-11-28 (水) 02:34:12

次の2点も見直す必要があるのでは？ -- 2007-12-03 (月) 13:29:25
1. lib/plugin.php のdo_plugin_convert() とdo_plugin_inline() からglobal変数の$digest を保持する機構*1を外す。
2. plugin_article_convert() と、plugin_comment_convert() 、plugin_insert_convert() 、plugin_memo_convert() 、plugin_paint_convert() 、plugin_vote_convert() でglobal変数の$digest ではなく、自力で生成するように変更する*2。

total:
today:
yesterday:
now: 1

convert_html()で無駄なdigest生成をしている†

メッセージ†

最新の16件