Refactor #19464
Move Dashboard::Manager out of app/registries/ to permit autoloading
Team Backlog:
Fixed in Releases:
Found in Releases:
Description
Dashbaord::Manager has always been excluded from autoloading as it kept a list of known widgets in a class variable. In #10927, it was moved to app/registries/ to preserve this status and keep it from being autoloaded.
It is possible to rewrite this class in an autoload-friendly manner, where the list of builtin widgets is hardcoded inside the class and then the total list of default widgets is retrieved from Foreman::Plugin when required, instead of plugins appending to the dashboard class variable.
This would:
- Allow the source file to be kept in app/services/, easier for people to find
- Allow the class to be reloaded in development, so changes to builtin widgets are immediately visible
- Allow initialisation to be removed from config/initializers/, speeding up the application load time
- Make the class more testable, by removing modifications to class variables
Related issues
Associated revisions
History
#1
Updated by Dominic Cleal over 1 year ago
Updated by - Related to Bug #10927: Resetting dashboard to default widgets adds no widgets added
#2
Updated by The Foreman Bot over 1 year ago
Updated by - Status changed from Assigned to Ready For Testing
- Pull request https://github.com/theforeman/foreman/pull/4506 added
#3
Updated by Dominic Cleal over 1 year ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset d3f474050ae0b1f3866b68dc14b3f984db2d03c0.
#4
Updated by Marek Hulán about 1 year ago
Updated by - Legacy Backlogs Release (now unused) set to 240
fixes #19464 - rewrite Dashboard::Manager to support autoloading
Replaces class-level widget storage with a static list of built-in
widgets, then appends the registered plugin widgets to get the full
default list of widgets. As a result, the class can safely be reloaded
when modified and is more testable as class-level state is removed.