[openstreetmap/openstreetmap-website] Rework UI to hide/delete/suspend users (PR #6564)

Pablo Brasero notifications at github.com
Thu Dec 4 13:47:17 UTC 2025


pablobm left a comment (openstreetmap/openstreetmap-website#6564)

Ah, you are right about the provenance of `@user`, I got a bit mixed up when putting together the writeup. But I also missed to include a detail: it's `current_user&.administrator?` that is the problem. In this test, the administrator is the current user. The user is suspended, so it's not `visible?` and only an administrator can see the heatmap, therefore a `current_user` is required, missing, and `@heatmap_data` is not set.

Also, either way the controller allows for `@heatmap_data` not to be set.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6564#issuecomment-3612351066
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/6564/c3612351066 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20251204/a27c631f/attachment-0001.htm>


More information about the rails-dev mailing list