mbrucedogs/mvm_core
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Shift logging load errors until after background handling is resolved.

Browse Source
This commit is contained in:
Kyle Matthew Hedden 2023-02-27 14:17:01 -05:00
parent 5006517bb1
commit a74aa6ef2d
1 changed files with 9 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
MVMCore/MVMCore/LoadHandling/MVMCoreLoadRequestOperation.m
View File

@ -439,6 +439,8 @@
NSString *messageStyle = [loadObject.responseInfoMap string:KeyMessageStyle];
if (messageStyle && (!loadObject.operation.backgroundLoad || loadObject.requestParameters.allowAlertsIfBackgroundRequest)) {
loadObject.operation.alertToShow = YES;
} else {
error.silentError = YES;
}
}
@ -750,17 +752,19 @@
return;
}
// Logs the error.
if (error.logError) {
[MVMCoreLoggingHandler addErrorToLog:error];
}
MVMCoreLog(@"Error: %@ %@ %@ %@ %@",[error stringErrorCode], error.domain, error.location,error.messageToDisplay, error.messageToLog);
if (showAlertForErrorIfApplicable && (!loadObject.operation.backgroundLoad || loadObject.requestParameters.allowAlertsIfBackgroundRequest) && !loadObject.requestParameters.handleErrorsSilently && !error.silentError && !error.errorScreenError) {
// Show alert for error.
[MVMCoreLoadRequestOperation createAndShowAlertForLoadObject:loadObject error:error delegateObject:loadObject.operation.delegateObject];
} else {
error.silentError = YES;
}
// Logs the error.
if (error.logError) {
[MVMCoreLoggingHandler addErrorToLog:error];
}
}