![[PukiWiki]](image/pukiwiki-dev.png "[PukiWiki]")
次の部分の処理が無駄だと思うのですが、いかがでしょう。
// $Id: tracker.inc.php,v 1.36 2007/03/29 15:05:52 henoheno Exp $
（中略）
class Tracker_list
{
（中略）
function add($page,$name)
{
（中略）
$source = plugin_tracker_get_source($page);
if (preg_match('/move\sto\s(.+)/',$source[0],$matches))
{
（中略）
return $this->add($page,$name);
}
$source = join('',preg_replace('/^(\*{1,3}.*)\[#[A-Za-z][\w-]+\](.*)$/','$1$2',$source)); //無駄だと思う部分
（中略）
function plugin_tracker_get_source($page)
{
$source = get_source($page);
// 見出しの固有ID部を削除
$source = preg_replace('/^(\*{1,3}.*)\[#[A-Za-z][\w-]+\](.*)$/m','$1$2',$source);
// #freezeを削除
return preg_replace('/^#freeze\s*$/im', '', $source);
}
plugin_tracker_get_source の途中、preg_replace でアンカー名を消す置き換えをしているのに、
plugin_tracker_get_source で呼び出したソースに、もう一度アンカー名を消す置き換えをしようとしています。
この部分は、
- $source = join('',preg_replace('/^(\*{1,3}.*)\[#[A-Za-z][\w-]+\](.*)$/','$1$2',$source));
+ $source = join('',$source);
と修正したほうが、よいと思います。
// $Id: tracker.inc.php,v 1.36 2007/03/29 15:05:52 henoheno Exp $
// Copyright (C) 2003-2005 PukiWiki Developers Team
（中略）
class Tracker_list
{
（中略）
function add($page,$name)
{
（中略）
if ($this->rows[$name]['_match'] = preg_match("/{$this->pattern}/s",$source,$matches))
{
+ unset($source);
array_shift($matches);
foreach ($this->pattern_fields as $key=>$field)
{
$this->rows[$name][$field] = trim($matches[$key]);
}
}
}
（中略）
function toString($limit=NULL)
{
（中略）
foreach ($this->rows as $key=>$row)
{
if (!TRACKER_LIST_SHOW_ERROR_PAGE and !$row['_match'])
{
+ unset($this->rows[$key]);
continue;
}
$this->items = $row;
foreach ($body as $line)
{
if (trim($line) == '')
{
$source .= $line;
continue;
}
$this->pipe = ($line{0} == '|' or $line{0} == ':');
$source .= preg_replace_callback('/\[([^\[\]]+)\]/',array(&$this,'replace_item'),$line);
}
+ unset($this->rows[$key]);
}
return convert_html($source);
}
1つ目のTracker_list->add() でのunset は、「1ページ分のソース×2」を解消するためなんですが、これはページのデータ量しだいで効果が変わってしまいそうです*1。// $Id: tracker.inc.php,v 1.36 2007/03/29 15:05:52 henoheno Exp $
（中略）
class Tracker_list
{
（中略）
function Tracker_list($page,$refer,&$config,$list)
{
（中略）
while (count($pattern))
{
$this->pattern .= preg_replace('/\s+/','\\s*','(?>\\s*'.trim(array_shift($pattern)).'\\s*)');
if (count($pattern))
{
$field = array_shift($pattern);
$this->pattern_fields[] = $field;
$this->pattern .= '(.*)';
}
}
+ $this->pattern_fields = array_intersect($this->pattern_fields, array_keys($this->fields));
// ページの列挙と取り込み
$this->rows = array();
少しでもデータが減った分、後の処理が軽くならないかな～。 --2007-06-10 (日) 14:40:03$this->pattern = '(?>\s*\*\s*)(.*)(?>\s*-ページ\:\s*)(.*)(?>\s*-投稿者\:\s*)(.*)(?>\s*-優先順位\:\s*)(.*)'. '(?>\s*-状態\:\s*)(.*)(?>\s*-カテゴリー\:\s*)(.*)(?>\s*-投稿日\:\s*)(.*)(?>\s*-バージョン\:\s*)(.*)'. '(?>\s*\*\*\s*メッセージ\s*)(.*)(?>\s*----\s*)(.*)(?>\s*\s*)' $this->pattern_fields = Array ( [0] => Summary [1] => _refer [2] => Proposer [3] => Severity [4] => Status [5] => Category [6] => _date [7] => Version [8] => Messages [9] => _block_comment ) 見にくくなるので適当に加工してます。です。
上の例だと、実際に$this->rows に入るのは各ページ10項目ですが、
_block_comment をリストの表示につかわない*4であろう、ということと、
同じプラグインが複数回使われたとしても、今の置き換えルールではその判別ができない（同じフィールド名になってしまう）という理由から、
設定されていない項目のデータを得なくてもかまわないだろう、という判断でこの修正案を挙げました。 -- 2007-06-11 (月) 11:33:43
class Tracker_list
{
（中略）
function sort($order)
{
if ($order == '')
{
return;
}
$names = array_flip(array_keys($this->fields));
$this->order = array();//これ
foreach (explode(';',$order) as $item)
{
初期化処理をTracker_list->Tracker_list() に移した時に、見逃したのだと思います。 -- 2007-06-11 (月) 19:33:51
例 全50件中、上位-20件を表示しています。 array_splice()は負の値にも対応しているので、リストには、(50 - 20) つまり30件が表示される表示上限としているので、負の値とした時に無効とするか、メッセージの側を対応させる必要があると思います。
（中略）
class Tracker_list
{
（中略）
function toString($limit=NULL)
{
global $_tracker_messages;
$source = array();
$body = array();
$count = count($this->rows);
if ($limit !== NULL && $count > $limit)
{
$source[] = str_replace(
array('$1', '$2'),
array($count,$limit),
$_tracker_messages['msg_limit']) . "\n";
$this->rows = array_splice($this->rows,0,$limit);
}
if (empty($this->rows))
{
return '';
}
（中略）
foreach ($this->rows as $row)
{
if (! TRACKER_LIST_SHOW_ERROR_PAGE && ! $row['_match'])
{
continue;
}
（以降、略）
表示上限が0 のときは、if (empty($this->rows)) が拾ってくれるので大丈夫だと思います。全50件中、上位20件を表示しています。となっていても、実際にはリストが20件でない可能性があります。 -- 2007-06-11 (月) 22:03:24
[[tracker/1]], [[tracker/2]], ... という形でtrackerが展開されていたとき、 [[tracker/3]] の中身が 'move to [[foobar]]' だったならば、tracker_list は [[tracker/3]] の代わりに [[foobar]] を表示する。しかし、[[tracker/3]] の 中身が例えば 'move to [[tracker/4]]' だった時、tracker_list は [[tracker/4]] を二回表示してしまう。 (移動先が tracker/ 以下でなければこのような挙動は起きない)
もし今、$pagename を採用して'_real'だけでソートしたら...（並び順のイメージ） 質問箱4/1 質問箱4/10 質問箱4/100 質問箱4/101 質問箱4/102 質問箱4/103 質問箱4/104 質問箱4/105 質問箱4/106 質問箱4/107 質問箱4/108 質問箱4/109 質問箱4/11 質問箱4/110
例えば、defaultの[Status]で、提案のスタイルを適用すると、BGCOLOR(#ffccff):データとなる。
単純に%s を各データに置き換えるだけなので、
表形式以外でこの機能を使うには、「&color(,#ffccff){%s};」のように設定する必要がある。例えば、textarea のデータを表形式に対応させるため改行を削除する（表形式以外でもこの処理は固定ですけど） 例えば、[_update]等の日時データを人が読める形式にする など...
例えば、defaultの[Status]は、提案は0番、着手は1番、...、却下は5番、これ以外は値そのまま、を数値的比較でソートする。 ただ、その他の値は、意図した順番（最初か最後）にならない可能性があります。
*test&br;TEST
:test&br;TEST|test&br;TEST
| test TEST | test TEST |
|test&br;TEST|test&br;TEST|
| test TEST | test TEST |
,test&br;TEST,test&br;TEST
-test&br;TEST
test
TEST
&size(22){test&br;TEST};
| Site admin: PukiWiki Development Team PukiWiki 1.5.4+ © 2001-2022 PukiWiki Development Team. Powered by PHP 8.2.12. HTML convert time: 0.215 sec. |
|