PackageMapClient check is triggering on map change

I have a possible fix, but wanted to run it by someone familar with the code.

I wanted to replace this code in PackageMapClient

`if (bIsNotReadOnlyOrAllowRemap || !NetGUIDLookup.Contains(Object))
{
if (CacheObjectPtr->ReadOnlyTimestamp > 0)
{
UE_LOG( LogNetPackageMap, Warning, TEXT( “GetObjectFromNetGUID: Attempt to reassign read-only guid. FullNetGUIDPath: %s” ), *FullNetGUIDPath( NetGUID ) );

if (bAllowClientRemap)
{
CacheObjectPtr->ReadOnlyTimestamp = 0;
}
}

NetGUIDLookup.Add( Object, NetGUID );`with the following:

`if (bIsNotReadOnlyOrAllowRemap || !NetGUIDLookup.Contains(Object))
{
if (CacheObjectPtr->ReadOnlyTimestamp > 0)
{
UE_LOG( LogNetPackageMap, Warning, TEXT( “GetObjectFromNetGUID: Attempt to reassign read-only guid. FullNetGUIDPath: %s” ), *FullNetGUIDPath( NetGUID ) );

if (bAllowClientRemap)
{
CacheObjectPtr->ReadOnlyTimestamp = 0;
}
}

// If the guid already exists, replace it if this guid is newer. Otherwise, keep the new one
if (NetGUIDLookup.Contains(Object))
{
if (NetGUIDLookup[Object] < NetGUID)
{
NetGUIDLookup[Object] = NetGUID;
}
}
else
{
NetGUIDLookup.Add( Object, NetGUID );
}`