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

further safety

Browse Source
This commit is contained in:
Kyle Matthew Hedden 2022-04-25 18:32:11 -04:00
parent e19ed416bb
commit c21858c48c
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
MVMCore/MVMCore/Models/ActionType/Client Parameters/ClientParameterHandler.swift
View File

@ -106,11 +106,12 @@
timingOutIn: timeout) { (receivedParameter) in timingOutIn: timeout) { (receivedParameter) in
// Queue the results for merge. // Queue the results for merge.
self.parametersWorkQueue.async { self.parametersWorkQueue.async {
let previouslySatisfied = returnedList.allSatisfy { $0 != nil }
if (returnedList[index] != nil) { if (returnedList[index] != nil) {
MVMCoreLoggingHandler.shared()?.addError(toLog: MVMCoreErrorObject(title: nil, message: "Client parameter \(parameterType) has already executed. The completion handler should only be called once!", code: ErrorCode.default.rawValue, domain: ErrorDomainNative, location: String(describing: ClientParameterHandler.self))!) MVMCoreLoggingHandler.shared()?.addError(toLog: MVMCoreErrorObject(title: nil, message: "Client parameter \(parameterType) has already executed. The completion handler should only be called once!", code: ErrorCode.default.rawValue, domain: ErrorDomainNative, location: String(describing: ClientParameterHandler.self))!)
} }
returnedList[index] = receivedParameter returnedList[index] = receivedParameter
if (returnedList.allSatisfy { $0 != nil }) { if (!previouslySatisfied && returnedList.allSatisfy { $0 != nil }) {
self.group.leave() // Leaving is only done after setup (barriered). self.group.leave() // Leaving is only done after setup (barriered).
} }
} }