🛠️ Refactor: Endpoints - Explicit Study Key Strategy

src/imednet/core/endpoint/mixins/params.py

@@ -6,7 +6,6 @@
     DefaultParamProcessor,
     KeepStudyKeyStrategy,
     OptionalStudyKeyStrategy,
-    PopStudyKeyStrategy,
     StudyKeyStrategy,
 )
 from imednet.core.endpoint.structs import ParamState
@@ -20,8 +19,6 @@ class ParamMixin:
     """Mixin for handling endpoint parameters and filters."""
 
     requires_study_key: bool = True
-    _pop_study_filter: bool = False
-    _missing_study_exception: type[Exception] = ValueError
 
     PARAM_PROCESSOR: Optional[ParamProcessor] = None
     PARAM_PROCESSOR_CLS: type[ParamProcessor] = DefaultParamProcessor
@@ -38,11 +35,8 @@ def study_key_strategy(self) -> StudyKeyStrategy:
         if self.STUDY_KEY_STRATEGY:
             return self.STUDY_KEY_STRATEGY
 
-        # Backward compatibility logic
         if self.requires_study_key:
-            if self._pop_study_filter:
-                return PopStudyKeyStrategy(exception_cls=self._missing_study_exception)
-            return KeepStudyKeyStrategy(exception_cls=self._missing_study_exception)
+            return KeepStudyKeyStrategy()
         return OptionalStudyKeyStrategy()
 
     @property

src/imednet/core/endpoint/protocols.py

@@ -17,8 +17,6 @@ class EndpointProtocol(Protocol):
     _enable_cache: bool
     requires_study_key: bool
     PAGE_SIZE: int
-    _pop_study_filter: bool
-    _missing_study_exception: type[Exception]
 
     def _auto_filter(self, filters: Dict[str, Any]) -> Dict[str, Any]:
         """Apply automatic filters (e.g., default study key)."""

src/imednet/endpoints/codings.py

@@ -3,6 +3,7 @@
 from imednet.core.endpoint.base import GenericEndpoint
 from imednet.core.endpoint.edc_mixin import EdcEndpointMixin
 from imednet.core.endpoint.mixins import FilterGetEndpointMixin, ListEndpointMixin
+from imednet.core.endpoint.strategies import PopStudyKeyStrategy
 from imednet.models.codings import Coding
 
 
@@ -21,5 +22,4 @@ class CodingsEndpoint(
     PATH = "codings"
     MODEL = Coding
     _id_param = "codingId"
-    _pop_study_filter = True
-    _missing_study_exception = KeyError
+    STUDY_KEY_STRATEGY = PopStudyKeyStrategy(exception_cls=KeyError)

src/imednet/endpoints/forms.py

@@ -3,6 +3,7 @@
 from imednet.core.endpoint.base import GenericEndpoint
 from imednet.core.endpoint.edc_mixin import EdcEndpointMixin
 from imednet.core.endpoint.mixins import FilterGetEndpointMixin, ListEndpointMixin
+from imednet.core.endpoint.strategies import PopStudyKeyStrategy
 from imednet.models.forms import Form
 
 
@@ -22,6 +23,5 @@ class FormsEndpoint(
     MODEL = Form
     _id_param = "formId"
     _enable_cache = True
-    _pop_study_filter = True
-    _missing_study_exception = KeyError
+    STUDY_KEY_STRATEGY = PopStudyKeyStrategy(exception_cls=KeyError)
     PAGE_SIZE = 500

src/imednet/endpoints/intervals.py

@@ -3,6 +3,7 @@
 from imednet.core.endpoint.base import GenericEndpoint
 from imednet.core.endpoint.edc_mixin import EdcEndpointMixin
 from imednet.core.endpoint.mixins import FilterGetEndpointMixin, ListEndpointMixin
+from imednet.core.endpoint.strategies import PopStudyKeyStrategy
 from imednet.models.intervals import Interval
 
 
@@ -22,6 +23,5 @@ class IntervalsEndpoint(
     MODEL = Interval
     _id_param = "intervalId"
     _enable_cache = True
-    _pop_study_filter = True
-    _missing_study_exception = KeyError
+    STUDY_KEY_STRATEGY = PopStudyKeyStrategy(exception_cls=KeyError)
     PAGE_SIZE = 500

src/imednet/endpoints/records.py

@@ -27,7 +27,6 @@ class RecordsEndpoint(
     PATH = "records"
     MODEL = Record
     _id_param = "recordId"
-    _pop_study_filter = False
     PARAM_PROCESSOR = MappingParamProcessor({"record_data_filter": "recordDataFilter"})
 
     def create(

src/imednet/endpoints/sites.py

@@ -3,6 +3,7 @@
 from imednet.core.endpoint.base import GenericEndpoint
 from imednet.core.endpoint.edc_mixin import EdcEndpointMixin
 from imednet.core.endpoint.mixins import FilterGetEndpointMixin, ListEndpointMixin
+from imednet.core.endpoint.strategies import PopStudyKeyStrategy
 from imednet.models.sites import Site
 
 
@@ -21,5 +22,4 @@ class SitesEndpoint(
     PATH = "sites"
     MODEL = Site
     _id_param = "siteId"
-    _pop_study_filter = True
-    _missing_study_exception = KeyError
+    STUDY_KEY_STRATEGY = PopStudyKeyStrategy(exception_cls=KeyError)

src/imednet/endpoints/users.py

@@ -3,7 +3,7 @@
 from imednet.core.endpoint.base import GenericEndpoint
 from imednet.core.endpoint.edc_mixin import EdcEndpointMixin
 from imednet.core.endpoint.mixins import FilterGetEndpointMixin, ListEndpointMixin
-from imednet.core.endpoint.strategies import MappingParamProcessor
+from imednet.core.endpoint.strategies import MappingParamProcessor, PopStudyKeyStrategy
 from imednet.models.users import User
 
 
@@ -22,7 +22,7 @@ class UsersEndpoint(
     PATH = "users"
     MODEL = User
     _id_param = "userId"
-    _pop_study_filter = True
+    STUDY_KEY_STRATEGY = PopStudyKeyStrategy()
     PARAM_PROCESSOR = MappingParamProcessor(
         mapping={"include_inactive": "includeInactive"},
         defaults={"include_inactive": False},

src/imednet/endpoints/variables.py

@@ -3,6 +3,7 @@
 from imednet.core.endpoint.base import GenericEndpoint
 from imednet.core.endpoint.edc_mixin import EdcEndpointMixin
 from imednet.core.endpoint.mixins import FilterGetEndpointMixin, ListEndpointMixin
+from imednet.core.endpoint.strategies import PopStudyKeyStrategy
 from imednet.models.variables import Variable
 
 
@@ -22,6 +23,5 @@ class VariablesEndpoint(
     MODEL = Variable
     _id_param = "variableId"
     _enable_cache = True
-    _pop_study_filter = True
-    _missing_study_exception = KeyError
+    STUDY_KEY_STRATEGY = PopStudyKeyStrategy(exception_cls=KeyError)
     PAGE_SIZE = 500