fix: allow null in middleware list, improve flowtype definitions

relay-tools · Nov 8, 2017 · c5417c3 · c5417c3
1 parent a7c48b8
commit c5417c3
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/RelayNetworkLayer.js b/src/RelayNetworkLayer.js
@@ -16,13 +16,13 @@ export default class RelayNetworkLayer {
   fetchFn: FetchFunction;
   subscribeFn: ?SubscribeFunction;
 
-  constructor(middlewares: Middleware[] | Middleware, subscribeFn?: SubscribeFunction) {
-    this._middlewares = Array.isArray(middlewares) ? middlewares : [middlewares];
+  constructor(middlewares: Array<?Middleware>, subscribeFn?: SubscribeFunction) {
+    this._middlewares = Array.isArray(middlewares) ? (middlewares: any) : [middlewares];
     this.subscribeFn = subscribeFn;
 
     this.fetchFn = (operation, variables, cacheConfig, uploadables) => {
       const req = new RelayRequest(operation, variables, cacheConfig, uploadables);
-      return fetchWithMiddleware(req, this._middlewares);
+      return fetchWithMiddleware(req, this._middlewares.filter(o => !!o));
     };
 
     this.execute = Network.create(this.fetchFn, this.subscribeFn);