Refactor #15828

Turn default value in lookup key to a lookup value

Added by Ori Rabin 10 months ago. Updated 2 months ago.

Status:Ready For Testing
Priority:Normal
Assigned To:Ori Rabin
Category:Smart Variables
Target version:Team Ivan Iteration 7
Difficulty: Bugzilla link:
Found in release: Pull request:https://github.com/theforeman/foreman/pull/3698
Story points-
Velocity based estimate-

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

Related to Foreman - Refactor #12822: Separate LookupValue to different classes per LookupKey type New 12/15/2015

History

#1 Updated by Ori Rabin 10 months ago

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

#2 Updated by Dominic Cleal 10 months ago

  • Category set to Smart Variables

#3 Updated by Ivan Necas 10 months ago

  • Target version set to Team Ivan backlog

#4 Updated by The Foreman Bot 10 months ago

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

#5 Updated by Ivan Necas 9 months ago

  • Target version changed from Team Ivan backlog to Team Ivan Iteration 1

#6 Updated by Ivan Necas 9 months ago

  • Target version changed from Team Ivan Iteration 1 to Team Ivan Iteration 2

#7 Updated by Ivan Necas 9 months ago

  • Target version changed from Team Ivan Iteration 2 to Team Ivan Iteration 3

#8 Updated by Ivan Necas 8 months ago

  • Target version changed from Team Ivan Iteration 3 to Team Ivan Iteration 4

#9 Updated by Ivan Necas 7 months ago

  • Target version changed from Team Ivan Iteration 4 to Team Ivan Iteration 5

#10 Updated by Ivan Necas 6 months ago

  • Target version changed from Team Ivan Iteration 5 to Team Ivan Iteration 7

Also available in: Atom PDF