Bug 1479769 - Bad error message when logged user try to remove himself
Bad error message when logged user try to remove himself
Status: POST
Product: Red Hat Satellite 6
Classification: Red Hat
Component: WebUI (Show other bugs)
Unspecified Unspecified
unspecified Severity medium (vote)
: Unspecified
: --
Assigned To: satellite6-bugs
Katello QA List
: Triaged
Depends On:
  Show dependency treegraph
Reported: 2017-08-09 07:21 EDT by Dominik Hlavac Duran
Modified: 2017-12-04 05:51 EST (History)
6 users (show)

See Also:
Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
Last Closed:
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)
user deletion (56.62 KB, image/png)
2017-08-10 10:41 EDT, Marek Hulan
no flags Details

External Trackers
Tracker ID Priority Status Summary Last Updated
Foreman Issue Tracker 19479 None None None 2017-12-04 05:51 EST

  None (edit)
Description Dominik Hlavac Duran 2017-08-09 07:21:54 EDT
Description of problem: When you want to remove user which is currently logged wrong error message appears 

Version-Release number of selected component (if applicable):

How reproducible:

Steps to Reproduce:
1.Create user "tuser1"
2.log in as "tuser1"
3.try to delete yourself

Actual results:
Error message is green and shows "Successful" but error message say "you cannot remove yourself while you are logged in"

Expected results:
Error message should be red and show ERROR and not Successful

Additional info:
Comment 1 Marek Hulan 2017-08-10 10:40:59 EDT
Rox, is it right that user can see delete button for his own account? After clicking on delete button the user gets green message, see screenshot - in top right corner. I wonder if we should show the button in the first place. I'd suggest disabling with it tooltip on hover explaining why it's disabled. Thanks for your input.
Comment 2 Marek Hulan 2017-08-10 10:41 EDT
Created attachment 1311792 [details]
user deletion
Comment 5 Roxanne Hoover 2017-08-29 12:37:03 EDT
Agreed. It should show as a red error, not green.
Comment 6 Sebastian Gräßl 2017-09-18 15:07:14 EDT
This has been fixed by an unrelated change[1].

[1] https://github.com/bastilian/foreman/commit/b9df73be192a8485610f51a113b67260dc5f9e49#diff-4e05ad0d64e6100656b63ad1e78f32c5

Note You need to log in before you can comment on or make changes to this bug.