fix(js): Fix cleanup API client (#12872)

Previously, if there was no callback function (specifically for `complete`), cleanup was skipped entirely.

Author: billyvg

src/sentry/static/sentry/app/api.jsx

@@ -88,25 +88,25 @@ export class Client {
   }
 
   wrapCallback(id, func, cleanup) {
-    /*eslint consistent-return:0*/
-    if (isUndefined(func)) {
-      return;
-    }
-
     return (...args) => {
       const req = this.activeRequests[id];
       if (cleanup === true) {
         delete this.activeRequests[id];
       }
+
       if (req && req.alive) {
         // Check if API response is a 302 -- means project slug was renamed and user
         // needs to be redirected
         if (this.hasProjectBeenRenamed(...args)) {
           return;
         }
 
+        if (isUndefined(func)) {
+          return;
+        }
+
         // Call success callback
-        return func.apply(req, args);
+        return func.apply(req, args); // eslint-disable-line
       }
     };
   }