mirror of
https://github.com/discourse/discourse.git
synced 2026-08-06 13:08:40 +08:00
Previously, picking a group in the bulk assign modal always failed with a bodyless 500: the client only sent `username`, `group_name` was not a permitted bulk action parameter, and the bulk operation only ever resolved a `User` — so `Assigner` received `nil`, treated it as a `Group`, and crashed dereferencing it. This change routes both the bulk and single-topic paths through one `DiscourseAssign::AssigneeResolver`, makes `Assigner#assign` return a reason instead of raising for caller-supplied input, and surfaces per-topic refusals through `@errors` so a partially applied bulk assign explains itself instead of reporting success. Ref - t/188206 |
||
|---|---|---|
| .. | ||
| page_objects | ||
| assign_post_spec.rb | ||
| assign_topic_spec.rb | ||
| bulk_assign_spec.rb | ||
| core_features_spec.rb | ||
| group_assigned_spec.rb | ||
| group_preferences_spec.rb | ||
| user_menu_spec.rb | ||
| user_preferences_spec.rb | ||
| viewing_assignments_in_the_bookmark_list_spec.rb | ||