tDiary

ただただしです。

2012年2月20日17:21 Kazuhiko <kazuhiko@...>:
> これって要するに、styleの置き場所を変更した後の「tdiary/style/以下と
> tdiary/以下の両方をスタイルの検索パスにする」という先日の修正では、非互
> 換対策として十分では無かった、ということですね。私もこの可能性を見落とし
> ていました。

Blogkitの実装手法をみんなもう忘れてるってことだよねぇ。
まぁ、しょうがない。

> tdiary-contrib/style/をみた感じだと、同様の修正がいるファイルはありませ
> んでしたが、自作のスタイルで同様に動かなくなるケースがありえそうです。
>
> # あとは、tdiary-contrib/plugin/select_style.rb も変更がいりそう。
>
> 個人的には、この仕様変更は（プロジェクトのレガシー度を鑑みると）ちょっと
> 筋が悪い気がしますが、どうでしょうか。

うん、まぁそうは思ったんだけど、スタイル作ってる人は非常に少ないし、
非互換には違いないけど影響は軽微と判断して上記修正にしました。

いっぽう、(今朝Twitter上で)@hsbtとBlogkitのテストしづらさの話をしてい
て、だったらいっそ、今のsp.pathみたいにstyleも指定した複数ディレクト
リを探索できるようにすればいいんじゃね? と。

つまり今回の修正は過渡的なものになりそうな気がしてます。

-- 
ただただし - ツッコミは、短く鋭く愛を込めて♪
http://tdtds.jp/
@tdtds

>> tdiary-contrib/style/をみた感じだと、同様の修正がいるファイルはありませ
>> んでしたが、自作のスタイルで同様に動かなくなるケースがありえそうです。
>>
>> # あとは、tdiary-contrib/plugin/select_style.rb も変更がいりそう。

直しておきました:)

> いっぽう、(今朝Twitter上で)@hsbtとBlogkitのテストしづらさの話をしてい
> て、だったらいっそ、今のsp.pathみたいにstyleも指定した複数ディレクト
> リを探索できるようにすればいいんじゃね? と。

style ならテストを書けないこともないのですが blogkit をテストするためには
本体の tdiary_style.rb が無いと上手く書けないんですよねえ。
その他のプラグインも含めて core を submodule で指定して中に入れた上で
blogkit ようのテストを書くようにしても良いかもしれません。

-- 
SHIBATA Hiroshi shibata.hiroshi@...
http://www.hsbt.org/

須藤です。

In <CAO7msPKiyYSscUKv+0O+GjwKKVSrqQdLvxraO2CuCdMZ-m_o7g@...>
  "Re: [tDiary-devel] tdiary/tdiary-blogkit [master] fixed error on loading styles caused by style directory changed." on Mon, 20 Feb 2012 17:52:39 +0900,
  TADA Tadashi <t@...> wrote:

> いっぽう、(今朝Twitter上で)@hsbtとBlogkitのテストしづらさの話をしてい
> て、だったらいっそ、今のsp.pathみたいにstyleも指定した複数ディレクト
> リを探索できるようにすればいいんじゃね? と。

こんなこともあろうかと2008年にパッチを書いていました。

  http://sourceforge.net/mailarchive/forum.php?thread_name=20080521.200304.748728741204897704.kou%40cozmixng.org&forum_name=tdiary-devel

ただただしです。

2012年2月20日17:58 SHIBATA Hiroshi <shibata.hiroshi@...>:
> style ならテストを書けないこともないのですが blogkit をテストするためには
> 本体の tdiary_style.rb が無いと上手く書けないんですよねえ。
> その他のプラグインも含めて core を submodule で指定して中に入れた上で
> blogkit ようのテストを書くようにしても良いかもしれません。

いや、たとえばcoreとblogkitを別々のディレクトリにcloneしたとして、
tdiary.confでこんな感じで指定できれば解決じゃね? という話:

@options['style.path'] = [
   'tdiary/style',
   'tdiary',
   '../blogkit/tdiary'
]

……とここまで書いてたら須藤くんからメールが来てたけど、そう、その
パッチのことを意識していました:-) 「テストのしづらさを解消する」とい
う新しい要件がでてきたので再検討の俎上にあげます。

-- 
ただただし - ツッコミは、短く鋭く愛を込めて♪
http://tdtds.jp/
@tdtds

> いや、たとえばcoreとblogkitを別々のディレクトリにcloneしたとして、
> tdiary.confでこんな感じで指定できれば解決じゃね? という話:
>
> @options['style.path'] = [
>   'tdiary/style',
>   'tdiary',
>   '../blogkit/tdiary'
> ]

残念ながら、CategrizebleDiary やらなにやらを include する関係上、本体の tdiary.rb を
テストコードで require しないと style のテストは実行できないのでした。
mock, stub すれば良いって話なんですが、それはそれで中々大変という。

> ……とここまで書いてたら須藤くんからメールが来てたけど、そう、その
> パッチのことを意識していました:-) 「テストのしづらさを解消する」とい
> う新しい要件がでてきたので再検討の俎上にあげます。

とりあえず、須藤さんのパッチを当てて動作確認してみます。