Project

General

Profile

Actions

Refactor #15828

open

Turn default value in lookup key to a lookup value

Added by Ori Rabin almost 8 years ago. Updated almost 6 years ago.

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

Description

As part of the efforts to refactor lookup values the default value in lookup key should be moved to a lookup value.
There is duplication of code that handles the default value and the rest of the values separately.
Changing the default value will allow to remove the duplication and make the rest of the refactoring simpler.


Related issues 1 (1 open0 closed)

Related to Foreman - Refactor #12822: Separate LookupValue to different classes per LookupKey typeNew12/15/2015Actions
Actions #1

Updated by Ori Rabin almost 8 years ago

  • Related to Refactor #12822: Separate LookupValue to different classes per LookupKey type added
Actions #2

Updated by Dominic Cleal almost 8 years ago

  • Category set to Parameters
Actions #3

Updated by Ivan Necas almost 8 years ago

  • Target version set to 113
Actions #4

Updated by The Foreman Bot almost 8 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Ori Rabin
  • Pull request https://github.com/theforeman/foreman/pull/3698 added
Actions #5

Updated by Ivan Necas almost 8 years ago

  • Target version changed from 113 to 1.7.0
Actions #6

Updated by Ivan Necas almost 8 years ago

  • Target version changed from 1.7.0 to 1.6.1
Actions #7

Updated by Ivan Necas over 7 years ago

  • Target version changed from 1.6.1 to 127
Actions #8

Updated by Ivan Necas over 7 years ago

  • Target version changed from 127 to 1.5.1
Actions #9

Updated by Ivan Necas over 7 years ago

  • Target version changed from 1.5.1 to 1.4.1
Actions #10

Updated by Ivan Necas over 7 years ago

  • Target version changed from 1.4.1 to 1.15.3
Actions #11

Updated by Tomer Brisker almost 6 years ago

  • Target version deleted (1.15.3)
Actions #12

Updated by Ori Rabin almost 6 years ago

  • Assignee deleted (Ori Rabin)
  • Status changed from Ready For Testing to New
  • Pull request deleted (https://github.com/theforeman/foreman/pull/3698)
Actions

Also available in: Atom PDF