Project

General

Profile

Bug #12339

Deleting the existing matcher and creating the same again throws error 'Match has already been taken'.

Added by Jitendra Yejare about 6 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Parameters
Target version:
Difficulty:
Triaged:
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

Description of problem:
In Configure -> smart class parameter, deleting the existing matcher and without saving the changes, creating the same again throws error 'Match has already been taken'.

Version-Release number of selected component (if applicable):
foreman-1.11.0-0.develop.201510221551git1d0315e.el7.noarch

How reproducible:
Always

Steps to Reproduce:
1. Create matcher for puppet class parameter with some attribute and some value.
2. Save the changes.
3. Again edit the puppet class parameter to delete the matcher created in step 1.
4. Without saving changes of step 3 in parameter, create the matcher with same attribute and same value as created in step 1 and try to save the change to the parameter.

Actual results:
The error of 'Match has already been taken' is displayed and not allowing to save the changes.

Expected results:
Deleting and creating of matcher with same attribute and value should be allowed without saving the changes to parameter.

Same_Matcher_not_allowed.png View Same_Matcher_not_allowed.png 239 KB Jitendra Yejare, 10/28/2015 05:58 AM
Same matcher not allowed

Associated revisions

Revision 88e4eb7c (diff)
Added by Adam Ruzicka over 5 years ago

Fixes #12339 - Reuse to-be-deleted matchers while creating new

History

#1 Updated by Dominic Cleal about 6 years ago

  • Legacy Backlogs Release (now unused) deleted (71)

Please don't set the release field on tickets.

#2 Updated by Adam Ruzicka over 5 years ago

  • Status changed from New to Assigned
  • Assignee set to Adam Ruzicka

#3 Updated by The Foreman Bot over 5 years ago

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

#4 Updated by Adam Ruzicka over 5 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#5 Updated by Dominic Cleal over 5 years ago

  • Legacy Backlogs Release (now unused) set to 136

Also available in: Atom PDF