Project

General

Profile

Bug #12577

Smart variable with space in its name allowed to create. Actually It should not.

Added by Jitendra Yejare about 4 years ago. Updated over 1 year ago.

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

Description

Description of problem:
Smart Variable with space in its name allowed to create in a puppet class. Which should not be allowed.
And also after creating the smart variable, it is not viewable, editable and deletable too.

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

How reproducible:
Always

Steps to Reproduce:
1. Try to create a smart variable with name having blank space, from Config -> Classes -> Class -> Smart Variables tab.

Actual results:
Smart Variable with space in its name allowed to create in a puppet class.

Expected results:
1.Smart Variable with space in its name should not be allowed to create in a puppet class.
2. There should be a proper error message displayed saying space not allowed in variable name.

blank space in var name.png View blank space in var name.png 212 KB Jitendra Yejare, 11/24/2015 03:47 AM
Blank%20space%20in%20var%20name

Related issues

Related to Foreman - Bug #14183: Smart variable not showing whitespace validationClosed2016-03-14

Associated revisions

Revision 54188a7c (diff)
Added by Ori Rabin almost 4 years ago

Fixes #12577 - Smart variable key should not contain space

History

#1 Updated by Ori Rabin almost 4 years ago

  • Status changed from New to Assigned
  • Assignee set to Ori Rabin

#2 Updated by The Foreman Bot almost 4 years ago

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

#3 Updated by Ori Rabin almost 4 years ago

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

#4 Updated by Dominic Cleal almost 4 years ago

  • Legacy Backlogs Release (now unused) set to 71

#5 Updated by The Foreman Bot over 3 years ago

  • Pull request https://github.com/theforeman/foreman/pull/3318 added

#6 Updated by Ori Rabin over 3 years ago

  • Related to Bug #14183: Smart variable not showing whitespace validation added

#7 Updated by Ori Rabin over 3 years ago

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

Also available in: Atom PDF