Bug #9592
closedUI should report errors to the user when External AD UserGroup addition fails
Description
Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1195637
Description of problem:
UI should report errors to the user when External AD UserGroup addition fails.
Processing by UsergroupsController#update as HTML
Parameters: {"utf8"=>"✓", "authenticity_token"=>"w4GDcBCBEdfi+TG0Lg7cPljNH4/96go+SS0giOfYNdY=", "usergroup"=>{"name"=>"foreman", "user_ids"=>["", "5", "", "4", "", "6", ""], "admin"=>"0", "role_ids"=>["", "11"], "external_usergroups_attributes"=>{"0"=>{"_destroy"=>"false", "name"=>"kedargrp", "auth_source_id"=>"4", "id"=>"8"}, "1"=>{"_destroy"=>"false", "name"=>"foobargroup", "auth_source_id"=>"3", "id"=>"7"}, "new_external_usergroups"=>{"_destroy"=>"false", "name"=>"", "auth_source_id"=>"3"}, "new_1424766851040"=>{"_destroy"=>"false", "name"=>"hdhdhd", "auth_source_id"=>"3"}}}, "id"=>"3-foreman"}
Failed to save: Name is not found in the authentication source
OR
Processing by UsergroupsController#update as HTML
Parameters: {"utf8"=>"✓", "authenticity_token"=>"w4GDcBCBEdfi+TG0Lg7cPljNH4/96go+SS0giOfYNdY=", "usergroup"=>{"name"=>"foreman", "user_ids"=>["", "5", "", "4", "", "6", ""], "admin"=>"0", "role_ids"=>["", "11"], "external_usergroups_attributes"=>{"0"=>{"_destroy"=>"false", "name"=>"kedargrp", "auth_source_id"=>"4", "id"=>"8"}, "1"=>{"_destroy"=>"false", "name"=>"foobargroup", "auth_source_id"=>"3", "id"=>"7"}, "new_external_usergroups"=>{"_destroy"=>"false", "name"=>"", "auth_source_id"=>"3"}, "new_1424767137133"=>{"_destroy"=>"false", "name"=>"foobargroup", "auth_source_id"=>"3"}}}, "id"=>"3-foreman"}
Failed to save: Name has already been taken
Version-Release number of selected component (if applicable):
sat6.1 Beta snap3
How reproducible:
Steps to Reproduce:
1. create usergroup
2. add an invalid external AD UserGroup.
3.
Actual results:
No error message seen on the UI.
foreman/production.log shows
a) "Failed to save: Name is not found in the authentication source"
b) Failed to save: Name has already been taken
Expected results:
Shouls display the actual error on the UI,
a) "Name is not found in the authentication source"
b) Name has already been taken
Additional info:
Updated by Marek Hulán over 9 years ago
- Category set to Web Interface
- Status changed from New to Assigned
- Priority changed from High to Normal
Updated by The Foreman Bot over 9 years ago
- Status changed from Assigned to Ready For Testing
- Pull request https://github.com/theforeman/foreman/pull/2216 added
- Pull request deleted (
)
Updated by Marek Hulán over 9 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset 8e170d2186edc805e440e587438a4cf8207facd9.
Updated by Dominic Cleal over 9 years ago
- Translation missing: en.field_release set to 28