Project

General

Profile

バグ #624

openimg: タグのスクレイピング方法を修正

Added by hycon miyagi over 6 years ago. Updated over 6 years ago.

Status:
終了
Priority:
通常
Assignee:
Target version:
プラグイン名:
openimg
ブランチ:
クラッシュする:

Description

タグの属性名に '-' などが使われているとスクレイピングで拾わないので修正しました。

ついでにTumblrの仕様変更に対応。…バグ修正前にこれに合わせて条件を書き換える作業をしていたのですが
うまく行かず、調査の結果 "data-src" という属性をスルーするような動きをするので発覚しました。
XMLの名前文字として使えないものも大幅にマッチする可能性がありますがちゃんと対応するとだるいし
現実のHTMLもブラウザもそんなことは誰も気にしないので問題無いかと。

パッチを添付しますのでよろしくお願いします。

全然関係無いけどここのアイコン変えるにはgravatar使うといいよとか書いとくと便利だと思う。


Files

openimg-20131102.diff (1.73 KB) openimg-20131102.diff hycon miyagi, 2013-11-02 15:37
openimg-20131102-2.diff (1.76 KB) openimg-20131102-2.diff hycon miyagi, 2013-11-02 15:43
openimg-20131104.diff (1.52 KB) openimg-20131104.diff hycon miyagi, 2013-11-04 00:33
#1

Updated by hycon miyagi over 6 years ago

上のパッチは無かったことにしてください…

#2

Updated by hycon miyagi over 6 years ago

また訂正です。他のサイトでテストちゃんとしてませんでした…。もう大丈夫です

#3

Updated by toshi_a 初音 over 6 years ago

  • Status changed from 新規 to レビュー待ち
  • Assignee set to hycon miyagi
  • Target version set to 0.2.2

hotfix-0.2.2 ブランチに取り込みました。確認おねがいします。

#4

Updated by hycon miyagi over 6 years ago

  • Status changed from レビュー待ち to 解決

確認しました。遅くなりましてすみません。

#5

Updated by toshi_a 初音 over 6 years ago

  • Status changed from 解決 to 終了

Also available in: Atom PDF