Enhancement（機能追加・改善） #3775
完了
小窓機能 google map が機能していないので修正してほしい
0%
説明
概要¶
http://www.openpne.jp/cmd/list/#maps.google.co.jp
GoogleマップはURLではなく、右端にある【このページのリンク】→【このリンクをメールに貼り付けて地図を共有できます】のURLを貼ることで表示することができます。
と説明されているが、現在のgooglemapにある「地図を共有または埋め込む」から取得できるURLはアドレスバーに表示されているものと同じであり、このURLを使っても地図は表示されない
仕様¶
仕様がある場合、仕様を記入
ファイル
isao sano さんがほぼ9年前に更新
- ステータス を Accepted（着手） から Pending Review（レビュー待ち） に変更
- 担当者 を isao sano にセット
- 進捗率 を 0 から 50 に変更
新しくなった Googole map の URL に含まれる緯度、経度、ズームの値を取得して小窓を表示するように修正しました。URLを直接貼ることで小窓化されます。
現段階では従来の形式のURLでも小窓が表示できるように分岐しています。
https://github.com/openpne/OpenPNE3/pull/310 にてプルリクエストしました。
isao sano さんが8年以上前に更新
- 関連している Backport（バックポート） #3892: 小窓機能 google map が機能していないので修正してほしい を追加
isao sano さんが8年以上前に更新
- 関連している Backport（バックポート） #3891: 小窓機能 google map が機能していないので修正してほしい を追加
Shinichi Urabe さんが8年以上前に更新
- ファイル 0001-refs-3775-Unified-indent-the-notation.patch 0001-refs-3775-Unified-indent-the-notation.patch を追加
- ファイル 0002-refs-3775-It-was-also-corresponds-to-the-case-where-.patch 0002-refs-3775-It-was-also-corresponds-to-the-case-where-.patch を追加
- ファイル 0003-Only-change-the-method-of-obtaining-the-value-in-the.patch 0003-Only-change-the-method-of-obtaining-the-value-in-the.patch を追加
- ステータス を Pending Review（レビュー待ち） から Rejected（差し戻し） に変更
- 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を二度記述しなくても、対応する方法はあると思います
修正案のパッチを添付しておきます
Shinichi Urabe さんが8年以上前に更新
isao sano さんが8年以上前に更新
- ステータス を Accepted（着手） から Pending Review（レビュー待ち） に変更
- 進捗率 を 0 から 50 に変更
https://github.com/openpne/OpenPNE3/pull/310 にてプルリクエストしました。
再度レビューお願いします。
Shinichi Urabe さんが8年以上前に更新
- ステータス を Pending Review（レビュー待ち） から Rejected（差し戻し） に変更
isao sano さんは書きました:
https://github.com/openpne/OpenPNE3/pull/310 にてプルリクエストしました。
再度レビューお願いします。
マイナスを含むパターンで [-]? とありますが [] は不要で -? でいいと思います
isao sano さんが8年以上前に更新
- ステータス を Accepted（着手） から Pending Review（レビュー待ち） に変更
- 進捗率 を 0 から 50 に変更
https://github.com/openpne/OpenPNE3/pull/310
に修正追加しました。
再度レビューお願いします。
Shinichi Urabe さんが8年以上前に更新
- ステータス を Pending Review（レビュー待ち） から Pending Testing（テスト待ち） に変更
- 進捗率 を 50 から 70 に変更
OK
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
にて、' や " を含む部分がマッチしないように修正しました。
@ が先頭もしくは / の直後に来る場合のみマッチするようにしました。
Shinichi Urabe さんが8年以上前に更新
- ステータス を Pending Review（レビュー待ち） から Rejected（差し戻し） に変更
isao sano さんは書きました:
https://github.com/openpne/OpenPNE3/pull/310
[...]
にて、' や " を含む部分がマッチしないように修正しました。
@ が先頭もしくは / の直後に来る場合のみマッチするようにしました。
result[1] の結果が変わるので (?:^|\/) のようにして result[1] に含まれないようにしたほうがいいです
isao sano さんが8年以上前に更新
- ステータス を Accepted（着手） から Pending Review（レビュー待ち） に変更
- 進捗率 を 0 から 50 に変更
note-16 の問題を修正しました。
https://github.com/openpne/OpenPNE3/pull/310
Shinichi Urabe さんが8年以上前に更新
- ステータス を Pending Review（レビュー待ち） から Pending Testing（テスト待ち） に変更
- 進捗率 を 50 から 70 に変更
Youichi Kimura さんが8年以上前に更新
- 関連している Backport（バックポート） #3948: Google MapsのURLのズームレベルに小数が含まれていると小窓化されない を追加
Youichi Kimura さんが8年以上前に更新
- 関連している Backport（バックポート） #3949: Google MapsのURLのズームレベルに小数が含まれていると小窓化されない を追加
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