Project

General

Profile

Actions

Bug #12577

closed

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

Added by Jitendra Yejare over 8 years ago. Updated almost 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Parameters
Target version:
Difficulty:
Triaged:
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.


Files

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

Related issues 1 (0 open1 closed)

Related to Foreman - Bug #14183: Smart variable not showing whitespace validationClosedOri Rabin03/14/2016Actions
Actions #1

Updated by Ori Rabin over 8 years ago

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

Updated by The Foreman Bot over 8 years ago

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

Updated by Ori Rabin over 8 years ago

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

Updated by Dominic Cleal over 8 years ago

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

Updated by The Foreman Bot about 8 years ago

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

Updated by Ori Rabin about 8 years ago

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

Updated by Ori Rabin about 8 years ago

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

Also available in: Atom PDF