Project

General

Profile

Actions

Refactor #9486

closed

pool.rb lazy_accessor tech debt

Added by Thomas McKay about 9 years ago. Updated almost 4 years ago.

Status:
Rejected
Priority:
Normal
Assignee:
Category:
-
Target version:
Branch:
Difficulty:
Triaged:
Yes
Fixed in Releases:
Found in Releases:

Description

File is overwhelmingly bad, having grown over the years into a big pile of crud. @jsherrill suggested updating things to properly use lazy_accessor. A lot of these attributes will/should move into the model directly for scoped search as well.

       lazy_accessor :remote_data, :pool_derived, :product_name, ...
                      :initializer => (lambda do |pool|
                        attributes = {}
                        pool.remote_data['attributes'].map do |attribute|
                          attributes[attribute[:name].camel_case] = attribute['value']
                        end
                      end)
Actions #1

Updated by Eric Helms about 9 years ago

  • Tracker changed from Bug to Refactor
  • translation missing: en.field_release set to 31
  • Triaged changed from No to Yes
Actions #2

Updated by Eric Helms about 9 years ago

  • translation missing: en.field_release deleted (31)
Actions #3

Updated by Eric Helms over 8 years ago

  • translation missing: en.field_release set to 114
Actions #4

Updated by Thomas McKay over 5 years ago

  • Status changed from Assigned to Rejected

Old issue, a lot changed since then

Actions #5

Updated by Jonathon Turel almost 4 years ago

  • Target version changed from Katello Backlog to Katello Recycle Bin
Actions

Also available in: Atom PDF