From f54a430eb1fb8aa59562afb2ae1f55c1ddd85f3e Mon Sep 17 00:00:00 2001
From: Andrew Omondi <anomondi@microsoft.com>
Date: Thu, 7 Mar 2024 09:55:40 +0300
Subject: [PATCH] Removes sites from having `add` and `remove` methods

---
 transforms/csdl/preprocess_csdl.xsl             |  3 +--
 transforms/csdl/preprocess_csdl_test_output.xml | 15 ++++-----------
 2 files changed, 5 insertions(+), 13 deletions(-)

diff --git a/transforms/csdl/preprocess_csdl.xsl b/transforms/csdl/preprocess_csdl.xsl
index c49c09761..0f2fe0197 100644
--- a/transforms/csdl/preprocess_csdl.xsl
+++ b/transforms/csdl/preprocess_csdl.xsl
@@ -2391,8 +2391,7 @@
 
      <!-- Add ExplicitOperationBindings for followedSites and sites navigation props --> 
     <xsl:template match="edm:Schema[@Namespace='microsoft.graph']/edm:EntityType[@Name='user']/edm:NavigationProperty[@Name='followedSites']|
-                         edm:Schema[@Namespace='microsoft.graph']/edm:EntityType[@Name='group']/edm:NavigationProperty[@Name='sites']|
-                         edm:Schema[@Namespace='microsoft.graph']/edm:EntityType[@Name='site']/edm:NavigationProperty[@Name='sites']">
+                         edm:Schema[@Namespace='microsoft.graph']/edm:EntityType[@Name='group']/edm:NavigationProperty[@Name='sites']">
         <xsl:copy>
             <xsl:apply-templates select="@* | node()"/>
                 <Annotation Term="Org.OData.Core.V1.ExplicitOperationBindings">
diff --git a/transforms/csdl/preprocess_csdl_test_output.xml b/transforms/csdl/preprocess_csdl_test_output.xml
index 75086f3a7..e5dafd06e 100644
--- a/transforms/csdl/preprocess_csdl_test_output.xml
+++ b/transforms/csdl/preprocess_csdl_test_output.xml
@@ -333,14 +333,7 @@
             </Collection>
           </Annotation>
         </NavigationProperty>
-        <NavigationProperty Name="sites" Type="Collection(graph.site)">
-          <Annotation Term="Org.OData.Core.V1.ExplicitOperationBindings" xmlns:edm="http://docs.oasis-open.org/odata/ns/edm">
-            <Collection>
-              <String>microsoft.graph.add</String>
-              <String>microsoft.graph.remove</String>
-            </Collection>
-          </Annotation>
-        </NavigationProperty>
+        <NavigationProperty Name="sites" Type="Collection(graph.site)" />
       </EntityType>
       <EntityType Name="baseSitePage" BaseType="graph.baseItem">
         <Property Name="pageLayout" Type="graph.pageLayoutType" />
@@ -1185,18 +1178,18 @@
         <ReturnType Type="Collection(graph.directoryObject)" Nullable="false" />
         <Annotation Term="Org.OData.Core.V1.RequiresExplicitBinding" xmlns:edm="http://docs.oasis-open.org/odata/ns/edm" />
       </Function>
-      <Function Name="boundingRect" IsBound="true" IsComposable="false">
+      <Function Name="boundingRect" IsBound="true" IsComposable="true">
         <Parameter Name="bindparameter" Type="graph.workbookRange" />
         <Parameter Name="anotherRange" Type="Edm.String" Unicode="false" />
         <ReturnType Type="graph.workbookRange" />
       </Function>
-      <Function Name="cell" IsBound="true" IsComposable="false">
+      <Function Name="cell" IsBound="true" IsComposable="true">
         <Parameter Name="bindparameter" Type="graph.workbookRange" />
         <Parameter Name="row" Type="Edm.Int32" Nullable="false" />
         <Parameter Name="column" Type="Edm.Int32" Nullable="false" />
         <ReturnType Type="graph.workbookRange" />
       </Function>
-      <Function Name="cell" IsBound="true" IsComposable="false">
+      <Function Name="cell" IsBound="true" IsComposable="true">
         <Parameter Name="bindparameter" Type="graph.workbookWorksheet" />
         <Parameter Name="row" Type="Edm.Int32" Nullable="false" />
         <Parameter Name="column" Type="Edm.Int32" Nullable="false" />