tDiary

柴田です。

毎度報告ありがとうございます。

> まず、これはtypoな気がします。

これはひどい。

> FileUtils.mkdir_pするかどこかで@cache_path/delicious/をmkdir
> するような処理をいれないといけないと思います。

mkdir_p というもの自体を知らなかったのですが、そちらを使う方が
楽そうなので置き換えました。

> ついでに確認したいのですが、add_delicious_jsonの中でエラーを
> 握りつぶしているのは意図的なのでしょうか？エラーメッセージが
> 見えないのでどこが悪いかを調べるのがとても大変なのですが。。。
>
> ログに出力するとかできないものでしょうか？

作った時はあまり深く考えていないと思います…。
@conf.debugを使ってログを出すようにしました。

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

柴田です。

> 日記の中でflickrプラグインを使うとエラーになります。
> 既存の日記には影響はありません(キャッシュを消すとダメですが)。

私も昨夜から色々調べていたのですが、APIの変更が原因とは思いませんでした。
パッチを当てておこうと思います。

なにやら open-uri の中でリダイレクトすると Insecure Operation になるのが
問題のようです。後でまとめてどこかに報告しておきます。

> 少し厄介なのはキャッシュです。
> パッチ適用前に一度でも日記を書いてプレビューするだけで
> flickr.rbは空のキャッシュファイルを作ります
> パッチ適用後も、キャッシュを見に行くのでエラーのままです。
> flickrのキャッシュは全部削除してしまうか、
> 空ファイルに限って消す必要があります。

こちらはやっかいですね。
設定画面からキャッシュクリアできるようにした方がよさそうです。

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

まちゅです。

>> 日記の中でflickrプラグインを使うとエラーになります。
>> 既存の日記には影響はありません(キャッシュを消すとダメですが)。

パッチありがとうございます。
この問題、全然気がついていませんでした…。

>> 少し厄介なのはキャッシュです。
>> パッチ適用前に一度でも日記を書いてプレビューするだけで
>> flickr.rbは空のキャッシュファイルを作ります
>> パッチ適用後も、キャッシュを見に行くのでエラーのままです。
>> flickrのキャッシュは全部削除してしまうか、
>> 空ファイルに限って消す必要があります。

> こちらはやっかいですね。
> 設定画面からキャッシュクリアできるようにした方がよさそうです。

Flickrプラグインの設定画面から
キャッシュを削除できるようにしました。
念のため、過去のキャッシュは
拡張子「.org」を付けて残すようにしています。

-- 
MATSUOKA Kohei < http://www.machu.jp/diary/ >

ただただしです。

2009/5/17 SHIBATA Hiroshi <shibata.hiroshi@...>:
> なにやら open-uri の中でリダイレクトすると Insecure Operation になるのが
> 問題のようです。後でまとめてどこかに報告しておきます。

この話題は数年前に一度あったようですが、立ち消えになってるっぽい
です。リダイレクト先のURLを信頼すべきかどうかという仕様上の問題
なので、ちゃんとパッチ書いて提案しないと難しそうだなぁ。

>> 少し厄介なのはキャッシュです。
>> パッチ適用前に一度でも日記を書いてプレビューするだけで
>> flickr.rbは空のキャッシュファイルを作ります
>> パッチ適用後も、キャッシュを見に行くのでエラーのままです。
>> flickrのキャッシュは全部削除してしまうか、
>> 空ファイルに限って消す必要があります。
>
> こちらはやっかいですね。
> 設定画面からキャッシュクリアできるようにした方がよさそうです。

キャッシュの読み込みに失敗したらAPIを呼んで再取得すれば
いいだけのような? というか、API呼び出しが失敗したらキャッ
シュを保存しないようにするわけにいかないのかな?

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

kayakayaです。

パッチの取り込みと、
キャッシュの削除機能追加ありがとうございました。

2009/05/17 15:26 TADA Tadashi <t@...>:
>> なにやら open-uri の中でリダイレクトすると Insecure Operation になるのが
>> 問題のようです。後でまとめてどこかに報告しておきます。
>
> この話題は数年前に一度あったようですが、立ち消えになってるっぽい
> です。リダイレクト先のURLを信頼すべきかどうかという仕様上の問題
> なので、ちゃんとパッチ書いて提案しないと難しそうだなぁ。

*tDiary: ニコ動プラグインが動いてない?
<http://sho.tdiary.net/20090305.html>

と同じ文脈ですよね、
と理解しましたが、間違ってたら指摘してください。

> キャッシュの読み込みに失敗したらAPIを呼んで再取得すれば
> いいだけのような? というか、API呼び出しが失敗したらキャッ
> シュを保存しないようにするわけにいかないのかな?

API呼出に失敗した場合のケースの仕訳が十分か確認させてください。

(1) open-uriでのエラー

これは有無は言わせずキャッシュは作らない

(2) API呼び出しの仕様としては正しいが期待した結果と異なる

     例えば、API Keyが異っている場合も考えられます。
    この場合では、このような書式のxmlが返ります。

<?xml version="1.0" encoding="utf-8" ?>
<rsp stat="fail">
	<err code="100" msg="Invalid API Key (Key not found)" />
</rsp>

仕様書を見る限りstat属性を判断すれば実用的に判断できそうです。
stat属性が"fail"ならキャッシュを作らないようにすれば良いかと。

このようにRubyのエラーの場合と、Flickr API上のエラーがあります。
この2つを押えていれば大丈夫かなと思っていますが
漏れているケースや意見ありましたらお願いします。

ちなみに、この方針でできるか少し書いてみているところです。
うまくいって、参考になると良いのですが……。

-- 
KAYA Satoshi
http://kayakaya.net/d/

kayakayaです。

Flickrプラグインで、APIの呼び出しに失敗した場合はキャッシュを作成しないパッチを作成してみました。

API Keyが間違っている場合と画像IDが異なる場合にもキャッシュに保存されましたが、このパッチではキャッシュが作成されません。

キャッシュファイルをopenしてからAPIをopenしているところを、APIをopenして中身を確認した後にキャッシュファイルをopenするように変更しています。処理順序を変えているので、多少怖いところはありますが、Flickrな方はテストしてやってダメだししてください。

-- 
KAYA Satoshi
http://kayakaya.net/d/

こんにちは、はしもとです。

TADA Tadashi さんは書きました:
> 2009/4/27 橋本敬介 <ksk@...>:
>> ■ プラグインのnavi_userとcounterを同時に使うとエラーが出ました。
>>
>> Plugin error in '50sp.rb'.
>> Plugin error in '/path/to/misc/plugin/counter.rb'.
>> can't convert nil into String
>> (plugin/50sp.rb):132:in `load_plugin'
> 
> これが再現できないのですが(設定画面で別のエラーが出たけどたぶん
> 違う)、もう少し再現条件を絞っていただけますか? どういうシーンで
> 発生しますか?

前回のメールを書いた時と同じ手順でもう一度 tDiary2.2.1、Ruby1.8.5
のデータをコピーしてやり直してみたのですが…今回はエラーが出ません
でした。
コピー元のデータは、当時より少し変わっています(3日分ほど日記を書い
たので)。

ということで、済みません、この件は問題無しということにしてください。
お手数をおかけしてしまって、申し訳ございませんでした…。

なお、「navi_userとcounterを同時に使うとエラー」と判断した理由は、
counterについて過去ログを検索したところ、これらのプラグインの問題
と修正の報告があったので、試しにnavi_userを外してみました。
するとcounterのエラーが出なくなり、再度両方を使用するとエラーに
なったためでした。


> ITSにチケットを切ってあるので、そちらに書いていただいてもけっこ
> うです:
> 
>    http://www.cozmixng.org/retro/projects/tdiary/tickets/148

こちらにも同じ内容を書いておきますね。
状態は…Invalidなのかな？ちょっとわからないので変更しないでおきます。


>> ■ プラグインのcalender2で、曜日の表示が文字化けしました。
> これはfix済みです(by hsbt)。

ありがとうございました。


>> 他に、データディレクトリのパスが違う別のサーバーからデータを持って
>> きた時に、recent_comment3.cacheとrecent_trackback3.cacheがデータディ
>> レクトリ内に保存されるtdiary.confでフルパスになっているので、このパ
>> スを手動で変更する必要がありました。
> 
> これは次期以降の検討課題にさせてください。

はい、それほど困ることでも無い気がしますし、簡単にできそうならばお願い
します。

-- 
橋本 敬介  - HASHIMOTO Keisuke -
mailto:ksk@...
http://www.scrapcode.net/
http://d.hatena.ne.jp/khashi/

ただただしです。

2009/5/2 FUDAN <joker@...>:
> そこで、 tDiary.rb の141行目の scan メソッドに u フラグを追
> 加したところ、手元の環境では、当該箇所の文字化けは解消し、
> W3C Markup Validator から門前払いを食らうこともなくなりまし
> た。
>
> 修正前:                    matched = body.gsub( /\n/, ' ' ).scan( /^.{0,#{length - 2}}/ )[0]
> 修正後:                    matched = body.gsub( /\n/, ' ' ).scan( /^.{0,#{length - 2}}/u )[0]
>
> 似たような話題をどこかで見たような気もするのですが(^^;

これはひどい重複コードですね ><
＃でもしょうがない。

このパッチをあてておきます。ありがとうございました。

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

なかだです。

At Sat, 2 May 2009 22:28:22 +0900,
TADA Tadashi wrote:
> > 修正前:                    matched = body.gsub( /\n/, ' ' ).scan( /^.{0,#{length - 2}}/ )[0]
> > 修正後:                    matched = body.gsub( /\n/, ' ' ).scan( /^.{0,#{length - 2}}/u )[0]
> >
> > 似たような話題をどこかで見たような気もするのですが(^^;

データによっては順序を逆にしたほうが若干効率はいいのではないかと思います。

  body.scan(/\A.{0,#{length - 2}}/u).tr("\n", ' ')

-- 
--- 僕の前にBugはない。
--- 僕の後ろにBugはできる。
    中田 伸悦

ただただしです。

2009/5/2 Nobuyoshi Nakada <nobu@...>:
> データによっては順序を逆にしたほうが若干効率はいいのではないかと思います。
>
>  body.scan(/\A.{0,#{length - 2}}/u).tr("\n", ' ')

なるほど。

でも、これだと指定した長さまで切り詰めるという仕様が
満たせないですよね、改行が入った文字列だと。

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

なかだです。

At Sun, 3 May 2009 00:25:06 +0900,
TADA Tadashi wrote:
> > データによっては順序を逆にしたほうが若干効率はいいのではないかと思います。
> >
> >  body.scan(/\A.{0,#{length - 2}}/u).tr("\n", ' ')
> 
> なるほど。
> 
> でも、これだと指定した長さまで切り詰めるという仕様が
> 満たせないですよね、改行が入った文字列だと。

mオプションをつけ忘れましたが、そもそもscanじゃダメですね。

body[/\A.{0,#{length - 2}}/mu].tr("\n", ' ')

-- 
--- 僕の前にBugはない。
--- 僕の後ろにBugはできる。
    中田 伸悦

須藤です。

>> この間（？）のzunda会のときにすっかり話し忘れたのですが、
>> image.rbにあるアップロード機能がcoreに入って、TDiary::IOBase
>> 経由で保存してくれるようになると嬉しいです。
> 
> なるほど、ありですね。

おぉ！

> retroにつっこんでおいてくれると、忘れずに済みます:-)

入れておきました。