Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add case insensitivity when validating uniqueness of name of new group/role #17197

Merged

Conversation

hstastna
Copy link

@hstastna hstastna commented Mar 23, 2018

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1537171

Add case insensitivity when validating uniqueness of the name of a new group within a region.
Add case insensitivity when validating uniqueness of the name of a new role + little refactoring.


After: (different capitalization but the same names => flash messages)
group_after
role_after

@hstastna hstastna force-pushed the Duplicate_RBAC_Role_Group_names_capitalization branch from dd99d20 to 5dcb11c Compare March 23, 2018 22:53
Hilda Stastna added 2 commits March 23, 2018 23:54
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1537171

Add case insensitivity when validating uniqueness of name of a new group within a region.
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1537171

Add case insensitivity when validating uniqueness of the name of a new role,
when creating a new role under Configuration > Access Control > Roles.
@hstastna hstastna force-pushed the Duplicate_RBAC_Role_Group_names_capitalization branch from 5dcb11c to 4ddb142 Compare March 23, 2018 22:54
@miq-bot
Copy link
Member

miq-bot commented Mar 23, 2018

Checked commits hstastna/manageiq@db23d63~...4ddb142 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
2 files checked, 0 offenses detected
Everything looks fine. 🍰

@hstastna
Copy link
Author

@miq-bot add_label bug

@miq-bot miq-bot added the bug label Mar 23, 2018
@lpichler
Copy link
Contributor

@miq-bot assign @gtanzillo

@bdunne bdunne merged commit ad57f99 into ManageIQ:master Mar 26, 2018
@bdunne bdunne added this to the Sprint 82 Ending Mar 26, 2018 milestone Mar 26, 2018
@bdunne bdunne assigned bdunne and unassigned gtanzillo Mar 26, 2018
simaishi pushed a commit that referenced this pull request Mar 27, 2018
…es_capitalization

Add case insensitivity when validating uniqueness of name of new group/role
(cherry picked from commit ad57f99)

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1561076
@simaishi
Copy link
Contributor

Gaprindashvili backport details:

$ git log -1
commit 5e8f0c4d775db8d1d69a9ecd8f80a3018ba9d5f4
Author: Brandon Dunne <brandondunne@hotmail.com>
Date:   Mon Mar 26 16:11:17 2018 -0400

    Merge pull request #17197 from hstastna/Duplicate_RBAC_Role_Group_names_capitalization
    
    Add case insensitivity when validating uniqueness of name of new group/role
    (cherry picked from commit ad57f99c011037384f8415ef6ae8ad41f75d81a8)
    
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1561076

simaishi pushed a commit that referenced this pull request Apr 9, 2018
…es_capitalization

Add case insensitivity when validating uniqueness of name of new group/role
(cherry picked from commit ad57f99)

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1561077
@simaishi
Copy link
Contributor

simaishi commented Apr 9, 2018

Fine backport details:

$ git log -1
commit 1f6387c48361d05194a2423fe0fc53cf364b592d
Author: Brandon Dunne <brandondunne@hotmail.com>
Date:   Mon Mar 26 16:11:17 2018 -0400

    Merge pull request #17197 from hstastna/Duplicate_RBAC_Role_Group_names_capitalization
    
    Add case insensitivity when validating uniqueness of name of new group/role
    (cherry picked from commit ad57f99c011037384f8415ef6ae8ad41f75d81a8)
    
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1561077

d-m-u pushed a commit to d-m-u/manageiq that referenced this pull request Jun 6, 2018
…roup_names_capitalization

Add case insensitivity when validating uniqueness of name of new group/role
(cherry picked from commit ad57f99)

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1561077
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants