画像の貼り付け

三輪です。
代替テキストを指定した画像リンクが、
画像貼り付けではなく単なるリンクになってしまいます。

[[alt|http://example.com/a.jpg]]

例えば書式詳細の中にある「New!」もおかしくなっています。
http://qwik.jp/qwikweb/TextFormat.html

バグだとは思うんですが、テストケースまで用意してあることから、
もしかして意図した仕様なのかも知れないと思い
念のため確認する次第です。

 ok([:img, {:alt=>'http://e.com/t.jpg', :src=>'http://e.com/t.jpg'}],
    'http://e.com/t.jpg')
 ok([:a, {:href=>'http://e.com/t.jpg'}, 's'],
    's|http://e.com/t.jpg')

もし単なるバグなら修正＆リファクタリングしてcommitしておきますが、
どうでしょうか。

江渡です。

代替テキストを指定した画像リンクが、 画像貼り付けではなく単なるリンクになってしまいます。

例えば書式詳細の中にある「New!」もおかしくなっています。 http://qwik.jp/qwikweb/TextFormat.html

バグだとは思うんですが、テストケースまで用意してあることから、 もしかして意図した仕様なのかも知れないと思い 念のため確認する次第です。

たしかに微妙です。この仕様は、リンク先の拡張子が.gifや.jpgだったら
img srcにするというすごくアドホックな実装になっていて、そのアドホック
さ加減がすごく嫌だったので、無意識の内にダウングレードしちゃっていたの
かもしれない。

とはいえ、普通のリンクとして指示したときはimg srcになるので、今回の
これは単なるバグとして修正しましょう。

ok([:img, {:alt=>'http://e.com/t.jpg', :src=>'http://e.com/t.jpg'}],
'http://e.com/t.jpg')
ok([:a, {:href=>'http://e.com/t.jpg'}, 's'],
's|http://e.com/t.jpg')

もし単なるバグなら修正＆リファクタリングしてcommitしておきますが、 どうでしょうか。

すいませんが、よろしくおねがいします。

三輪です。
svnにコミットしておきました。

ロジックがかなり複雑になっていたので、
parse_ref全体をリファクタリングさせていただきました。
既存のテストに対しては完全に互換性を保っています（img srcの件除く）。
同時にいくつか際どいケースに対するテストを追加しました。
こちらは一部互換性が損なわれましたが、
元々考慮外のケースだったわけで、問題にはならないはずです。

あと完全に別件ですが、act-diary.rb内の定数名の間違いにより
warningが出る問題も修正しておきました
（D_PluginInclude_ja -> D_PluginDiary_ja）。

この仕様は、リンク先の拡張子が.gifや.jpgだったら img srcにするというすごくアドホックな実装になっていて、そのアドホック さ加減がすごく嫌だったので、無意識の内にダウングレードしちゃっていたの かもしれない。

画像の貼り付け専用のsyntaxがあった方が良いのでしょうかね。
他のWikiでも色々のようですけど…。