Skip to content

[Fix] Update operationIds of navigation property paths with OData type cast segments #442

@irvinesunday

Description

@irvinesunday

Recently, we enable the expansion of derived types navigation properties here.
This created paths like below:

'/deviceAppManagement/mobileApps/{mobileApp-id}/microsoft.graph.androidStoreApp/categories/{mobileAppCategory-id}':
    description: Provides operations to manage the categories property of the microsoft.graph.mobileApp entity.
    get:
      tags:
        - deviceAppManagement.mobileApp
      summary: Get categories from deviceAppManagement
      description: The list of categories for this app.
      operationId: deviceAppManagement.mobileApps.GetCategories

Since we only generate operationIds of navigation property paths based on the navigation property segments, we end up with duplicate operationIds for 'similar' navigation property paths which only differ on the OData type cast segment, e.g.:

'/deviceAppManagement/mobileApps/{mobileApp-id}/microsoft.graph.iosLobApp/categories/{mobileAppCategory-id}':
    description: Provides operations to manage the categories property of the microsoft.graph.mobileApp entity.
    get:
      tags:
        - deviceAppManagement.mobileApp
      summary: Get categories from deviceAppManagement
      description: The list of categories for this app.
      operationId: deviceAppManagement.mobileApps.GetCategories

We need to update the operationIds so as to include the OData type cast segment name to differentiate the above two scenarios.

Metadata

Metadata

Assignees

Labels

priority:p1High priority but not blocking. Causes major but not critical loss of functionality SLA <=7daystype:bugA broken experience

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions