Project

General

Profile

Enhancement(機能追加・改善) #3775

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

Added by kaoru n over 4 years ago. Updated over 2 years ago.

Status:
Won't fix(対応せず)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
2015-02-10
Due date:
% Done:

0%


Description

概要

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

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

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

仕様

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

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

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

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

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

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


Related issues

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

History

#1 Updated by kaoru n over 4 years ago

#2 Updated by isao sano over 3 years ago

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

#3 Updated by isao sano over 3 years ago

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

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

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

#4 Updated by isao sano over 3 years ago

#5 Updated by isao sano over 3 years ago

#6 Updated by Shinichi Urabe over 3 years ago

  • 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 Updated by Shinichi Urabe over 3 years ago

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

#9 Updated by isao sano over 3 years ago

  • Status changed from Rejected(差し戻し) to Accepted(着手)
  • % Done changed from 50 to 0

#10 Updated by isao sano over 3 years ago

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

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

#11 Updated by Shinichi Urabe over 3 years ago

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

isao sano さんは書きました:

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

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

#12 Updated by isao sano over 3 years ago

  • Status changed from Rejected(差し戻し) to Accepted(着手)
  • % Done changed from 50 to 0

#13 Updated by isao sano over 3 years ago

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

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

#14 Updated by Shinichi Urabe over 3 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

OK

#15 Updated by isao sano over 3 years ago

  • Status changed from Pending Testing(テスト待ち) to Pending Review(レビュー待ち)
  • % Done changed from 70 to 50

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

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

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

#16 Updated by Shinichi Urabe over 3 years ago

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

isao sano さんは書きました:

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

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

#17 Updated by isao sano over 3 years ago

  • Status changed from Rejected(差し戻し) to Accepted(着手)
  • % Done changed from 50 to 0

#18 Updated by isao sano over 3 years ago

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

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

#19 Updated by Shinichi Urabe over 3 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

#20 Updated by Youichi Kimura over 3 years ago

#21 Updated by Youichi Kimura over 3 years ago

#22 Updated by Youichi Kimura over 3 years ago

  • Status changed from Pending Testing(テスト待ち) to Rejected(差し戻し)
  • % Done changed from 70 to 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 Updated by isao sano about 3 years ago

  • Status changed from Rejected(差し戻し) to Accepted(着手)
  • % Done changed from 50 to 0

#24 Updated by isao sano about 3 years ago

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

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

#25 Updated by kaoru n over 2 years ago

  • Status changed from Pending Review(レビュー待ち) to Won't fix(対応せず)
  • Target version changed from OpenPNE 3.9.0-old to OpenPNE 3.9.0
  • % Done changed from 50 to 0

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

Also available in: Atom PDF