Project

General

Profile

Bug #21545

Role cloning treats taxonomies differently in UI and API

Added by Ondřej Pražák over 4 years ago. Updated almost 2 years ago.

Status:
Closed
Priority:
Normal
Category:
Users, Roles and Permissions
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

Description of problem:
From Hammer CLI, if you clone a role that has taxonomies, then after cloning the same taxonomies are copied over to the cloned role. When clonning in UI, the taxonomies are not selected by default, so role is clonned without taxonomies. We should unify the behaviour.

How reproducible:
Always

Steps to Reproduce:
1. Create a role with org A and location A
2. Clone above role from hammer CLI without providing any org and location id's
  1. hammer role clone --id 45 --new-name clied_cloned
    3. Clone the same role from UI - just click 'clone' and submit without making any custom changes.

Actual results:
The cloned role using Hammer has taxonomies copied from its parent role while role clonned in UI has no taxonomies.

Expected results:
Cloned role should have same taxonomies regardless of whe way they are clonned.


Related issues

Related to Foreman - Bug #32026: Saving invalid cloned role ends up with Internal server errorClosed

Associated revisions

Revision e61362d1 (diff)
Added by Dominik Hlavac Duran over 3 years ago

Fixes #21545 - Unified role cloning via UI and hammer

Unified role clonning. Now role is cloned via UI and Hammer with
original role taxonomies

History

#1 Updated by Ondřej Pražák over 4 years ago

  • Target version set to 115
  • Category set to 218
  • Subject changed from Role cloning treats taxonomies differently in UI and API to Role cloning treats taxonomies differently in UI and API

#2 Updated by Dominik Hlavac Duran over 3 years ago

  • Triaged set to No
  • Assignee set to Dominik Hlavac Duran

#3 Updated by The Foreman Bot over 3 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/6181 added

#4 Updated by Lukas Zapletal over 3 years ago

  • Fixed in Releases 1.21.0 added

#5 Updated by Dominik Hlavac Duran over 3 years ago

  • Status changed from Ready For Testing to Closed

#6 Updated by Tomer Brisker almost 2 years ago

  • Category changed from 218 to Users, Roles and Permissions

#7 Updated by Ondřej Ezr over 1 year ago

  • Related to Bug #32026: Saving invalid cloned role ends up with Internal server error added

Also available in: Atom PDF