From d240df33f6f6140d6013c74b637a5908f9e2f9db Mon Sep 17 00:00:00 2001
From: Dave Schuyler <nthparameter@gmail.com>
Date: Wed, 20 Jul 2005 01:20:44 +0000
Subject: [PATCH] is valid location tuple

---
 direct/src/distributed/DoCollectionManager.py | 16 +++++++++++-----
 1 file changed, 11 insertions(+), 5 deletions(-)

diff --git a/direct/src/distributed/DoCollectionManager.py b/direct/src/distributed/DoCollectionManager.py
index a27af0f8f8..8b909e817b 100755
--- a/direct/src/distributed/DoCollectionManager.py
+++ b/direct/src/distributed/DoCollectionManager.py
@@ -200,10 +200,15 @@ class DoCollectionManager:
             assert do.doId not in self.doId2do
             self.doId2do[do.doId]=do
 
-            if location is not None:
+            if self.isValidLocationTuple(location):
                 assert do.doId not in self.zoneId2doIds.get(location,{})
                 self.zoneId2doIds.setdefault(location, {})
                 self.zoneId2doIds[location][do.doId]=do
+
+        def isValidLocationTuple(self, location):
+            return (location is not None
+                and location != (0xffffffff, 0xffffffff)
+                and location != (0, 0))
     else:
         # NON OTP
         def addDOToTables(self, do, zoneId=None):
@@ -259,7 +264,7 @@ class DoCollectionManager:
             newLocation = (newParentId, newZoneId)
             # HACK: DistributedGuildMemberUD starts in -1,-1, which isnt ever put in the
             # zoneId2doIds table
-            if oldLocation != (BAD_DO_ID, BAD_ZONE_ID):
+            if self.isValidLocationTuple(oldLocation):
                 assert self.notify.debugStateCall(self)
                 assert oldLocation in self.zoneId2doIds
                 assert do.doId in self.zoneId2doIds[oldLocation]
@@ -268,9 +273,10 @@ class DoCollectionManager:
                 del(self.zoneId2doIds[oldLocation][do.doId])
                 if len(self.zoneId2doIds[oldLocation]) == 0:
                     del self.zoneId2doIds[oldLocation]
-            # add to new zone
-            self.zoneId2doIds.setdefault(newLocation, {})
-            self.zoneId2doIds[newLocation][do.doId]=do
+            if self.isValidLocationTuple(newLocation):
+                # add to new zone
+                self.zoneId2doIds.setdefault(newLocation, {})
+                self.zoneId2doIds[newLocation][do.doId]=do
 
         def getObjectsInZone(self, location):
             """ call this to get a dict of doId:distObj for a zone.