BugTrack/332

commentプラグインのnodateオプションが無効となっている†

ページ: BugTrack
投稿者: にぶんのに
優先順位: 低
状態: 完了
カテゴリー: プラグイン
投稿日: 2003-05-17 (土) 06:13:15
バージョン:

メッセージ†

commentプラグインで挿入する文字列に日付を付加しないオプションnodateを指定しても日付が挿入されてしまう
sf.jp上の1.34および1.42で確認。orgではnodateが正しく働く。以下、修正案。

+	$now_format = ($post['nodate']=='1') ? '' : COMMENT_NOW_FORMAT ;
-	if ($post['nodate']=='1') {
-		$comment_format = str_replace('$now','',$comment_format);
-	}

	$postdata = '';
	$postdata_old  = get_source($post['refer']);
	$comment_no = 0;
	
	$name = '';
	if ($post['name'] != '') {
		$name = str_replace('$name',$post['name'],COMMENT_NAME_FORMAT);
	}
	
	$head = '';
	if (preg_match('/^(-{1,2})(.*)/',$post['msg'],$match)) {
		$head = $match[1];
		$post['msg'] = $match[2];
	}
	
	$comment = str_replace("\x08MSG\x08",str_replace('$msg',$post['msg'],COMMENT_MSG_FORMAT),$comment_format);
	$comment = str_replace("\x08NAME\x08",$name,$comment);
+	$comment = str_replace("\x08NOW\x08",str_replace('$now',$now,$now_format),$comment);
-	$comment = str_replace("\x08NOW\x08",str_replace('$now',$now,COMMENT_NOW_FORMAT),$comment);
	$comment = $head.$comment;

commentプラグインで上方／下方どちらにコメントを挿入するかを引数で指定可能とする拡張を加えようとして気付いたんですが、それまでnodateオプションの存在すら知らなかったです :p -- にぶんのに 2003-05-17 (土) 06:15:31
これは…BugTrack/277の修正の時に作ったバグですね。 XD -- ぱんだ 2003-05-17 (土) 10:58:14
cvsに投入しました。cvs:plugin/comment.inc.php (v1.4:r1.18) (v1.3.4:r1.9.2.3) -- ぱんだ 2003-05-17 (土) 10:58:14
げげっ…、1.17→1.18で激変しちゃってますねぇ…。ぼちぼち追いかけるのが難しくなってきた。正式版が出るまでに難民になりそう(^^;; -- reimy 2003-05-17 (土) 17:26:10
- とりあえず、ls2.inc.phpに続いて、comment.inc.phpのバージョンアップはあきらめた(;_;)。nodateオプションなんて使わないからいいか。 -- reimy 2003-05-17 (土) 17:28:48
そんなには変わってないと思います… {の位置が行末から次の行に移っているのは趣味ですが。 -- ぱんだ 2003-05-17 (土) 17:45:26
- diffファイルが150行を超えてるので、手作業ではかなり難しい…。nodateだけでなんでこんなに大きく変わっちゃったんだろ。 -- reimy 2003-05-17 (土) 23:49:49

total:
today:
yesterday:
now: 1

commentプラグインのnodateオプションが無効となっている†

メッセージ†

最新の16件