Hot fix for #1237 and #1202 (#1256)

* 1. Fix typo PFPreconditionFailOnError and PFPreconditionFailAndSetError. 2. If there is a new object in OfflineStore, should not send the exception when the saved object update its OfflineStore cache

* Change the condition about newly saved object

* fix typo

* Revert to PFPreconditionBailOnError and PFPreconditionBailAndSetError

Author: ananfang

Parse/Parse/Internal/LocalDataStore/OfflineStore/PFOfflineStore.m

@@ -1028,7 +1028,9 @@ - (void)updateObjectIdForObject:(PFObject *)object
     @synchronized(self.lock) {
         // See if there's already an entry for new objectId.
         PFObject *existing = [self.classNameAndObjectIdToObjectMap objectForKey:key];
-        PFConsistencyAssert(existing == nil || existing == object,
+        // If this object is just saved, update local store without exception.
+        BOOL isNewlySaved = (oldObjectId == nil) && (object.createdAt != nil);
+        PFConsistencyAssert(isNewlySaved || existing == nil || existing == object,
                             @"Attempted to change an objectId to one that's already known to the OfflineStore. className: %@ old: %@, new: %@",
                             className, oldObjectId, newObjectId);