refactor: use return_raw_content flag instead of hardcoded name check

Replace _is_content_download name matching with a explicit
return_raw_content flag on FunctionQuery. Callers set the flag
at construction time — cleaner and extensible.

Author: Vadim Grem

office365/onedrive/driveitems/driveItem.py

@@ -427,7 +427,7 @@ def get_content(self, format_name: str | None = None) -> ClientResult[AnyStr]:
         action_name = "content"
         if format_name is not None:
             action_name = action_name + rf"?format={format_name}"
-        qry = FunctionQuery(self, action_name, None, return_type)
+        qry = FunctionQuery(self, action_name, None, return_type, return_raw_content=True)
         self.context.add_query(qry)
         return return_type
 

office365/runtime/odata/request.py

@@ -92,10 +92,7 @@ def process_response(self, response: Response, query: ClientQuery) -> None:
     @staticmethod
     def _is_content_download(query: ClientQuery) -> bool:
         """Check if the query is a raw content download (file content, not OData metadata)."""
-        if isinstance(query, FunctionQuery):
-            name = query.name
-            return name in ("content", "$value")
-        return False
+        return bool(getattr(query, "return_raw_content", False))
 
     def map_json(
         self,

office365/runtime/queries/function.py

@@ -15,6 +15,7 @@ def __init__(
         method_name: str | None = None,
         method_params: list | dict | ClientValue | None = None,
         return_type: ReturnT | None = None,
+        return_raw_content: bool = False,
     ) -> None:
         """Initialize a function query.
 
@@ -23,9 +24,17 @@ def __init__(
             method_name: The name of the method to call
             method_params: Parameters for the method call
             return_type: The expected return type
+            return_raw_content: When True, the response body is treated as raw content
+                (e.g. file download) rather than parsed as OData JSON.
         """
         super().__init__(binding_type.context, binding_type, None, None, return_type)
         self._path = ServiceOperationPath(method_name or "", method_params, binding_type.resource_path)
+        self._return_raw_content = return_raw_content
+
+    @property
+    def return_raw_content(self) -> bool:
+        """Whether the response should be treated as raw content, not OData JSON."""
+        return self._return_raw_content
 
     def __repr__(self) -> str:
         return f"FunctionQuery(name={self.path.name})"

office365/sharepoint/files/file.py

@@ -140,7 +140,7 @@ def _file_loaded():
     def get_content(self) -> ClientResult[bytes]:
         """Downloads a file content"""
         return_type = ClientResult(self.context, bytes())
-        qry = FunctionQuery(self, "$value", return_type=return_type)
+        qry = FunctionQuery(self, "$value", return_type=return_type, return_raw_content=True)
         self.context.add_query(qry)
         return return_type