Fix "Video - Nico Nico Douga" Extractor #79 #194
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
試しています。 |
|
もしかしたら Qバージョンってのにしないとなりませんか？ |
|
@syoichi が書かれている「ニコニコ動画ログアウト時の動画ページ」を取ってこれると Open Graph を使ってもっと簡単になりそうですね。 |
|
強引ですけど、こんなのはどうでしょうか？ ちょっと強引過ぎますね。すみません。 |
|
検証して頂きありがとうございます。原宿バージョンへの対応などを行いました。 |
|
ありがとうございます。こちらの環境(原宿バージョン)でも動きました。 |
|
ありがとうございます!! これなら大丈夫そうですね, merge します! |
Constellation
added a commit
that referenced
this pull request
Apr 29, 2013
Fix "Video - Nico Nico Douga" Extractor #79
Closed
wantora
pushed a commit
to wantora/tombfix-core
that referenced
this pull request
May 13, 2016
* Rename "Video - Nico Nico Douga" to "Video - Nicovideo" via https://github.com/Constellation/taberareloo/blob/1a6e47f03e8a93ace56a7fece05fd214123893ce/src/lib/extractors.js#L1225 taberareloo/taberareloo#194
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#79 の問題を解決する為にニコニコ動画のExtractorを修正しました。
#95 でConstellationさんが示された、公式で提供されている外部プレーヤーのコード(script要素で外部プレーヤーを生成)、あるいはそのコードで使用されているJavaScriptを
eval()で実行して得た情報を利用せず、ニコニコ動画におけるOpen Graph Protocolのmeta要素の情報(ニコニコ動画ログアウト時の動画ページのhead要素を参照)を参考にembed要素で外部プレーヤーを表示できようにしました。これにより、TumblrではDashboardでもTaberarelooで投稿されたVideoポストは外部プレーヤーで動画を視聴できるようになります。また、#95 のYungSangさんのコードを参考にGoogle+へ投稿する際の対応も行なっています。こちらでは他のExtractorでもポストタイプの変換処理を適用できるようにしています。
この変更の動作は、Windows 7 Home Premium SP1 64bit上のChrome 26.0.1410.64、拡張のバージョンは2.0.79-dev( 17f1047 までの変更を含む)という環境で確認しています。