dpa99c / cordova-diagnostic-plugin

Cordova/Phonegap plugin to manage device settings
540 stars 361 forks source link

getBackgroundRefreshStatus missing return statement? #389

Closed scyclops closed 4 years ago

scyclops commented 4 years ago

It looks like there might need to be a return statement after line 84 in getBackgroundRefreshStatus in Diagnostic.m in order to prevent the error callback from being triggered along with another error or success callback immediately after.

https://github.com/dpa99c/cordova-diagnostic-plugin/blob/a55fb69218f8b7556cb739a3266d9352af535eaf/src/ios/Diagnostic.m#L84

dpa99c commented 4 years ago

Thanks, you are right.