Project

General

Profile

Actions

Refactor #9142

closed

Unify validate_regexp and validate_list for lookup value/key

Added by Daniel Lobato Garcia about 9 years ago. Updated almost 9 years ago.

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

Description

There is some 100% duplicated code in lookup value/key that should be DRYed as soon as possible, perhaps take out such logic to a concern about validating ERB.

https://github.com/theforeman/foreman/pull/2080/files#r23497738 <- take into account if you're fixing this.


Related issues 2 (0 open2 closed)

Related to Foreman - Bug #8052: Smart class parameters with ERB snippets are broken for other that String typesClosedMarek Hulán10/23/2014Actions
Is duplicate of Foreman - Refactor #10232: move validations and casting out of lookup keys and values into single-responsibility classesClosedTom Caspy04/22/2015Actions
Actions #1

Updated by Dominic Cleal about 9 years ago

  • Category set to Parameters
Actions #2

Updated by Dominic Cleal about 9 years ago

  • Related to Bug #8052: Smart class parameters with ERB snippets are broken for other that String types added
Actions #3

Updated by Dominic Cleal almost 9 years ago

  • Is duplicate of Refactor #10232: move validations and casting out of lookup keys and values into single-responsibility classes added
Actions #4

Updated by Dominic Cleal almost 9 years ago

  • Status changed from New to Duplicate

#10232 seems to have done this.

Actions

Also available in: Atom PDF