diff --git a/backend/python/app/sources/external/microsoft/one_note/one_note.py b/backend/python/app/sources/external/microsoft/one_note/one_note.py index 76dbc9ad19..4d0cf41c63 100644 --- a/backend/python/app/sources/external/microsoft/one_note/one_note.py +++ b/backend/python/app/sources/external/microsoft/one_note/one_note.py @@ -1,5 +1,3 @@ - - import json import logging from dataclasses import asdict @@ -7160,38 +7158,52 @@ async def me_onenote_notebooks_section_groups_sections_update_pages( """ # Build query parameters including OData for OneNote try: - # Use typed query parameters - query_params = RequestConfiguration() - # Set query parameters using typed object properties - if select: - query_params.select = select if isinstance(select, list) else [select] - if expand: - query_params.expand = expand if isinstance(expand, list) else [expand] - if filter: - query_params.filter = filter - if orderby: - query_params.orderby = orderby - if search: - query_params.search = search - if top is not None: - query_params.top = top - if skip is not None: - query_params.skip = skip - - # Create proper typed request configuration - config = RequestConfiguration() - config.query_parameters = query_params - - if headers: - config.headers = headers - - # Add consistency level for search operations in OneNote - if search: - if not config.headers: - config.headers = {} - config.headers['ConsistencyLevel'] = 'eventual' - - response = await self.client.me.onenote.notebooks.by_notebook_id(notebook_id).section_groups.by_section_group_id(sectionGroup_id).sections.by_onenote_section_id(onenoteSection_id).pages.by_onenote_page_id(onenotePage_id).patch(body=request_body, request_configuration=config) + # Only create objects if really needed to avoid double-initialization + qp_set = any([ + select, expand, filter, orderby, search, top is not None, skip is not None + ]) + # Only instantiate RequestConfiguration objects if necessary + if qp_set: + # Direct instance with bulk assignment + query_params = RequestConfiguration() + if select: + query_params.select = select if isinstance(select, list) else [select] + if expand: + query_params.expand = expand if isinstance(expand, list) else [expand] + if filter: + query_params.filter = filter + if orderby: + query_params.orderby = orderby + if search: + query_params.search = search + if top is not None: + query_params.top = top + if skip is not None: + query_params.skip = skip + else: + query_params = None + + if query_params is not None or headers is not None or search: + config = RequestConfiguration() + if query_params is not None: + config.query_parameters = query_params + if headers: + config.headers = headers + # Add consistency level header if search parameter is used + if search: + if not hasattr(config, "headers") or config.headers is None: + config.headers = {} + config.headers['ConsistencyLevel'] = 'eventual' + else: + config = None + + # Fetch endpoint shortcut just once to minimize lookup time in call chain + endpoint = self.client.me.onenote.notebooks \ + .by_notebook_id(notebook_id) \ + .section_groups.by_section_group_id(sectionGroup_id) \ + .sections.by_onenote_section_id(onenoteSection_id) \ + .pages.by_onenote_page_id(onenotePage_id) + response = await endpoint.patch(body=request_body, request_configuration=config) return self._handle_onenote_response(response) except Exception as e: return OneNoteResponse(