Respond to CR.

pull/1/head
Matthew Chen 7 years ago
parent f00f608833
commit 794914353d

@ -469,10 +469,9 @@ typedef void (^ProfileManagerFailureBlock)(NSError *error);
OWSLogInfo(@"successfully uploaded avatar with key: %@", formKey); OWSLogInfo(@"successfully uploaded avatar with key: %@", formKey);
successBlock(formKey); successBlock(formKey);
} }
failure:^(NSURLSessionDataTask *_Nullable uploadTask, NSError *_Nonnull error) { failure:^(NSURLSessionDataTask *_Nullable uploadTask, NSError *error) {
OWSLogError(@"uploading avatar failed with error: %@", error); OWSLogError(@"uploading avatar failed with error: %@", error);
return failureBlock( return failureBlock(error);
OWSErrorWithCodeDescription(OWSErrorCodeAvatarUploadFailed, @"Avatar upload failed."));
}]; }];
} }
failure:^(NSURLSessionDataTask *task, NSError *error) { failure:^(NSURLSessionDataTask *task, NSError *error) {
@ -483,8 +482,7 @@ typedef void (^ProfileManagerFailureBlock)(NSError *error);
} }
OWSLogError(@"Failed to get profile avatar upload form: %@", error); OWSLogError(@"Failed to get profile avatar upload form: %@", error);
return failureBlock( return failureBlock(error);
OWSErrorWithCodeDescription(OWSErrorCodeAvatarUploadFailed, @"Avatar upload failed."));
}]; }];
}); });
} }
@ -718,7 +716,7 @@ typedef void (^ProfileManagerFailureBlock)(NSError *error);
if ([error isKindOfClass:[NSError class]]) { if ([error isKindOfClass:[NSError class]]) {
failure(error); failure(error);
} else { } else {
failure(OWSErrorWithCodeDescription(OWSErrorCodeProfileUpdateFailed, @"Profile key rotation failed.")); failure(OWSErrorMakeAssertionError(@"Profile key rotation failure missing error."));
} }
}); });
[promise retainUntilComplete]; [promise retainUntilComplete];

Loading…
Cancel
Save