Namazu-devel-ja(旧)


[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: filename_to_title Re: filename検索他修正



寺西です。

Yukio USUDA wrote:
> 
> > > complete_field_info後の
> > >  $content .= " $filename ";
> > > を
> > >  $content .= " $fields{'$filename'} ";
> > > にすればよいのでは
> >
> > その時点では $fields{'filename'} は未定義にされている可能性が
> > あるのです。
> 
> では、
> $content .= " $fields{'$filename'} ";
> を
> complete_field_info
> と
> clean_field_index
> の間に移してはどうでしょうか?

$Indexer->init の直前に $content につっこみたいということから、
その位置でつっこんでいます。
同様の意図で、
$content .= $weighted_str;
が $Indexer->init の直前にあります。
$content .= $weighted_str; の前でも良いので、最終的には
$content .= $weighted_str; の直前でつっこむようにしています。

なぜその位置で、つっこみたいのかというと、将来 $content が
clean_field_index 以降で書き換えたりするような修正が入った
場合のことを想定しています。

> > 多段フィルタの時の動作とか、html_filter でセットするのはちょっと
> > 特別な意味があるかもしれない(かな)と思って、手を出していません。
> 
> フィルタからfilename_to_tiltleを呼び出していたものは以下のとおりです。
...
> 心配されているタイプのフィルタでは使用していません
> 全部消しても大丈夫でしょう。

filename_to_title だけではなくて、$field{'title'} を設定する優先順位
とか、$field{'title'} と $field{'subject'} との関係も見ないことには、
大丈夫とは言えないわけでして。そこをきちんと調べてないので、
手を出していないのです。

# ま、大丈夫でしょうけど。何せ未チェックなもので。
-- 
=====================================================================
寺西 忠勝(TADAMASA TERANISHI)  yw3t-trns@xxxxxxxxxxxxxxx
http://www.asahi-net.or.jp/~yw3t-trns/index.htm
Key fingerprint =  474E 4D93 8E97 11F6 662D  8A42 17F5 52F4 10E7 D14E