BugTrack/2260

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

• ChangeLog より、cvs:plugin/tracker.inc.php (1.42) -- 2007-09-04 23:09
• ChangeLog より、cvs:plugin/ls2.inc.php (1.26) -- 2007-10-25 23:51

• 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/258やBugTrack2/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
  • cvs:plugin/edit.inc.php (1.48)
• BugTrack/2213の対応の中で解決しました。グローバル変数$scriptを使っていた箇所はすべてget_base_uri()呼び出しに置き換えています commit:d4fd351122 -- umorigu 2017-09-04 (月) 03:14:57

• BugTrack
• 開発版
  • リポジトリ
  • 開発日記
• :CategoryDev
  • PukiWiki2
  • PukiWiki/1.5
  • PukiWiki/1.4
  • PukiWiki/1.3
  • メーリングリスト
  • 開発談義
  • 雑談
• 開発者向け情報
  • Books
  • プラグイン

最新の16件