[Fswiki-dev] Re: HTMLParser.pm への機能 追加

アーカイブの一覧に戻る

Naoki Takezoe takez****@aa*****
2005年 8月 6日 (土) 11:57:33 JST


竹添です。

遅くなりましたが下記の修正をコミットしました。
ご報告ありがとうございました。

あき wrote:
> あきです。
> 
> デフォルトの書式選択リストボックスについてですが、
> 他のプラグインのテストをしていて不具合があることに気が付きました。
> lib/Wiki.pmのget_format_namesにif文を一つ追加する必要があります。
> 
> ----------------------------------------------------------------
> sub get_format_names {
> 	my $self = shift;
> 	my @list = keys(%{$self->{'format'}});
> 	if(!scalar(@list)){                       <---この行を追加
> 		push(@list, "FSWiki");                  <---この行を追加
> 	}                                         <---この行を追加
> 	return sort(@list);
> }
> ----------------------------------------------------------------
> 
> これが無いと、formatプラグインOFFでかつクッキーにも保存されていない場合、
> 環境設定画面でリストが空になってしまいます。
> この辺りは確認したつもりでしたが、クッキーにフォーマットが保存された状態
> で確認していたか何かで、不備に気が付きませんでした。
> 申し訳ございません。
> 
> 
> 
>>>竹添です。
>>>
>>>AdminConfigHandler.pmでフォーマットの一覧を取得しているところは
>>>$wiki->get_format_names()で取得するべきですね。
>>
>>失礼致しました。
>>そのとおりですね。
>>FormatSelector.pmで使っていたのに(存在は知っていたのに)、
>>気が付きませんでした。
>>
>>その部分だけですが修正しました。
>>
>>
>>>機能的にはよいと思います。
>>>
>>>
>>>>インターフェイスを見直しました。
>>>>
>>>>ソースは美しくないかもしれませんが、機能的にはこんな感じでよろしいでしょ
>>>>うか?
>>>>
>>>>ファイル構成
>>>>3.5.8のソースを元に、変更が加わったファイルだけを下記に記載します。
>>>>
>>>> lib/
>>>>  Wiki.pm
>>>> plugin/
>>>>  admin/
>>>>    AdminConfigHandler.pm
>>>> tmpl/
>>>>  admin_config.tmpl
>>>>
>>>>不備がございましたらご指摘下さい。
>>>>
>>>>_______________________________________________
>>>>Fswiki-dev mailing list
>>>>Fswik****@lists*****
>>>>http://lists.sourceforge.jp/mailman/listinfo/fswiki-dev
> 
> 
> _______________________________________________
> Fswiki-dev mailing list
> Fswik****@lists*****
> http://lists.sourceforge.jp/mailman/listinfo/fswiki-dev
> 
> 

-- 
Naoki Takezoe <takez****@aa*****>



Fswiki-dev メーリングリストの案内
アーカイブの一覧に戻る