食べログ小窓が表示されない

[imamura623]

■現象

食べログ小窓が表示されない

■原因

食べログ側の変更によってAPIが正常に稼働しなくなったため

■修正内容

食べログ側の変更にあわせAPI用のJSを修正

この修正で食べログ小窓が表示されるようになりました

■関連情報

comment:3

元のDescription

​http://sns.openpne.jp/?m=pc&a=page_fh_diary&target_c_diary_id=19269

どうもURLの定義が変わった？

http://r.tabelog.com/tokyo/A1314/A131401/13007842/


うーん・・・ 

食べログ側の変更によってAPIが正常に稼動しなくなったものかと思われますが・・・

[nakasone]

以下のような確認を行ったところ再現できました。

■確認ブラウザ

• !FireFox3

■確認バージョン

• 2.10.x
• 2.12.x
• trunk

■手順

1. 食べログでページのアドレスをコピーする
2. 日記を書く→本文にコピーしたアドレスを貼り付ける
3. 日記を確認→小窓が表示されず、アドレスが表示される

[tejimaa]

お返事頂きました！ こんな感じでJSの変更をお願いします。

URL変更に伴い、
http://sns.openpne.jp/cmd/r.tabelog.com.js
を変更する必要がございますが、新しいURL構造は、
http://r.tabelog.com/tokyo/A1306/A130602/13002113/
のように、
http://r.tabelog.com/[a-z]+/A[0-9]+/A[0-9]+/[0-9]+/?
という形です。

[akamine]

対応します。

[akamine]

以下のリビジョンを修正しました。

• 2.10.x - r9145
• 2.12.x - r9143
• trunk - r9144

[ebihara]

​http://r.tabelog.com/tokyo/A1306/A130602/13002113#pagetop というURLが入力された場合に変換がおこなわれないのは意図的でしょうか。

[akamine]

以下のリビジョンを再度修正しました。

• 2.10.x - r9204
• 2.12.x - r9205
• trunk - r9206

[akamine]

以下のリビジョンを再度修正しました。

• 2.10.x - r9268
• 2.12.x - r9265
• trunk - r9266

[ebihara]

• 新形式のルールと旧形式のルールの両方のチェックをおこなうことで、どちらのURLに関しても小窓が表示されるようになると思います。既に稼働中のSNSを考慮し、旧形式のURLであっても小窓化するように修正してください。

[akamine]

以下のリビジョンを修正しました。

• 2.10.x - r9282
• 2.12.x - r9280
• trunk - r9281

[kiwa]

http://r.tabelog.com/okinawa/A4705/A470501/47000327/dtlmenu/

上記のような（店舗詳細の「お店トップ」以外の）URLが入力された場合、「システムエラーが発生しました。」と表示された食べログ小窓が表示されます。小窓にするのか、URLにするのか、どちらかで対応したほうがいいと思います。

[akamine]

以下のリビジョンを修正しました。
comment:15 を受け、今回は従来通り#pagetopや/dtlmenu/等が入った場合urlとして返すよう統一しました。

• 2.10.x - r9291
• 2.12.x - r9289
• trunk - r9290

[ebihara]

comment:14 で指摘しそびれていました、ごめんなさい。以下の修正をお願いします。

• 以下の表記はわかりにくいです。

    2     if (url.match(/^http:\/\/r\.tabelog\.com\/[a-z]+\/A[0-9]+\/A[0-9]+\/([0-9]+)\/?/) || url.match(/^http:\/\/r\.tabelog\.com\/[a-z]+\/rstdtl\/([0-9]+)\/?/)) {
    3     } else {
  

  たとえば以下のように修正してはどうでしょう？

    2     if (url.match(/^http:\/\/r\.tabelog\.com\/[a-z]+\/A[0-9]+\/A[0-9]+\/([0-9]+)\/?/) || url.match(/^http:\/\/r\.tabelog\.com\/[a-z]+\/rstdtl\/([0-9]+)\/?/    )) {  3         var rcd = RegExp.$1;  4         main(rcd);
    5         return;
    6     }
    7 
    8     pne_url2a(url);
  

[akamine]

以下のリビジョンでコードの記述を修正しました。

• 2.10.x - r9294
• 2.12.x - r9292
• trunk - r9293

[kiwa]

確認しました。問題ありません。