BugTrack/2198

plugin が空ページを返すときの $defaultpage の表示†

ページ: BugTrack
投稿者: しましま
優先順位: 重要
状態: 完了
カテゴリー: 本体バグ
投稿日: 2006-09-02 (土) 12:02:40
バージョン: 1.4.7
リリース予定バージョン: 1.5.2

メッセージ†

list, backup, topicpath, attach などのプラグインでリスト表示する内容がないなどの理由で body 部分に '' を返す場合があります．

このとき，$defaultpage が表示される仕様になっているとおもいます．このとき，閲覧者に $defaultpage に閲覧権限があるかどうかチェックされることなく，表示されてしまいます．

プラグイン側で空を返さないようにする方法は，私程度でも対処できそうでした．list の例だと

--- list.inc.php.orig	2006-09-02 11:49:20.000000000 +0900
+++ list.inc.php	2006-08-15 20:04:16.000000000 +0900
@@ -19,12 +19,12 @@
// Get a list
function plugin_list_getlist($withfilename = FALSE)
{
-	global $non_list, $whatsnew;
+	global $non_list, $whatsnew, $_err_notexist;

	$pages = array_diff(get_existpages(), array($whatsnew));
	if (! $withfilename)
		$pages = array_diff($pages, preg_grep('/' . $non_list . '/S', $pages));
-	if (empty($pages)) return '';
+	if (empty($pages)) return $_err_notexist;

	return page_list($pages, 'read', $withfilename);
}

※ $_err_notexist は，現在は yetlist 専用なので '該当項目はありません' や '表示すべき内容はありません' とかに変える必要があると思います．

しかし，該当するプラグインは多く，やはり本体側で対処すべき問題ではないかと思います．対処法は

プラグインから空が返されたときでも，$defaultpage の表示前に閲覧権限をチェックする
プラグインから空が返されたは，$defaultpage ではなく '表示すべき内容はありません' といったメッセージを表示する

すみませんが，私では，プラグインから帰ってきたときに $defaultpage を表示している部分を見つけられずパッチを作れませんでした．

恐れりますが，よろしくお願いします．

関連：BugTrack/366 -- yetlist が空を返すときの対処

どうにか，自力でパッチを作成できました． -- しましま 2007-02-03 (土) 09:42:05

lib/pukiwki.ini で返された $body が null で，$base も null のときは，閲覧権限のチェックなしで $defaultpage が表示されているようです．そこで，check_readable をその前に呼び出して，閲覧権限をチェックするようにしました．

--- pukiwiki.php.orig	2007-02-03 01:02:33.000000000 +0900
+++ pukiwiki.php	2007-02-03 01:34:44.000000000 +0900
@@ -112,6 +112,7 @@
	$body = & $retvars['body'];
} else {
	if ($base == '' || ! is_page($base)) {
+		check_readable($defaultpage, true, true);
		$base  = & $defaultpage;
		$title = htmlspecialchars(strip_bracket($base));
		$page  = make_search($base);

pukiwki-1.4.7 用です．よろしければ，採用していただければと思います．

パッチありがとうございます。採用しました commit:17a96ad228 -- umorigu 2018-12-31 (月) 15:02:29

total:
today:
yesterday:
now: 1

plugin が空ページを返すときの $defaultpage の表示†

メッセージ†

最新の16件