Project

General

Profile

Backport(バックポート) #3523

google mapのURLがhttpsで始まる場合、小窓化されない

Added by Mutsumi Imamura about 8 years ago. Updated almost 8 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
2010-11-02
Due date:
% Done:

100%


Description

Overview (現象)

google mapのURLがhttpsで始まる場合、小窓化されない。
httpで始まるURLであれば小窓化されます。

再現バージョン

下記のバージョンで再現確認済み。

  • OpenPNE3.6.13
  • OpenPNE3.8.9

再現手順

日記にhttpsで始まる google map の URLを入力し、表示を確認する

Causes (原因)

小窓化用の maps.google.com.js がhttpsに対応していないため

Way to fix (修正内容)

https://redmine.openpne.jp/issues/1755#note-4 に添付されている差分より

diff --git a/web/cmd/maps.google.co.jp.js b/web/cmd/maps.google.co.jp.js
index a38b2c1..99ede78 100644
--- a/web/cmd/maps.google.co.jp.js
+++ b/web/cmd/maps.google.co.jp.js
@@ -1,6 +1,6 @@
 function url2cmd(url, googlemapsUrl) {
-  var maps_google_co_jp_maps = url.match(/^http:\/\/maps\.google\.co\.jp\/maps\?(.+)/);
-  var maps_google_co_jp = url.match(/^http:\/\/maps\.google\.co\.jp\/\?(.+)/);
+  var maps_google_co_jp_maps = url.match(/^https?:\/\/maps\.google\.co\.jp\/maps\?(.+)/);
+  var maps_google_co_jp = url.match(/^https?:\/\/maps\.google\.co\.jp\/\?(.+)/);

   if (maps_google_co_jp_maps || maps_google_co_jp) {
      var id = RegExp.$1;
diff --git a/web/cmd/maps.google.com.js b/web/cmd/maps.google.com.js
index 9de34e3..4e8eee5 100644
--- a/web/cmd/maps.google.com.js
+++ b/web/cmd/maps.google.com.js
@@ -1,6 +1,6 @@
 function url2cmd(url, googlemapsUrl) {
-  var maps_google_com_maps = url.match(/^http:\/\/maps\.google\.com\/maps\?(.+)/);
-  var maps_google_com = url.match(/^http:\/\/maps\.google\.com\/\?(.+)/);
+  var maps_google_com_maps = url.match(/^https?:\/\/maps\.google\.com\/maps\?(.+)/);
+  var maps_google_com = url.match(/^https?\/\/maps\.google\.com\/\?(.+)/);

   if (maps_google_com_maps || maps_google_com) {
      var id = RegExp.$1;

Related issues

Related to OpenPNE 3 - Bug(バグ) #3521: google mapのURLがhttpsで始まる場合、小窓化されない Won't fix(対応せず) 2010-11-02
Copied to OpenPNE 3 - Bug(バグ) #3551: 表示しているページのURLがhttpsの場合にユーザが httpのURL の小窓を作成した場合ブラウザのエラーが表示される New(新規) 2014-02-04

Associated revisions

Revision 273d4013 (diff)
Added by Youichi Kimura about 8 years ago

fix pattern for Google Maps url in cmd scripts (fixes #3523 BP from #3521)
(cherry picked from commit c588f6f2a338cd4bbe4a058f7c4d392cc0fdf521)

patch from https://redmine.openpne.jp/issues/1755#note-4

History

#1 Updated by Mutsumi Imamura about 8 years ago

  • Tracker changed from Bug(バグ) to Backport(バックポート)

#2 Updated by Youichi Kimura about 8 years ago

  • Description updated (diff)

#3 Updated by isao sano about 8 years ago

  • Status changed from New(新規) to Accepted(着手)

#4 Updated by isao sano about 8 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

元チケットの修正を取り込んだものをpull request行いました。
https://github.com/openpne/OpenPNE3/pull/100

#5 Updated by Mutsumi Imamura about 8 years ago

  • Assignee set to isao sano

#6 Updated by Shinichi Urabe almost 8 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

修正内容自体問題ないと判断しプルリクエストを取り込みました。

フィードバックします

  • Google マップ以外にも YouTube など、他の小窓でも本現象が発生しますが、対応する必要はないでしょうか
    • 別チケットでの対応でよいかとは思いますが、バグチケットは見当たりませんでした
  • 元々の問題ですが本文の URL が https | http であるか区別せず小窓を表示する対応になっている点は問題無いですが、表示しているページの URL が https の場合にユーザが http の URL を貼り付けた場合、ブラウザからエラーが通知されたり、コンテンツが表示されない現象が発生しますが、その対処は不要でしょうか

#7 Updated by Youichi Kimura almost 8 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)

更新履歴 273d4013113611a20fb5b41086c7bfaaa00c2da0 で適用されました。

#8 Updated by Shinichi Urabe almost 8 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

#9 Updated by Mutsumi Imamura almost 8 years ago

  • Copied to Bug(バグ) #3550: URLがhttpsで始まる場合、小窓化されない小窓がある added

#10 Updated by Mutsumi Imamura almost 8 years ago

  • Copied to deleted (Bug(バグ) #3550: URLがhttpsで始まる場合、小窓化されない小窓がある)

#11 Updated by Mutsumi Imamura almost 8 years ago

  • Copied to Bug(バグ) #3551: 表示しているページのURLがhttpsの場合にユーザが httpのURL の小窓を作成した場合ブラウザのエラーが表示される added

#12 Updated by Mutsumi Imamura almost 8 years ago

  • Status changed from Rejected(差し戻し) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

Shinichi Urabe は書きました:

フィードバックします

  • Google マップ以外にも YouTube など、他の小窓でも本現象が発生しますが、対応する必要はないでしょうか
    • 別チケットでの対応でよいかとは思いますが、バグチケットは見当たりませんでした
  • 元々の問題ですが本文の URL が https | http であるか区別せず小窓を表示する対応になっている点は問題無いですが、表示しているページの URL が https の場合にユーザが http の URL を貼り付けた場合、ブラウザからエラーが通知されたり、コンテンツが表示されない現象が発生しますが、その対処は不要でしょうか

フィードバック内容については下記のチケットで対応することとします。
よって、本チケットの修正範囲としては問題無いと判断し、ステータスを変更します。

  • #3550 URLがhttpsで始まる場合、小窓化されない小窓がある
  • #3551 表示しているページのURLがhttpsの場合にユーザが httpのURL の小窓を作成した場合ブラウザのエラーが表示される

#13 Updated by Chiharu Nakajima almost 8 years ago

試験実施完了

#14 Updated by Chiharu Nakajima almost 8 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

Also available in: Atom PDF