プロジェクト

全般

プロフィール

Enhancement（機能追加・改善） #3775

小窓機能 google map が機能していないので修正してほしい

kaoru n が約9年前に追加. 約7年前に更新.

ステータス:
Won't fix（対応せず）
優先度:
Normal（通常）
担当者:
isao sano
対象バージョン:
OpenPNE 3.9.0
開始日:
2015-02-10
期日:
進捗率:

0%

説明

概要

http://www.openpne.jp/cmd/list/#maps.google.co.jp

GoogleマップはURLではなく、右端にある【このページのリンク】→【このリンクをメールに貼り付けて地図を共有できます】のURLを貼ることで表示することができます。

と説明されているが、現在のgooglemapにある「地図を共有または埋め込む」から取得できるURLはアドレスバーに表示されているものと同じであり、このURLを使っても地図は表示されない

仕様

仕様がある場合、仕様を記入

0001-refs-3775-Unified-indent-the-notation.patch 表示 (2.92 KB) Shinichi Urabe, 2016-03-09 23:29

0002-refs-3775-It-was-also-corresponds-to-the-case-where-.patch 表示 (943 Bytes) Shinichi Urabe, 2016-03-09 23:29

0003-Only-change-the-method-of-obtaining-the-value-in-the.patch 表示 (3.01 KB) Shinichi Urabe, 2016-03-09 23:29

0004-Modified-so-that-each-parameter-to-match-the-case-wh.patch 表示 (874 Bytes) Shinichi Urabe, 2016-03-10 00:36

0005-It-is-made-to-correspond-to-the-satellite-view-refs-.patch 表示 (1.04 KB) Shinichi Urabe, 2016-03-10 00:36

関連するチケット

関連している OpenPNE 3 - Task（タスク） #3759: 小窓スクリプトの動作確認（2015年03月リリース分） Fixed（完了） 2016-01-27
関連している OpenPNE 3 - Backport（バックポート） #3892: 小窓機能 google map が機能していないので修正してほしい Fixed（完了） 2016-03-09
関連している OpenPNE 3 - Backport（バックポート） #3891: 小窓機能 google map が機能していないので修正してほしい Fixed（完了） 2016-03-09
関連している OpenPNE 3 - Backport（バックポート） #3948: Google MapsのURLのズームレベルに小数が含まれていると小窓化されない Fixed（完了） 2016-03-09
関連している OpenPNE 3 - Backport（バックポート） #3949: Google MapsのURLのズームレベルに小数が含まれていると小窓化されない Fixed（完了） 2016-03-09

履歴

#1 kaoru n が約9年前に更新

  • 関連している Task（タスク） #3759: 小窓スクリプトの動作確認（2015年03月リリース分） を追加

#2 isao sano が約8年前に更新

  • ステータス を New（新規） から Accepted（着手） に変更

#3 isao sano が約8年前に更新

  • ステータス を Accepted（着手） から Pending Review（レビュー待ち） に変更
  • 担当者 を isao sano にセット
  • 進捗率 を 0 から 50 に変更

新しくなった Googole map の URL に含まれる緯度、経度、ズームの値を取得して小窓を表示するように修正しました。URLを直接貼ることで小窓化されます。
現段階では従来の形式のURLでも小窓が表示できるように分岐しています。

https://github.com/openpne/OpenPNE3/pull/310 にてプルリクエストしました。

#4 isao sano が約8年前に更新

#5 isao sano が約8年前に更新

