cleaned up the "seen" parameter in setResources after Micah's optimization made that unnecessary

Author: sbazerque

src/data/model/immutable/HashedObject.ts

@@ -290,15 +290,13 @@ abstract class HashedObject {
         return Hashing.forValue('#' + this.getId() + '.' + fieldName);
     }
 
-    setResources(resources: Resources, seen: Set<HashedObject> = new Set()): void {
+    setResources(resources: Resources): void {
         if (this._resources === resources) return;
-        if (seen?.has(this)) return;
 
         this._resources = resources;
-        seen.add(this)
 
         for (const subobj of this.getDirectSubObjects().values()) {
-            subobj.setResources(resources, seen);
+            subobj.setResources(resources);
         }
     }
 

src/data/model/mutable/MutableObject.ts

@@ -655,10 +655,9 @@ abstract class  MutableObject extends HashedObject {
         return this._acceptedMutationOpClasses;
     }
 
-    setResources(resources: Resources, seen = new Set<HashedObject>()): void {
-        if (seen.has(this)) return;
-        
-        seen.add(this);
+    setResources(resources: Resources): void {
+
+        if (this.getResources() === resources) return;
 
         let reBindToStore = false;
 
@@ -675,7 +674,7 @@ abstract class  MutableObject extends HashedObject {
 
         for (const aliases of this.getMutableContents().values()) {
             for (const obj of aliases) {
-                obj.setResources(resources, seen);
+                obj.setResources(resources);
             }
         }