Skip to content

Commit e7c59cf

Browse files
committed
Removed superfluous registry
The mode_registry and the mode_permissions_ensurance_registry were always in sync, anyway. Furhtermore, instantiating entries from the latter is complicated by the fact that they are a Protocol, and mypy implicity assumes that its constructor has no arguments. This will not be the case when the WatoMode constructor gets an argument, which it will in an upcoming commit. Change-Id: I4ee1f1b378226fa4c2921dcdbbbd93e7f9a73ec8
1 parent 68a965d commit e7c59cf

File tree

4 files changed

+2
-33
lines changed

4 files changed

+2
-33
lines changed

cmk/gui/search/engines/setup.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@
6363
from cmk.gui.utils.output_funnel import output_funnel
6464
from cmk.gui.utils.roles import UserPermissions, UserPermissionSerializableConfig
6565
from cmk.gui.utils.urls import file_name_and_query_vars_from_url
66-
from cmk.gui.watolib.mode_permissions import mode_permissions_ensurance_registry
66+
from cmk.gui.watolib.mode import mode_registry
6767
from cmk.gui.watolib.rulesets import may_edit_ruleset
6868
from cmk.shared_typing.unified_search import ProviderName, UnifiedSearchResultItem
6969
from cmk.utils import paths
@@ -323,7 +323,7 @@ def _check_page_handler(self, url: str) -> bool:
323323

324324
try:
325325
if mode:
326-
mode_permissions_ensurance_registry[mode]().ensure_permissions()
326+
mode_registry[mode]().ensure_permissions()
327327
else:
328328
self._check_if_handling_page_triggers_exception(file_name)
329329
return True

cmk/gui/watolib/BUILD

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,6 @@ py_library(
6161
"mode/_base.py",
6262
"mode/_helpers.py",
6363
"mode/_registry.py",
64-
"mode_permissions.py",
6564
],
6665
imports = ["../../.."],
6766
visibility = [
@@ -214,7 +213,6 @@ py_library(
214213
"mode/_base.py",
215214
"mode/_helpers.py",
216215
"mode/_registry.py",
217-
"mode_permissions.py",
218216
# Already in other leaf targets
219217
"config_domain_core.py",
220218
"groups_io.py",

cmk/gui/watolib/mode/_registry.py

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@
66
# mypy: disable-error-code="type-arg"
77

88
from cmk.ccc.plugin_registry import Registry
9-
from cmk.gui.watolib.mode_permissions import mode_permissions_ensurance_registry
109

1110
from ._base import WatoMode
1211

@@ -15,10 +14,5 @@ class ModeRegistry(Registry[type[WatoMode]]):
1514
def plugin_name(self, instance: type[WatoMode]) -> str:
1615
return instance.name()
1716

18-
def register(self, instance: type[WatoMode]) -> type[WatoMode]:
19-
super().register(instance)
20-
mode_permissions_ensurance_registry.register(instance)
21-
return instance
22-
2317

2418
mode_registry = ModeRegistry()

cmk/gui/watolib/mode_permissions.py

Lines changed: 0 additions & 23 deletions
This file was deleted.

0 commit comments

Comments
 (0)