プロジェクト

全般

プロフィール

Backport(バックポート) #3523

完了

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

Mutsumi Imamura さんがほぼ11年前に追加. ほぼ11年前に更新.

ステータス:
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;

関連するチケット 2 (1件未完了1件完了)

関連している OpenPNE 3 - Bug(バグ) #3521: google mapのURLがhttpsで始まる場合、小窓化されないWon't fix(対応せず)Youichi Kimura2010-11-02

操作
コピー先 OpenPNE 3 - Bug(バグ) #3551: 表示しているページのURLがhttpsの場合にユーザが httpのURL の小窓を作成した場合ブラウザのエラーが表示されるNew(新規)2014-02-04

操作

Mutsumi Imamura さんがほぼ11年前に更新

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

Youichi Kimura さんがほぼ11年前に更新

isao sano さんがほぼ11年前に更新

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

isao sano さんがほぼ11年前に更新

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

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

Mutsumi Imamura さんがほぼ11年前に更新

  • 担当者isao sano にセット

Shinichi Urabe さんがほぼ11年前に更新

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

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

フィードバックします

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

Youichi Kimura さんがほぼ11年前に更新

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

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

Shinichi Urabe さんがほぼ11年前に更新

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

Mutsumi Imamura さんがほぼ11年前に更新

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

Mutsumi Imamura さんがほぼ11年前に更新

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

Mutsumi Imamura さんがほぼ11年前に更新

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

Mutsumi Imamura さんがほぼ11年前に更新

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

Shinichi Urabe は書きました:

フィードバックします

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

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

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

Chiharu Nakajima さんがほぼ11年前に更新

試験実施完了

Chiharu Nakajima さんがほぼ11年前に更新

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

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