Actions
Refactor #6596
closed'type' field in Katello::System incorrectly triggers STI for Host::Managed
Pull request:
Fixed in Releases:
Found in Releases:
Description
I'm not sure why self.abstract_class = true in Katello::Model is not disabling STI.
Updated by Joseph Magen over 10 years ago
if the type of Katello::System instance is 'Host', then when working the rails console, the class type returns Host::Managed
a = Katello::System.first
=> Host::Manageda.class
Updated by Joseph Magen over 10 years ago
- Subject changed from refactor 'type' field in Katello::System that incorrect triggers STI to 'type' field in Katello::System incorrectly triggers STI for Host::Managed
Updated by Ohad Levy over 10 years ago
the main concern here, is that once you initialize a katello::system object, from rails, its actually a host::managed object.
if you perform any action on it (like changing an environment_id) it would change the host table and not the katello systems table!!
Updated by The Foreman Bot over 10 years ago
- Status changed from New to Ready For Testing
- Target version changed from 56 to 49
- Pull request https://github.com/Katello/katello/pull/4421 added
- Pull request deleted (
)
Updated by Joseph Magen over 10 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset katello|19b8185e435b4d2b0df8e430bec33f98be8242bf.
Updated by Eric Helms over 10 years ago
- Translation missing: en.field_release set to 13
Actions