#6 Shinichi Urabe が約8年前に更新

  • URL が https://www.google.co.jp/maps?xxx ではなく https://www.google.co.jp/maps/xxx の場合もあるので、それにマッチしません (maps の後ろが クエスチョン ではなく スラッシュ)
  • 元々おかしいのと JS にはコーディング規約はないですが、統一した方がいいと思われる箇所があります
    • インデントが 統一されていません (JS にはコーディング規約はないですが、インデントは統一したほうがいいです
    • if () {, for () { の波括弧の前にスペースがない カッコの前にスペースがない、カッコの中にスペースがあるなど統一されていないです
  • 旧URLに ＠ マークが混ざるケースはないでしょうか (確認不可)
  • 現行の GoogleMap には、マップで例えば 「@マーク」で検索すると、URL がhttps://www.google.co.jp/maps/search/'@%E3%83%9E%E3%83%BC%E3%82%AF'/@111,111,25z のようになります, この状態で単純に id.match(/@/) では、取得できる値が想定しないものとなり、まずくないでしょうか
  • 上記のケースから @ があるかだけの正規表現とするのはよくないと思います
  • var html を二度記述しなくても、対応する方法はあると思います

修正案のパッチを添付しておきます

#8 Shinichi Urabe が約8年前に更新

添付したパッチでは抜けてましたが、東経 西経 北緯 南緯 において、マイナスがつく場合もありますね

#9 isao sano が約8年前に更新

  • ステータス を Rejected（差し戻し） から Accepted（着手） に変更
  • 進捗率 を 50 から 0 に変更

#10 isao sano が約8年前に更新

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

https://github.com/openpne/OpenPNE3/pull/310 にてプルリクエストしました。
再度レビューお願いします。

#11 Shinichi Urabe が約8年前に更新

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

isao sano さんは書きました:

https://github.com/openpne/OpenPNE3/pull/310 にてプルリクエストしました。
再度レビューお願いします。

マイナスを含むパターンで [-]? とありますが [] は不要で -? でいいと思います

#12 isao sano が約8年前に更新

  • ステータス を Rejected（差し戻し） から Accepted（着手） に変更
  • 進捗率 を 50 から 0 に変更

#13 isao sano が約8年前に更新

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

https://github.com/openpne/OpenPNE3/pull/310
に修正追加しました。
再度レビューお願いします。

#14 Shinichi Urabe が約8年前に更新

  • ステータス を Pending Review（レビュー待ち） から Pending Testing（テスト待ち） に変更
  • 進捗率 を 50 から 70 に変更

OK

#15 isao sano が約8年前に更新

  • ステータス を Pending Testing（テスト待ち） から Pending Review（レビュー待ち） に変更
  • 進捗率 を 70 から 50 に変更

https://github.com/openpne/OpenPNE3/pull/310 

Modify as quoted part does not match (refs #3775) 
commit 964f54c2755a15d214c66b3cd6144189fe558123

にて、' や " を含む部分がマッチしないように修正しました。
@ が先頭もしくは / の直後に来る場合のみマッチするようにしました。

#16 Shinichi Urabe が約8年前に更新

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

isao sano さんは書きました:

https://github.com/openpne/OpenPNE3/pull/310
[...]
にて、' や " を含む部分がマッチしないように修正しました。
@ が先頭もしくは / の直後に来る場合のみマッチするようにしました。

result[1] の結果が変わるので (?:^|\/) のようにして result[1] に含まれないようにしたほうがいいです

#17 isao sano が約8年前に更新

  • ステータス を Rejected（差し戻し） から Accepted（着手） に変更
  • 進捗率 を 50 から 0 に変更

#18 isao sano が約8年前に更新

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

note-16 の問題を修正しました。
https://github.com/openpne/OpenPNE3/pull/310

#19 Shinichi Urabe が約8年前に更新

  • ステータス を Pending Review（レビュー待ち） から Pending Testing（テスト待ち） に変更
  • 進捗率 を 50 から 70 に変更

#20 Youichi Kimura がほぼ8年前に更新

#21 Youichi Kimura がほぼ8年前に更新

#22 Youichi Kimura がほぼ8年前に更新

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

ズームレベルに小数が含まれた URL の場合に小窓化されない不具合を確認したため差し戻します。
3.6.x, 3.8.x 向けのバックポートは #3891, #3892 が既に Fixed となっていたため、新たに #3948, #3949 を作成しました。

例: https://www.google.co.jp/maps/@35.7108947,139.7113625,20.5z?hl=ja

#23 isao sano がほぼ8年前に更新

  • ステータス を Rejected（差し戻し） から Accepted（着手） に変更
  • 進捗率 を 50 から 0 に変更

#24 isao sano がほぼ8年前に更新

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

https://github.com/openpne/OpenPNE3/pull/310 にて #note-22 の問題を修正しました。

#25 kaoru n が約7年前に更新

  • ステータス を Pending Review（レビュー待ち） から Won't fix（対応せず） に変更
  • 対象バージョン を OpenPNE 3.9.0-old から OpenPNE 3.9.0 に変更
  • 進捗率 を 50 から 0 に変更

OpenPNE 3.8.19 にて対応済みであったため、対応せずとします。

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