Project

General

Profile

Actions

Bug #19317

closed

Plugin and dashboard widget registry auto-reloaded

Added by Dominic Cleal over 7 years ago. Updated over 6 years ago.

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

Description

Since #18445, the plugin and related classes may be reloaded, losing registrations - probably for plugin registrations, default dashboard widgets, modified menu items, and permissions.

The change to use require_dependency will allow Rails to track the constants and reload them when the source files are modified, discarding the class variable. The use of require_dependency is however required to ensure eager loading doesn't load the file twice (#18445).

The source files may need to be moved to prevent double-loading and to be excluded from autoloading.


Related issues 5 (0 open5 closed)

Related to Foreman - Bug #18445: "Cannot define multiple 'included' blocks" for audit_extensions with eager_load enabledClosedDominic Cleal02/09/2017Actions
Related to Foreman - Bug #10927: Resetting dashboard to default widgets adds no widgetsClosedDominic Cleal06/25/2015Actions
Related to Katello - Bug #19351: transient test failureClosedJustin Sherrill04/21/2017Actions
Has duplicate Foreman - Bug #19318: Plugins infrastructure don't work reliably in developmentDuplicate04/19/2017Actions
Has duplicate Katello - Bug #19290: Unable to retrieve API bindings via hammerDuplicate04/17/2017Actions
Actions #1

Updated by Dominic Cleal over 7 years ago

  • Related to Bug #18445: "Cannot define multiple 'included' blocks" for audit_extensions with eager_load enabled added
Actions #2

Updated by Tomer Brisker over 7 years ago

  • Related to Bug #10927: Resetting dashboard to default widgets adds no widgets added
Actions #3

Updated by Dominic Cleal over 7 years ago

  • Has duplicate Bug #19318: Plugins infrastructure don't work reliably in development added
Actions #4

Updated by John Mitsch over 7 years ago

  • Has duplicate Bug #19290: Unable to retrieve API bindings via hammer added
Actions #5

Updated by The Foreman Bot over 7 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/4472 added
Actions #6

Updated by Justin Sherrill over 7 years ago

  • Related to Bug #19351: transient test failure added
Actions #7

Updated by Dominic Cleal over 7 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100
Actions #8

Updated by The Foreman Bot over 7 years ago

  • Pull request https://github.com/theforeman/foreman/pull/4507 added
Actions #9

Updated by Marek Hulán over 7 years ago

  • Translation missing: en.field_release set to 240
Actions

Also available in: Atom PDF