fix some smells

ormar/fields/base.py

@@ -35,8 +35,7 @@ class BaseField:
             default = cls.default if cls.default is not None else cls.server_default
             if callable(default):
                 return Field(default_factory=default)
-            else:
-                return Field(default=default)
+            return Field(default=default)
         return None
 
     @classmethod

ormar/models/newbasemodel.py

@@ -119,11 +119,11 @@ class NewBaseModel(pydantic.BaseModel, ModelTableProxy, metaclass=ModelMetaclass
     def __getattribute__(self, item: str) -> Any:
         if item in ("_orm_id", "_orm_saved", "_orm", "__fields__"):
             return object.__getattribute__(self, item)
-        elif item != "_extract_related_names" and item in self._extract_related_names():
+        if item != "_extract_related_names" and item in self._extract_related_names():
             return self._extract_related_model_instead_of_field(item)
-        elif item == "pk":
+        if item == "pk":
             return self.__dict__.get(self.Meta.pkname, None)
-        elif item != "__fields__" and item in self.__fields__:
+        if item != "__fields__" and item in self.__fields__:
             value = self.__dict__.get(item, None)
             value = self._convert_json(item, value, "loads")
             return value

ormar/queryset/queryset.py

@@ -134,11 +134,11 @@ class QuerySet:
     async def get(self, **kwargs: Any) -> "Model":
         if kwargs:
             return await self.filter(**kwargs).get()
+
+        if not self.filter_clauses:
+            expr = self.build_select_expression().limit(2)
         else:
-            if not self.filter_clauses:
-                expr = self.build_select_expression().limit(2)
-            else:
-                expr = self.build_select_expression()
+            expr = self.build_select_expression()
 
         rows = await self.database.fetch_all(expr)