Project

General

Profile

Bug #21545

Role cloning treats taxonomies differently in UI and API

Added by Ondřej Pražák almost 2 years ago. Updated 9 months ago.

Status:
Closed
Priority:
Normal
Category:
Users
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.

Associated revisions

Revision e61362d1 (diff)
Added by Dominik Hlavac Duran 9 months 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 almost 2 years ago

  • Target version set to 115
  • Category set to Users
  • 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 11 months ago

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

#3 Updated by The Foreman Bot 11 months ago

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

#4 Updated by Lukas Zapletal 9 months ago

  • Fixed in Releases 1.21.0 added

#5 Updated by Dominik Hlavac Duran 9 months ago

  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF