Locations Plugin 🌍

  • FIX: グローバルなトピックマップを直接開いても問題が発生しないようになりました
  • FIX: カテゴリマップとグローバルマップ間の遷移が正しく機能するようになりました
  • 機能しない WIP マップテスト

@vincefrommtl@Stephane_Roy @packman、このアップデートをお試しいただけますでしょうか?修正の範囲にご注意ください。もうすぐ別の修正がリリースされます。

(技術的な注意:Leaflet ライブラリはグローバルオブジェクトであるため、QUnit がそれを認識できないようです。そのため、これらの修正に対する完全に統合された FE テストを提供できませんでした。Leaflet と QUnit を連携させる方法をご存知の方がいらっしゃいましたら、お知らせください。)

「いいね!」 3

フィードバック:

  • グローバルマップとカテゴリマップ間の遷移は機能しています。しかし、その逆(カテゴリマップからグローバルマップへ)は、カテゴリからカテゴリへの遷移と同様に壊れています。
  • アップグレード後、このエラーはログで見ていませんActionView::Template::Error (No route matches {:action=>"map_feed", :controller=>"list"}):slight_smile:
  • 新しいバグ: マップ上のクラスタ化されたリンクが、ズームインしたときに位置マーカーのみを表示し、ラベルは表示されないことがあります。
  • 新しいバグ: マップ上のマーカーのラベルが、対応するトピックにリンクされないことがあります。
  • 新しいバグ: グローバルマップもカテゴリマップと同じバグの影響を受けています:マーカーが少数しか表示されません(下記参照)。

マーカーが本来より少なく表示されるマップについて:詳しく調べましたが、過去20日以内(またはそれに近い期間)にアクティブだったトピックのマーカーのみが表示され、最大でも30個です。フォーラムの設定の問題の可能性がありますか?非常に具体的です!しかし、これらの設定は何も変更していません。私は最大1700のロケーションマップの最大トピック数を持っており、過去20日間のトピックのみを表示するような時間に関連する設定があるとは思いません。

「いいね!」 1

アップグレード後、グローバルマップを開きましたが、マーカー(トピック)が表示されませんでした。マーカーのあるトピックの場所を編集して何も変更せずに保存すると、マーカーが表示されます。私のマーカーはすべて20日以上経過しているので、@vincefrommtl が見ているのと同じ問題かもしれません。

編集:グローバルマップに表示されないマーカーの場合でも、各トピックの「マップを表示」ボタンは正しいマーカーの場所を示しています。

「いいね!」 1

ありがとう、もう一度見てみます。

「いいね!」 2

これは完全な修正ではありませんが、外部リンクまたはブラウザの更新から直接読み込んだ際のマップエラーを回避しながら、古いトピックの場所を復元できるはずです。

既知の問題:直接マップ => カテゴリマップへの遷移は解決していません(ただし、これはエッジケースのはずです)が、それ以外はすべて改善されているはずです。

「いいね!」 1

明日テストしますが、ユーザーから、米国の場所が「市、国、州」の形式で表示されるようになったと指摘されました。例:アトランタ、米国、ジョージア州

クイックテスト:この新しいアップデートで、マップ上のすべてのマーカーが表示されることを確認できました。どうもありがとうございました。:slight_smile:

現時点では、壊れた遷移を回避するために、ユーザーにカテゴリマップ間を移動しないようにアドバイスします。

「いいね!」 1

最近の変更はマップのポピュレーションに関連するものだけです。

国/州の順序は最新バージョンではありません。ここ数日のバージョンでそれを見ていないように記憶しているので、言及しました。

「いいね!」 1

@packman 現時点では、以前の問題を修正しました。

これはテストでカバーされるようになりました。

この件については、バンカーから救出してくれた@davidに改めて感謝します。

「いいね!」 3

@merefield こんにちは、ちょっとした情報ですが、modify_user_params メソッドはしばらくの間非推奨となっており、Discourse がバージョン 3.2.0.beta1-dev になったため、まもなく削除される予定です。

discourse-locations/lib/users_map.rb
35:  def modify_user_params(attrs)

お時間のある時に、users_controller_update_user_params モディファイアを使用するように更新していただけますでしょうか?よろしくお願いします!

「いいね!」 1

承知しました。追加情報ありがとうございます。今週中に確認します。

