From 5390806420666f1529b0fc95d193290e1a0045aa Mon Sep 17 00:00:00 2001
From: Benjamin Pasero <benjpas@microsoft.com>
Date: Tue, 16 Oct 2018 11:41:47 +0200
Subject: [PATCH] storage - :lipstick:

---
 src/vs/base/node/storage.ts | 15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)

diff --git a/src/vs/base/node/storage.ts b/src/vs/base/node/storage.ts
index 2dce7239d3a..b797ca0fb24 100644
--- a/src/vs/base/node/storage.ts
+++ b/src/vs/base/node/storage.ts
@@ -290,7 +290,7 @@ export class SQLiteStorageImpl {
 						return reject(error);
 					}
 
-					resolve();
+					return resolve();
 				});
 			});
 		});
@@ -351,21 +351,26 @@ export class SQLiteStorageImpl {
 					return reject(error);
 				}
 
-				resolve();
+				return resolve();
 			});
 		});
 	}
 
 	private each(db: Database, sql: string, callback: (row: any) => void): Promise<void> {
 		return new Promise((resolve, reject) => {
+			let hadError = false;
 			db.each(sql, (error, row) => {
 				if (error) {
 					this.logger.error(`[storage ${this.name}] each(): ${error}`);
 
+					hadError = true;
+
 					return reject(error);
 				}
 
-				callback(row);
+				if (!hadError) {
+					callback(row);
+				}
 			}, error => {
 				if (error) {
 					this.logger.error(`[storage ${this.name}] each(): ${error}`);
@@ -373,7 +378,7 @@ export class SQLiteStorageImpl {
 					return reject(error);
 				}
 
-				resolve();
+				return resolve();
 			});
 		});
 	}
@@ -392,7 +397,7 @@ export class SQLiteStorageImpl {
 						return reject(error);
 					}
 
-					resolve();
+					return resolve();
 				});
 			});
 		});
-- 
GitLab