tdiary / tdiary-contrib

tDiaryの非正規リポジトリ。どなたでもcommit可能です。commit権希望の方、まずはパッチ等をpull requestして下さい。
https://tdiary.org/
68 stars 46 forks source link

4.0.2でsearch-default.rbが動かない #68

Closed kp1 closed 11 years ago

kp1 commented 11 years ago

一応以下のパッチでなおりましたが、よくわかっていません。

diff --git a/plugin/search-default.rb b/plugin/search-default.rb
index 018d949..021492b 100644
--- a/plugin/search-default.rb
+++ b/plugin/search-default.rb
@@ -99,7 +99,7 @@ module DefaultIOSearch
        def diary_class(style)
                c = DIARY_CLASS_CACHE[style]
                return c if c
-               require "tdiary/style/#{style.downcase}_style.rb"
+               require "tdiary/style/#{style.downcase}"
                c = TDiary.const_defined?('Style') ? eval("TDiary::Style::#{style.capitalize}Diary") : eval("TDiary::#{style.capitalize}Diary")
                c.__send__(:include, DiaryClassDelta)
                DIARY_CLASS_CACHE[style] = c
@@ -227,7 +227,7 @@ def search_input_form( q )
 end

 def search_result
-       unless @conf.io_class == TDiary::DefaultIO
+       unless @conf.io_class == TDiary::IO::Default
                return %Q|<p class="message">could not use this plugin under #{@conf.io_class}.</p>| 
        end
hsbt commented 11 years ago

報告ありがとうございます。添付していただいたパッチで直ると思います。contrib まで目がまわっていませんでした...

kp1 commented 11 years ago

37 ではsearch-default.rbをcoreに移動する話になってますね。

minimum2scp commented 11 years ago

後方互換性を考慮した修正をしてたのですがローカルで直して検証してるうちに忘れていました tdiary/tdiary-contrib#69 にパッチを投げていますが、後方互換性の考慮以外は @kp1 さんと同じです。 (でも確か古いtdiary-coreで検証してない…)

tdtds commented 11 years ago

contribではそんなに後方互換性を意識しなくてもいいとは思いますが(今回の相手は開発中ではない最新リリース版ですし)、 @minimum2scp さんのパッチはすばらしいですね。これをマージしておきます。

minimum2scp commented 11 years ago

ありがとうございますー!