Project

General

Profile

Actions

Bug #17870

closed

Saving an empty matcher for smart class parameter of type boolean saves the value false

Added by Ori Rabin about 7 years ago. Updated over 5 years ago.

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

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1357049

Description of problem:
'Required' functionality is not working for smart class parameter type 'boolean'.

Version-Release number of selected component (if applicable):
Sat 6.2 snap 19.1 GA

How reproducible:
Always

Steps to Reproduce:
1. Set default value for parameter type 'boolean' to true.
2. Select required checkbox to validate that value shouldnt be blank.
3. Create a matcher with empty/blank value.
4. Attempt to submit it.

Actual results:
The parameter is submitted by setting matcher value to 'false' automatically.

Expected results:
1. The parameter should not be submitted and should throw an error 'Value can't be blank'.
2. Autoset of boolean value to false for matcher should not happen.


Related issues 1 (0 open1 closed)

Related to Foreman - Bug #18718: Saving a boolean matcher with no value changes the value to falseClosedOri Rabin02/28/2017Actions
Actions #1

Updated by Ori Rabin about 7 years ago

  • Status changed from New to Assigned
  • Assignee set to Ori Rabin
  • Target version set to 1.15.3
Actions #2

Updated by The Foreman Bot about 7 years ago

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

Updated by Ori Rabin about 7 years ago

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

Updated by Dominic Cleal about 7 years ago

  • translation missing: en.field_release set to 210
Actions #5

Updated by The Foreman Bot about 7 years ago

  • Pull request https://github.com/theforeman/foreman/pull/4345 added
Actions #6

Updated by Ori Rabin about 7 years ago

  • Related to Bug #18718: Saving a boolean matcher with no value changes the value to false added
Actions #7

Updated by Ori Rabin about 7 years ago

  • Pull request deleted (https://github.com/theforeman/foreman/pull/4345)
Actions

Also available in: Atom PDF