removed some unnecessary checks

Author: eadwinCode

ellar/common/interfaces/context.py

@@ -3,7 +3,6 @@
 
 from ellar.common.constants import empty_receive
 from ellar.common.types import T, TReceive, TScope, TSend
-from ellar.di import injectable, request_scope
 from starlette.requests import empty_send
 from starlette.responses import Response
 
@@ -40,7 +39,6 @@ def get_client(self) -> "WebSocket":
         """Returns WebSocket instance"""
 
 
-@injectable(scope=request_scope)
 class IHostContext(ABC, metaclass=ABCMeta):
     @abstractmethod
     def get_service_provider(self) -> "EllarInjector":
@@ -75,7 +73,6 @@ def user(self, value: t.Any) -> None:
         """Sets user identity"""
 
 
-@injectable(scope=request_scope)
 class IExecutionContext(IHostContext, ABC):
     @abstractmethod
     def get_handler(self) -> t.Callable:
@@ -136,8 +133,8 @@ def create_context_type(self, context: IHostContext) -> T:
 
 
 class IHTTPConnectionContextFactory(SubHostContextFactory[IHTTPHostContext], ABC):
-    context_typ: t.Type[IHTTPHostContext]
+    """HttpContext Factory Interface"""
 
 
 class IWebSocketContextFactory(SubHostContextFactory[IWebSocketHostContext], ABC):
-    context_type: t.Type[IWebSocketHostContext]
+    """WebsocketContext Factory Interface"""

ellar/common/params/args/base.py

@@ -50,7 +50,8 @@
 
 
 def add_get_resolver_generator(
-    param: params.ParamFieldInfo, resolver_gen: t.Type[BulkArgsResolverGenerator]
+    param: t.Type[params.ParamFieldInfo],
+    resolver_gen: t.Type[BulkArgsResolverGenerator],
 ) -> None:
     """
     Add a custom Bulk resolver generator a custom route function parameter field type
@@ -384,20 +385,6 @@ async def resolve_body(self, ctx: IExecutionContext) -> ResolverResult:
         """Body Resolver Implementation"""
         return ResolverResult({}, [], {})
 
-    def __deepcopy__(
-        self, memodict: t.Optional[t.Dict] = None
-    ) -> "EndpointArgsModel":  # pragma: no cover
-        if memodict is None:
-            memodict = {}
-        return self.__copy__(memodict)
-
-    def __copy__(
-        self, memodict: t.Optional[t.Dict] = None
-    ) -> "EndpointArgsModel":  # pragma: no cover
-        if memodict is None:
-            memodict = {}
-        return self
-
     def build_body_field(self) -> None:  # pragma: no cover
         raise NotImplementedError
 

ellar/common/params/resolvers/parameter.py

@@ -276,15 +276,8 @@ async def resolve_handle(
 
         if (
             value is None
-            or (
-                isinstance(self.model_field.field_info.resolver, FormParameterResolver)
-                and value == ""
-            )
-            or (
-                isinstance(self.model_field.field_info.resolver, FormParameterResolver)
-                and is_sequence_field(self.model_field)
-                and len(value) == 0
-            )
+            or value == ""
+            or (is_sequence_field(self.model_field) and len(value) == 0)
         ):
             if self.model_field.required:
                 return ResolverResult(

ellar/pydantic/types.py

@@ -1,15 +1,16 @@
 import types
 import typing as t
 
+from pydantic.fields import FieldInfo
 from pydantic_core import PydanticUndefined, PydanticUndefinedType
 from pydantic_core import Url as Url
 
 UnionType = getattr(types, "UnionType", t.Union)
 NoneType = getattr(types, "NoneType", type(None))
+FieldInfoType = t.TypeVar("FieldInfoType", bound=FieldInfo)
 
 IncEx = t.Union[t.Set[int], t.Set[str], t.Dict[int, t.Any], t.Dict[str, t.Any]]
 
-
 Required = PydanticUndefined
 Undefined = PydanticUndefined
 UndefinedType = PydanticUndefinedType

ellar/pydantic/utils.py

@@ -1,6 +1,5 @@
 import typing as t
 from collections import deque
-from copy import copy
 from dataclasses import is_dataclass
 
 from pydantic import (
@@ -21,9 +20,6 @@
 from .fields import ModelField
 from .types import NoneType, Undefined, UnionType
 
-FieldInfoType = t.TypeVar("FieldInfoType", bound=FieldInfo)
-
-
 sequence_annotation_to_type = {
     t.Sequence: list,
     t.List: list,
@@ -98,17 +94,6 @@ def get_definitions(
     return field_mapping, definitions  # type: ignore[return-value]
 
 
-def copy_from_field_info(
-    *, from_: FieldInfo, to_: t.Type[t.Union[FieldInfoType, FieldInfo]], **kwargs: t.Any
-) -> t.Union[FieldInfoType, FieldInfo]:
-    # copy_of_from = copy(from_)
-    _attributes_set = getattr(from_, "_attributes_set", {})
-    _attributes_set.update(kwargs)
-    new_field = to_(**_attributes_set)
-    new_field.metadata = copy(from_.metadata)
-    return new_field
-
-
 def is_scalar_field(field: ModelField) -> bool:
     from ellar.common.params.params import BodyFieldInfo, WsBodyFieldInfo
 
@@ -251,14 +236,14 @@ def is_bytes_annotation(annotation: t.Any) -> bool:
     return False
 
 
-def __default_skip(type_: t.Any) -> bool:
+def __default_skip__(type_: t.Any) -> bool:
     return False
 
 
 def search_in_annotation(
     annotations: t.Sequence[t.Any],
     lookup: t.Tuple,
-    skip: t.Callable[..., bool] = __default_skip,
+    skip: t.Callable[..., bool] = __default_skip__,
 ) -> t.Union[bool, t.Any]:
     for item in annotations:
         item_origin = get_origin(item) or item