Project

General

Profile

Actions

Refactor #261

closed

switch parameters to use a single reference id

Added by Paul Kelly almost 14 years ago. Updated almost 14 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Rails
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

Parameters use multiple id fields to refer to their associated object. This is not required.

Actions #1

Updated by Paul Kelly almost 14 years ago

  • Subject changed from switch parameters to use STI to switch parameters to use a single reference id
Actions #2

Updated by Paul Kelly almost 14 years ago

  • Category set to Rails
  • Status changed from New to Assigned

Ohad, I am unclear as to whether I have misunderstood your parameters implementation.
There are host parameters for all hosts for these keys ["hostmode", "hosttype", "longsitename", "puppetmaster", "sitename"]. Are these left over from an experiment or are we migrating to use these as the source of that information. I would also expect some of these to be domainparameters.

Actions #3

Updated by Paul Kelly almost 14 years ago

  • Status changed from Assigned to Feedback
  • Branch set to refactor/261-parameters-use-reference_id

Rebased against develop
Tests pass.

Actions #4

Updated by Paul Kelly almost 14 years ago

Refactored concrete class validations back into the abstract base class.
Removed obsolete controllers and their tests
Fixed some tests so as to always use a concrete class
Tests pass

Actions #5

Updated by Ohad Levy almost 14 years ago

  • Target version set to 0.1-6
Actions #6

Updated by Paul Kelly almost 14 years ago

Rebased
Fixed bug where hosts were marked :dependent => :destroy. Ouch!

Actions #7

Updated by Paul Kelly almost 14 years ago

There were issues with the uniqueness constraint that are now fixed.
Added some tests, to be sure.

Actions #8

Updated by Paul Kelly almost 14 years ago

  • Status changed from Feedback to Ready For Testing
  • % Done changed from 0 to 100
Actions #9

Updated by Ohad Levy almost 14 years ago

  • Status changed from Ready For Testing to Closed
Actions

Also available in: Atom PDF