Project

General

Profile

Actions

Refactor #17939

closed

Refactor Classification object to reduce its complaexity

Added by Shimon Shtein about 7 years ago. Updated over 5 years ago.

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

Description

It's already done as part of https://github.com/theforeman/foreman/pull/3701, but I want to extract the Classification objects modification from there, and make it a standalone PR.

Main files:
ValuesHashQuery: query object that is responsible to return values hash for a given set of keys
Classification concern: A concern that adds two scopes (values_hash, inherited_values) that are implemented as calls to the ValuesHashQuery object.
ClassificationResult: Class that encapsulates rendering of values for a given key. Contains an option to show unrendered values too.

Actions #1

Updated by The Foreman Bot about 7 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/4163 added
Actions #2

Updated by Tomer Brisker over 5 years ago

  • Target version deleted (1.10.3)
  • Triaged set to No
Actions #3

Updated by The Foreman Bot over 5 years ago

  • Status changed from Ready For Testing to New
  • Pull request deleted (https://github.com/theforeman/foreman/pull/4163)
Actions #4

Updated by Shimon Shtein over 5 years ago

  • Status changed from New to Closed
Actions #5

Updated by Tomer Brisker over 5 years ago

  • Status changed from Closed to Rejected
Actions

Also available in: Atom PDF