get_base_uri() を使わずに、global $scriptを参照しているファイルが残っている

  • 元タイトル: update_entities.inc.phpでglobal $scriptを参照している。
  • ページ: BugTrack
  • 投稿者: g@kko
  • 優先順位: 低
  • 状態: 完了
  • カテゴリー: プラグイン
  • 投稿日: 2007-07-18 (水) 12:38:31
  • バージョン: 1.4.7 (update_entities.inc.php,v 1.15)
  • リリース予定バージョン: 1.5.2

メッセージ

BugTrack2/213によるとget_script_uri() を用いるべきとなっています。

38行目より

function plugin_update_entities_action()
{
-	global $script, $vars;
+	global $vars;
	global $_entities_messages;

	if (PKWK_READONLY) die_message('PKWK_READONLY prohibits this');

+	$script = get_script_uri();
	$msg = $body = '';
	if (empty($vars['action']) || empty($vars['adminpass']) || ! pkwk_login($vars['adminpass'])) {
		$msg   = & $_entities_messages['title_update'];

  • vote.inc.php,v 1.24も同様 -- 2007-07-18 (水) 12:48:57
    • 他にもたくさん -- 2007-07-18 (水) 12:49:12
  • 何か大きな見直しを行う時に修正を実施。もしくは、他の修正に合せて個別に対応して行く。で良いと思いますが、いかがですか? -- 2007-07-27 (金) 08:38:47
  • 2007-10-27 (土) 08:00:00 時点のCVS版ファイルから、global で$script を呼んでいる場所を検索した結果を貼ります。
    lib/auth.php(125):	global $script, $_title_cannotedit, $_msg_unfreeze;
    
    lib/func.php(226):	global $script, $whatsnew, $non_list, $search_non_list;
    
    lib/html.php(14):	global $script, $vars, $arg, $defaultpage, $whatsnew, $help_page, $hr;
    lib/html.php(158):	global $script, $vars, $rows, $cols, $hr, $function_freeze;
    lib/html.php(273):	global $script, $vars, $rule_related_str, $related_str;
    lib/html.php(361):	global $script;
    
    lib/make_link.php(514):	global $script;
    lib/make_link.php(777):	global $script, $vars, $link_compact, $related, $_symbol_noexists;
    
    plugin/amazon.inc.php(116):	global $script, $vars, $asin, $asin_all;
    plugin/amazon.inc.php(191):	global $vars, $script, $edit_auth, $edit_auth_users;
    
    plugin/article.inc.php(48):	global $script, $post, $vars, $cols, $rows, $now;
    plugin/article.inc.php(145):	global $script, $vars, $digest;
    
    plugin/attach.inc.php(389):	global $script, $vars, $_attach_messages;
    plugin/attach.inc.php(503):	global $script, $_attach_messages;
    plugin/attach.inc.php(526):	global $script, $_attach_messages;
    
    plugin/back.inc.php(20):	global $_msg_back_word, $script;
    
    plugin/bugtrack.inc.php(256):	global $script, $vars, $_plugin_bugtrack;
    
    plugin/calendar.inc.php(12):	global $script, $weeklabels, $vars, $command;
    
    plugin/calendar2.inc.php(12):	global $script, $vars, $post, $get, $weeklabels, $WikiName, $BracketName;
    
    plugin/calendar_viewer.inc.php(54):	global $vars, $get, $post, $script, $weeklabels;
    plugin/calendar_viewer.inc.php(289):	global $vars, $get, $post, $script;
    
    plugin/comment.inc.php(23):	global $script, $vars, $now, $_title_updated, $_no_name;
    
    plugin/diff.inc.php(28):	global $script, $hr;
    plugin/diff.inc.php(75):	global $script, $vars;
    
    plugin/dump.inc.php(213):	global $script, $defaultpage;
    
    plugin/edit.inc.php(85):	global $script, $vars, $fixed_heading_anchor_edit;
    
    plugin/freeze.inc.php(14):	global $script, $vars, $function_freeze;
    
    plugin/include.inc.php(55):	global $script, $vars, $get, $post, $menubar, $_msg_include_restrict;
    
    plugin/includesubmenu.inc.php(10):  global $script,$vars;
    
    plugin/insert.inc.php(12):	global $script, $vars, $cols, $rows;
    plugin/insert.inc.php(71):	global $script, $vars, $digest;
    
    plugin/links.inc.php(37):	global $script, $post, $vars, $foot_explain;
    
    plugin/memo.inc.php(11):	global $script, $vars, $cols, $rows;
    plugin/memo.inc.php(71):	global $script, $vars, $digest;
    
    plugin/navi.inc.php(49):	global $vars, $script, $head_tags;
    
    plugin/newpage.inc.php(8):	global $script, $vars, $_btn_edit, $_msg_newpage, $BracketName;
    
    plugin/paint.inc.php(41):	global $script, $vars, $pkwk_dtd, $_paint_messages;
    plugin/paint.inc.php(155):	global $script,$vars,$digest;
    plugin/paint.inc.php(211):	global $script,$vars,$now,$do_backup;
    
    plugin/random.inc.php(23):	global $script, $vars;
    
    plugin/ref.inc.php(110):	global $script, $vars;
    
    plugin/related.inc.php(19):	global $vars, $script, $defaultpage;
    
    plugin/rename.inc.php(93):	global $script, $_rename_messages;
    plugin/rename.inc.php(135):	global $script, $_rename_messages;
    plugin/rename.inc.php(223):	global $script, $_rename_messages;
    
    plugin/search.inc.php(65):	global $script, $_btn_and, $_btn_or, $_btn_search;
    
    plugin/template.inc.php(10):	global $script, $vars;
    
    plugin/topicpath.inc.php(31):	global $script, $vars, $defaultpage;
    
    plugin/unfreeze.inc.php(14):	global $script, $vars, $function_freeze;
    
    plugin/update_entities.inc.php(38):	global $script, $vars;
    
    plugin/vote.inc.php(11):	global $vars, $script, $cols,$rows;
    plugin/vote.inc.php(79):	global $script, $vars,  $digest;
    Copyright の件や、主要プラグインが認証に対応していない件と一緒に解決すると、大半は終了するでしょう。
    後は、html.php ならBugTrack2/146、make_link.php ならBugTrack2/7、back.inc.php ならBugTrack2/258BugTrack2/263、bugtrack.inc.php ならBugTrack2/187、update_entities.inc.php ならBugTrack2/252、とか*1
    他に未解決の件ってあったかな? -- 2007-10-27 (土) 15:00:59
  • 関連: BugTrack/481 -- 2007-11-06 (火) 22:40:19
  • edit.inc.php について追加。 -- henoheno 2007-11-11 (日) 22:49:52
  • BugTrack/2213の対応の中で解決しました。グローバル変数$scriptを使っていた箇所はすべてget_base_uri()呼び出しに置き換えています commit:d4fd351122 -- umorigu 2017-09-04 (月) 03:14:57


*1 採用するかは別として、適当にまとめてみた。

トップ   編集 凍結 差分 バックアップ 添付 複製 名前変更 リロード   新規 一覧 検索 最終更新   ヘルプ   最終更新のRSS
Last-modified: 2017-09-04 (月) 21:48:24
Site admin: PukiWiki Development Team

PukiWiki 1.5.2+ © 2001-2019 PukiWiki Development Team. Powered by PHP 5.6.40-0+deb8u7. HTML convert time: 0.253 sec.

OSDN