プロジェクト

全般

プロフィール

Backport(バックポート) #3523

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

Mutsumi Imamura10年以上前に追加. 約10年前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2010-11-02
期日:
進捗率:

100%


説明

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;

関連するチケット

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

関係しているリビジョン

リビジョン 273d4013 (差分)
Youichi Kimura約10年前に追加

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

履歴

#1 Mutsumi Imamura10年以上前に更新

  • トラッカーBug(バグ) から Backport(バックポート) に変更

#2 Youichi Kimura10年以上前に更新

  • 説明 を更新 (diff)

#3 isao sano約10年前に更新

  • ステータスNew(新規) から Accepted(着手) に変更

#4 isao sano約10年前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

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

#5 Mutsumi Imamura約10年前に更新

  • 担当者isao sano にセット

#6 Shinichi Urabe約10年前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

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

フィードバックします

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

#7 Youichi Kimura約10年前に更新

  • ステータスRejected(差し戻し) から Pending Review(レビュー待ち) に変更

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

#8 Shinichi Urabe約10年前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

#9 Mutsumi Imamura約10年前に更新

  • コピー先 Bug(バグ) #3550: URLがhttpsで始まる場合、小窓化されない小窓がある を追加

#10 Mutsumi Imamura約10年前に更新

  • コピー先 を削除 (Bug(バグ) #3550: URLがhttpsで始まる場合、小窓化されない小窓がある)

#11 Mutsumi Imamura約10年前に更新

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

#12 Mutsumi Imamura約10年前に更新

  • ステータスRejected(差し戻し) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

Shinichi Urabe は書きました:

フィードバックします

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

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

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

#13 Chiharu Nakajima約10年前に更新

試験実施完了

#14 Chiharu Nakajima約10年前に更新

  • ステータスPending Testing(テスト待ち) から Fixed(完了) に変更
  • 進捗率70 から 100 に変更

他の形式にエクスポート: Atom PDF