fix import circular issue for NavigationNode

Author: vgrem

office365/sharepoint/brandcenter/brand_center.py

@@ -17,6 +17,7 @@ def __init__(self, context):
         super(BrandCenter, self).__init__(context, ResourcePath("SP.BrandCenter"))
 
     def configuration(self):
+        """ """
         return_type = ClientResult(self.context, BrandCenterConfiguration())
         qry = ServiceOperationQuery(
             self, "Configuration", None, None, None, return_type
@@ -25,6 +26,7 @@ def configuration(self):
         return return_type
 
     def get_site_themes(self):
+        """ """
         return_type = ClientResult(self.context, SiteThemes())
         qry = ServiceOperationQuery(
             self, "GetSiteThemes", None, None, None, return_type

office365/sharepoint/navigation/node.py

@@ -2,7 +2,6 @@
 
 from office365.runtime.paths.resource_path import ResourcePath
 from office365.sharepoint.entity import Entity
-from office365.sharepoint.navigation.node_collection import NavigationNodeCollection
 from office365.sharepoint.translation.user_resource import UserResource
 
 
@@ -20,8 +19,12 @@ def __repr__(self):
 
     @property
     def children(self):
-        # type: () -> NavigationNodeCollection
+        # type: () -> 'NavigationNodeCollection'
         """Gets the collection of child nodes of the navigation node."""
+        from office365.sharepoint.navigation.node_collection import (
+            NavigationNodeCollection,
+        )
+
         return self.properties.get(
             "Children",
             NavigationNodeCollection(
@@ -66,7 +69,7 @@ def is_external(self):
 
     @property
     def parent_collection(self):
-        # type: () -> NavigationNodeCollection
+        # type: () -> 'NavigationNodeCollection'
         return self._parent_collection
 
     @property
@@ -84,13 +87,3 @@ def get_property(self, name, default_value=None):
             property_mapping = {"TitleResource": self.title_resource}
             default_value = property_mapping.get(name, None)
         return super(NavigationNode, self).get_property(name, default_value)
-
-    def set_property(self, name, value, persist_changes=True):
-        super(NavigationNode, self).set_property(name, value, persist_changes)
-        # fallback: create a new resource path
-        if self._resource_path is None:
-            if name == "Id":
-                self._resource_path = self.parent_collection.get_by_id(
-                    value
-                ).resource_path
-        return self