「いいね!」 1

ブレイクさん、完了しました。レビュー担当者として追加してもよろしいですか?

「いいね!」 3

素晴らしい、ありがとうございます!

「いいね!」 2

ありがとうございます、マージされました!

「いいね!」 1

申し訳ありませんが、バグの報告があります。

現在最新版を実行しており、本日アップデートしました:3.2.0.beta1-dev (7ca5ee6cd2)

2週間前のアップデートでは問題なかったのですが、カスタムフィールドを持つユーザープロファイルを編集できなくなるという奇妙な問題が発生しています。

編集したいユーザーはこちらです: https://example.com/u/username/preferences/profile

ページは正常に読み込まれます:

保存をクリックすると、エラーが発生します:

この問題が発覚したのは、他のウェブサイトでクラブメンバーがアップグレードした際に「membership」カスタムフィールドが更新されるのですが、その際にDiscourse APIエンドポイントが500エラーを返していたためです。

Postmanで再現しようとしても、同様の問題が発生します:

Discourseのログには以下が表示されます:

そして:

plugins/discourse-locations/plugin.rb:188:in `block (2 levels) in activate!'

lib/discourse_plugin_registry.rb:293:in `apply_modifier'

app/controllers/users_controller.rb:2036:in `user_params'

app/controllers/users_controller.rb:198:in `update'

actionpack (7.0.7) lib/action_controller/metal/basic_implicit_render.rb:6:in `send_action'

actionpack (7.0.7) lib/abstract_controller/base.rb:215:in `process_action'

actionpack (7.0.7) lib/action_controller/metal/rendering.rb:165:in `process_action'

actionpack (7.0.7) lib/abstract_controller/callbacks.rb:234:in `block in process_action'

activesupport (7.0.7) lib/active_support/callbacks.rb:118:in `block in run_callbacks'

app/controllers/application_controller.rb:420:in `block in with_resolved_locale'

i18n (1.14.1) lib/i18n.rb:322:in `with_locale'

app/controllers/application_controller.rb:420:in `with_resolved_locale'

activesupport (7.0.7) lib/active_support/callbacks.rb:127:in `block in run_callbacks'

activesupport (7.0.7) lib/active_support/callbacks.rb:138:in `run_callbacks'

actionpack (7.0.7) lib/abstract_controller/callbacks.rb:233:in `process_action'

actionpack (7.0.7) lib/action_controller/metal/rescue.rb:23:in `process_action'

actionpack (7.0.7) lib/action_controller/metal/instrumentation.rb:67:in `block in process_action'

activesupport (7.0.7) lib/active_support/notifications.rb:206:in `block in instrument'

activesupport (7.0.7) lib/active_support/notifications/instrumenter.rb:24:in `instrument'

activesupport (7.0.7) lib/active_support/notifications.rb:206:in `instrument'

actionpack (7.0.7) lib/action_controller/metal/instrumentation.rb:66:in `process_action'

actionpack (7.0.7) lib/action_controller/metal/params_wrapper.rb:259:in `process_action'

activerecord (7.0.7) lib/active_record/railties/controller_runtime.rb:27:in `process_action'

actionpack (7.0.7) lib/abstract_controller/base.rb:151:in `process'

actionview (7.0.7) lib/action_view/rendering.rb:39:in `process'

actionpack (7.0.7) lib/action_controller/metal.rb:188:in `dispatch'

actionpack (7.0.7) lib/action_controller/metal.rb:251:in `dispatch'

actionpack (7.0.7) lib/action_dispatch/routing/route_set.rb:49:in `dispatch'

actionpack (7.0.7) lib/action_dispatch/routing/route_set.rb:32:in `serve'

actionpack (7.0.7) lib/action_dispatch/journey/router.rb:50:in `block in serve'

actionpack (7.0.7) lib/action_dispatch/journey/router.rb:32:in `each'

actionpack (7.0.7) lib/action_dispatch/journey/router.rb:32:in `serve'

actionpack (7.0.7) lib/action_dispatch/routing/route_set.rb:852:in `call'

lib/middleware/omniauth_bypass_middleware.rb:74:in `call'

rack (2.2.8) lib/rack/tempfile_reaper.rb:15:in `call'

rack (2.2.8) lib/rack/conditional_get.rb:40:in `call'

rack (2.2.8) lib/rack/head.rb:12:in `call'

actionpack (7.0.7) lib/action_dispatch/http/permissions_policy.rb:38:in `call'

lib/content_security_policy/middleware.rb:12:in `call'

lib/middleware/anonymous_cache.rb:389:in `call'

lib/middleware/gtm_script_nonce_injector.rb:10:in `call'

rack (2.2.8) lib/rack/session/abstract/id.rb:266:in `context'

rack (2.2.8) lib/rack/session/abstract/id.rb:260:in `call'

actionpack (7.0.7) lib/action_dispatch/middleware/cookies.rb:704:in `call'

actionpack (7.0.7) lib/action_dispatch/middleware/callbacks.rb:27:in `block in call'

activesupport (7.0.7) lib/active_support/callbacks.rb:99:in `run_callbacks'

actionpack (7.0.7) lib/action_dispatch/middleware/callbacks.rb:26:in `call'

actionpack (7.0.7) lib/action_dispatch/middleware/debug_exceptions.rb:28:in `call'

actionpack (7.0.7) lib/action_dispatch/middleware/show_exceptions.rb:29:in `call'

logster (2.12.2) lib/logster/middleware/reporter.rb:43:in `call'

railties (7.0.7) lib/rails/rack/logger.rb:40:in `call_app'

railties (7.0.7) lib/rails/rack/logger.rb:27:in `call'

config/initializers/100-quiet_logger.rb:20:in `call'

config/initializers/100-silence_logger.rb:29:in `call'

actionpack (7.0.7) lib/action_dispatch/middleware/remote_ip.rb:93:in `call'

actionpack (7.0.7) lib/action_dispatch/middleware/request_id.rb:26:in `call'

lib/middleware/enforce_hostname.rb:24:in `call'

rack (2.2.8) lib/rack/method_override.rb:24:in `call'

actionpack (7.0.7) lib/action_dispatch/middleware/executor.rb:14:in `call'

rack (2.2.8) lib/rack/sendfile.rb:110:in `call'

actionpack (7.0.7) lib/action_dispatch/middleware/host_authorization.rb:131:in `call'

message_bus (4.3.7) lib/message_bus/rack/middleware.rb:60:in `call'

lib/middleware/request_tracker.rb:233:in `call'

railties (7.0.7) lib/rails/engine.rb:530:in `call'

railties (7.0.7) lib/rails/railtie.rb:226:in `public_send'

railties (7.0.7) lib/rails/railtie.rb:226:in `method_missing'

rack (2.2.8) lib/rack/urlmap.rb:74:in `block in call'

rack (2.2.8) lib/rack/urlmap.rb:58:in `each'

rack (2.2.8) lib/rack/urlmap.rb:58:in `call'

unicorn (6.1.0) lib/unicorn/http_server.rb:634:in `process_client'

unicorn (6.1.0) lib/unicorn/http_server.rb:739:in `worker_loop'

unicorn (6.1.0) lib/unicorn/http_server.rb:547:in `spawn_missing_workers'

unicorn (6.1.0) lib/unicorn/http_server.rb:143:in `start'

unicorn (6.1.0) bin/unicorn:128:in `<top (required)>'

vendor/bundle/ruby/3.2.0/bin/unicorn:25:in `load'

vendor/bundle/ruby/3.2.0/bin/unicorn:25:in `<main>'

奇妙なことに、私自身のプロフィールは編集でき、カスタムユーザーフィールドも問題なく更新できます。

しかし、他のユーザーを編集することはできなくなり、systemもAPI経由で更新を実行できなくなりました。

Locationsプラグインを無効にすると、すべて正常に機能します:

「いいね!」 3

@Richie レポートありがとうございます。

再現できました。

場所がない場合にのみ発生するようです。

このコードは、コアの非推奨のため最近リファクタリングされましたが、(かなり大きな)ケースを見落としていました。
すぐに修正します。

「いいね!」 5

ああ、だから私のプロフィールを編集できたのは、場所に印がついているからかもしれないんですね :slight_smile:

心配いりません、引き続きのサポートありがとうございます :slight_smile:

これで修正されたはずです。

「いいね!」 8

今朝のアップデートで、HTTP/500エラーはなくなりました。

迅速な修正をありがとうございました、Robert :smiley: :clap:

「いいね!」 3