Enhancement(機能追加・改善) #3775
完了小窓機能 google map が機能していないので修正してほしい
kaoru n さんがほぼ10年前に追加. 7年以上前に更新.
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) 0001-refs-3775-Unified-indent-the-notation.patch | Shinichi Urabe, 2016-03-09 23:29 | ||
0002-refs-3775-It-was-also-corresponds-to-the-case-where-.patch (943 Bytes) 0002-refs-3775-It-was-also-corresponds-to-the-case-where-.patch | Shinichi Urabe, 2016-03-09 23:29 | ||
0003-Only-change-the-method-of-obtaining-the-value-in-the.patch (3.01 KB) 0003-Only-change-the-method-of-obtaining-the-value-in-the.patch | Shinichi Urabe, 2016-03-09 23:29 | ||
0004-Modified-so-that-each-parameter-to-match-the-case-wh.patch (874 Bytes) 0004-Modified-so-that-each-parameter-to-match-the-case-wh.patch | Shinichi Urabe, 2016-03-10 00:36 | ||
0005-It-is-made-to-correspond-to-the-satellite-view-refs-.patch (1.04 KB) 0005-It-is-made-to-correspond-to-the-satellite-view-refs-.patch | Shinichi Urabe, 2016-03-10 00:36 |
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