Remove caching for HideActionHook results.

Review Request #13509 — Created Jan. 23, 2024 and submitted

Information

Review Board
release-6.x

Reviewers

The action implementation was caching the result of evaluating all of
the registered HideActionHook instances, but that ends up potentially
giving bad results. If an extension is being enabled or disabled, the
cached value can be incorrect. This is particularly noticable in the
devserver, but could affect production deployments too.

Ran unit tests.

Summary ID
Remove caching for HideActionHook results.
The action implementation was caching the result of evaluating all of the registered HideActionHook instances, but that ends up potentially giving bad results. If an extension is being enabled or disabled, the cached value can be incorrect. This is particularly noticable in the devserver, but could affect production deployments too. Testing Done: Ran unit tests.
f88bd104c7d691b5f645e973d6c971eca9370e0f
maubin
  1. Ship It!
  2. 
      
david
Review request changed
Status:
Completed
Change Summary:
Pushed to release-6.x (52d18b7)