* 閲覧権限のないMenuBarが表示される． [#m5af2f08]
- ページ: [[BugTrack2]]
- 投稿者: [[しましま]]
- 優先順位: 重要
- 状態: 提案
- カテゴリー: 本体バグ
- 投稿日: 2006-04-22 (土) 01:26:12
- バージョン: 1.4.6
#contents
-----
** メッセージ [#ed9b5d63]
閲覧権限のないページを見ると，対象となるページの閲覧は制限されますが，それに付随する MenuBar は閲覧権限がなくても表示されて，閲覧できてしまいます．
define('MENU_ENABLE_SUBMENU', TRUE);
とすることで，ディレクトリごとに MenuBar を変更して，ディレクトリごとに閲覧制限をかけて，権限によってメニューを変えています．
しかし，上記の問題があって，意図したことが実現できません．
menu.inc.phpプラグインを修正すれば対処できるのではと思い
--- menu.inc.php.orig 2006-04-20 22:23:58.000000000 +0900
+++ menu.inc.php 2006-04-22 00:35:19.000000000 +0900
@@ -47,6 +47,8 @@
if (! is_page($page)) {
return '';
+ } else if (!check_readable($page, false, false)) {
+ return '';
} else if ($vars['page'] == $page) {
return '<!-- #menu(): You already view ' . htmlspecialchars($page) . ' -->';
} else {
のように check_readable 関数を加えてみましたが，
Headers already sent at [サイトのルート]/lib/html.php line 496.
のエラーで止まってしまいました．
PHPも，PukiWikiの構造も知らないので，私の力量ではここでお手上げになってしまいました．
** 修正内容 [#w9cb270c]
認証中失敗時にはページを表示しないように cmd を変更する．(Thanks teanan さん)
** パッチ [#x205ea91]
lib/auth.php
--- auth.php.orig 2006-04-22 10:33:22.000000000 +0900
+++ auth.php 2006-04-22 12:51:40.000000000 +0900
@@ -168,7 +168,7 @@
// Basic authentication
function basic_auth($page, $auth_flag, $exit_flag, $auth_pages, $title_cannot)
{
- global $auth_method_type, $auth_users, $_msg_auth;
+ global $auth_method_type, $auth_users, $_msg_auth, $vars;
// Checked by:
$target_str = '';
@@ -216,6 +216,7 @@
$body = $title = str_replace('$1',
htmlspecialchars(strip_bracket($page)), $title_cannot);
$page = str_replace('$1', make_search($page), $title_cannot);
+ $vars['cmd'] = 'auth_error';
catbody($title, $page, $body);
exit;
}
*[[関連>PukiWiki/1.4/ちょっと便利に/任意のページごとの閲覧・編集制限/関連ページ]] [#l74bae8c]
#include(PukiWiki/1.4/ちょっと便利に/任意のページごとの閲覧・編集制限/関連ページ,notitle)
--------
*コメント [#qf4d86bd]
- MenuBarってskinから呼んでいる関係上、リダイレクトできないんですよねぇ…。一番手っ取り早いのは、認証する際にメニューを表示しないように小細工する方法ですが、例えばこんな感じでやってしまうのも手ですね。 -- [[teanan]] &new{2006-04-22 (土) 02:17:29};
lib/auth.php : 218行目付近
$page = str_replace('$1', make_search($page), $title_cannot);
+ global $vars;
+ $vars['cmd'] = 'error';
catbody($title, $page, $body);
exit;
-- skinで「arg_check("read")」がかかっている前提だとこれでいいのですが、デザイン上、この条件を取り除いてしまうこともあります。明示的にmenuプラグインの中で「arg_check("error")」を判断した方がいいのかもしれません。 -- [[teanan]] &new{2006-04-22 (土) 02:18:44};
--- teananさんの方針でうまくいきました．menuプラグインを以下のようにして，pukiwki.skin.php の arg_check('read') をはずしてみましたが，menu の改造前後で差が分かりませんでした．menu で check をしない問題点をお教えいただけますでしょうか？ -- [[しましま]] &new{2006-04-22 (土) 13:27:22};
--- menu.inc.php.orig 2006-04-22 13:34:27.000000000 +0900
+++ menu.inc.php 2006-04-22 13:29:32.000000000 +0900
@@ -16,6 +16,8 @@
global $vars, $menubar;
static $menu = NULL;
+ if (arg_check('auth_error')) return('');
+
$num = func_num_args();
if ($num > 0) {
// Try to change default 'MenuBar' page name (only)
--- teananさんが書いているのは、たとえば配布しているpukiwki.skin.php では単語検索や一覧表示時はメニューバーが出ませんが、人によっては常時出したいと考えて、以下のように改造している可能性があり、このとき結果が違ってくるという事を言っているのでは？
--- pukiwiki_orig.skin.php Fri Apr 21 00:25:32 2006
+++ pukiwiki.skin.php Sat Apr 22 22:03:51 2006
@@ -166,7 +166,7 @@
<?php echo $hr ?>
-<?php if (arg_check('read') && exist_plugin_convert('menu')) { ?>
+<?php if (exist_plugin_convert('menu')) { ?>
<table border="0" style="width:100%">
<tr>
<td class="menubar">
--- そのとおりです。フォローありがとうございます :) -- [[teanan]] &new{2006-04-23 (日) 02:52:18};
--- read 以外のlistやrecentのときにもmenuプラグインが呼ばれて表示されたりということですね．~
defaultスキンでarg_check('read')をはずして実験してみました．menu.inc.php 側にはパッチが特になくても，上記の auth.php だけのパッチだけでOKのようでした．~
どうもありがとうございました． -- [[しましま]] &new{2006-04-23 (日) 11:23:37};
#comment
![[PukiWiki]](image/pukiwiki-dev.png "[PukiWiki]")
