@sap/xsodata 8.2.1 → 8.3.1

package/lib/processor/resourceProcessor.js

@@ -6,15 +6,15 @@
  * @type {exports}
  */
 
-var postProcessor = require('./resourceProcessorPost');
-var putProcessor = require('./resourceProcessorPut');
-var putPostLinksProcessor = require('./resourceProcessorPutPostLinks');
-var deleteProcessor = require('./resourceProcessorDelete');
-var deleteLinksProcessor = require('./resourceProcessorDeleteLinks');
-var getProcessor = require('./resourceProcessorGet');
-var MethodNotAllowedError = require('./../utils/errors/http/methodNotAllowed');
-var NotImplementedError = require('./../utils/errors/http/notImplemented');
-var db = require('./../db/connect');
+const postProcessor = require('./resourceProcessorPost');
+const putProcessor = require('./resourceProcessorPut');
+const putPostLinksProcessor = require('./resourceProcessorPutPostLinks');
+const deleteProcessor = require('./resourceProcessorDelete');
+const deleteLinksProcessor = require('./resourceProcessorDeleteLinks');
+const getProcessor = require('./resourceProcessorGet');
+const MethodNotAllowedError = require('./../utils/errors/http/methodNotAllowed');
+const NotImplementedError = require('./../utils/errors/http/notImplemented');
+const db = require('./../db/connect');
 
 exports.process = function (context, asyncDone) {
 
@@ -23,7 +23,7 @@ exports.process = function (context, asyncDone) {
             return asyncDone(err, asyncDone);
         }
 
-        var method = context.request.method;
+        const method = context.request.method;
 
         switch (method) {
             case "GET":
@@ -31,24 +31,18 @@ exports.process = function (context, asyncDone) {
             case "POST":
                 if (context.oData.dbSegment.isLinks) {
                     return putPostLinksProcessor.process(context, asyncDone);
-                } else {
-                    return postProcessor.process(context, asyncDone);
                 }
-                break;
+                return postProcessor.process(context, asyncDone);
             case "PUT":
                 if (context.oData.dbSegment.isLinks) {
                     return putPostLinksProcessor.process(context, asyncDone);
-                } else {
-                    return putProcessor.process(context, asyncDone);
                 }
-                break;
+                return putProcessor.process(context, asyncDone);
             case "DELETE":
                 if (context.oData.dbSegment.isLinks) {
                     return deleteLinksProcessor.process(context, asyncDone);
-                } else {
-                    return deleteProcessor.process(context, asyncDone);
                 }
-                break;
+                return deleteProcessor.process(context, asyncDone);
             // We do not support PATCH or MERGE
             case "PATCH":
                 return asyncDone(new MethodNotAllowedError("Method not allowed"), context);
@@ -64,7 +58,7 @@ exports.processInBatchCreateTables = function (context, asyncDone) {
             return asyncDone(err, asyncDone);
         }
 
-        var method = context.request.method;
+        const method = context.request.method;
 
         switch (method) {
             case "GET":
@@ -72,24 +66,18 @@ exports.processInBatchCreateTables = function (context, asyncDone) {
             case "POST":
                 if (context.oData.dbSegment.isLinks) {
                     return putPostLinksProcessor.processInBatchCreateTables(context, asyncDone);
-                } else {
-                    return postProcessor.processInBatchCreateTables(context, asyncDone);
                 }
-                break;
+                return postProcessor.processInBatchCreateTables(context, asyncDone);
             case "PUT":
                 if (context.oData.dbSegment.isLinks) {
                     return putPostLinksProcessor.processInBatchCreateTables(context, asyncDone);
-                } else {
-                    return putProcessor.processInBatchCreateTables(context, asyncDone);
                 }
-                break;
+                return putProcessor.processInBatchCreateTables(context, asyncDone);
             case "DELETE":
                 if (context.oData.dbSegment.isLinks) {
                     return deleteLinksProcessor.processInBatchCreateTables(context, asyncDone);
-                } else {
-                    return deleteProcessor.processInBatchCreateTables(context, asyncDone);
                 }
-                break;
+                return deleteProcessor.processInBatchCreateTables(context, asyncDone);
             // We do not support PATCH or MERGE
             case "PATCH":
                 return asyncDone(new MethodNotAllowedError("Method not allowed"), context);
@@ -106,7 +94,7 @@ exports.processInBatch = function (context, asyncDone) {
         if (err) {
             return asyncDone(err, asyncDone);
         }
-        var method = context.request.method;
+        const method = context.request.method;
 
         switch (method) {
             case "GET":
@@ -114,24 +102,18 @@ exports.processInBatch = function (context, asyncDone) {
             case "POST":
                 if (context.oData.dbSegment.isLinks) {
                     return putPostLinksProcessor.processInBatch(context, asyncDone);
-                } else {
-                    return postProcessor.processInBatch(context, asyncDone);
                 }
-                break;
+                return postProcessor.processInBatch(context, asyncDone);
             case "PUT":
                 if (context.oData.dbSegment.isLinks) {
                     return putPostLinksProcessor.processInBatch(context, asyncDone);
-                } else {
-                    return putProcessor.processInBatch(context, asyncDone);
                 }
-                break;
+                return putProcessor.processInBatch(context, asyncDone);
             case "DELETE":
                 if (context.oData.dbSegment.isLinks) {
                     return deleteLinksProcessor.processInBatch(context, asyncDone);
-                } else {
-                    return deleteProcessor.processInBatch(context, asyncDone);
                 }
-                break;
+                return deleteProcessor.processInBatch(context, asyncDone);
             // We do not support PATCH or MERGE
             case "PATCH":
                 return asyncDone(new MethodNotAllowedError("Method not allowed"), context);
@@ -146,7 +128,7 @@ exports.processInBatchPreCommitRun = function (context, asyncDone) {
         if (err) {
             return asyncDone(err, asyncDone);
         }
-        var method = context.request.method;
+        const method = context.request.method;
 
         switch (method) {
             case "GET":
@@ -154,24 +136,18 @@ exports.processInBatchPreCommitRun = function (context, asyncDone) {
             case "POST":
                 if (context.oData.dbSegment.isLinks) {
                     return putPostLinksProcessor.processInBatchPreCommitRun(context, asyncDone);
-                } else {
-                    return postProcessor.processInBatchPreCommitRun(context, asyncDone);
                 }
-                break;
+                return postProcessor.processInBatchPreCommitRun(context, asyncDone);
             case "PUT":
                 if (context.oData.dbSegment.isLinks) {
                     return putPostLinksProcessor.processInBatchPreCommitRun(context, asyncDone);
-                } else {
-                    return putProcessor.processInBatchPreCommitRun(context, asyncDone);
                 }
-                break;
+                return putProcessor.processInBatchPreCommitRun(context, asyncDone);
             case "DELETE":
                 if (context.oData.dbSegment.isLinks) {
                     return deleteLinksProcessor.processInBatchPreCommitRun(context, asyncDone);
-                } else {
-                    return deleteProcessor.processInBatchPreCommitRun(context, asyncDone);
                 }
-                break;
+                return deleteProcessor.processInBatchPreCommitRun(context, asyncDone);
             // We do not support PATCH or MERGE
             case "PATCH":
                 return asyncDone(new MethodNotAllowedError("Method not allowed"), context);
@@ -187,7 +163,7 @@ exports.processInBatchPostCommitRun = function (context, asyncDone) {
             return asyncDone(err, asyncDone);
         }
 
-        var method = context.request.method;
+        const method = context.request.method;
 
         switch (method) {
             case "GET":
@@ -195,29 +171,23 @@ exports.processInBatchPostCommitRun = function (context, asyncDone) {
             case "POST":
                 if (context.oData.dbSegment.isLinks) {
                     return putPostLinksProcessor.processInBatchPostCommitRun(context, asyncDone);
-                } else {
-                    return postProcessor.processInBatchPostCommitRun(context, asyncDone);
                 }
-                break;
+                return postProcessor.processInBatchPostCommitRun(context, asyncDone);
             case "PUT":
                 if (context.oData.dbSegment.isLinks) {
                     return putPostLinksProcessor.processInBatchPostCommitRun(context, asyncDone);
-                } else {
-                    return putProcessor.processInBatchPostCommitRun(context, asyncDone);
                 }
-                break;
+                return putProcessor.processInBatchPostCommitRun(context, asyncDone);
             case "DELETE":
                 if (context.oData.dbSegment.isLinks) {
                     return deleteLinksProcessor.processInBatchPostCommitRun(context, asyncDone);
-                } else {
-                    return deleteProcessor.processInBatchPostCommitRun(context, asyncDone);
                 }
-                break;
+                return deleteProcessor.processInBatchPostCommitRun(context, asyncDone);
             // We do not support PATCH or MERGE
             case "PATCH":
-                return asyncDone(MethodNotAllowedError("Method not allowed"), context);
+                return asyncDone(new MethodNotAllowedError("Method not allowed"), context);
             default:
-                return asyncDone(NotImplementedError("Not implemented"), context);
+                return asyncDone(new NotImplementedError("Not implemented"), context);
         }
     });
 };

package/lib/processor/resourceProcessorDelete.js

@@ -1,17 +1,17 @@
 'use strict';
-var async = require('async');
+const async = require('async');
 
-var connect = require('../db/connect');
-var utils = require('./../utils/utils');
-var contentIdHelper = require('./contentIdHelper');
-var sqlDelete = require('./../sql/createDeleteStatements');
-var dataCollectorDelete = require('./../sql/dataCollectorDelete');
-var serializer = require('./../serializer/serializer');
-var exitProcessor = require('./exitProcessor');
+const connect = require('../db/connect');
+const utils = require('./../utils/utils');
+const contentIdHelper = require('./contentIdHelper');
+const sqlDelete = require('./../sql/createDeleteStatements');
+const dataCollectorDelete = require('./../sql/dataCollectorDelete');
+const serializer = require('./../serializer/serializer');
+const exitProcessor = require('./exitProcessor');
 
 function eventBefore(context, asyncDone) {
     context.logger.info('delete', 'event start before');
-    var eventFunction = exitProcessor.eventHandler('before', 'delete');
+    const eventFunction = exitProcessor.eventHandler('before', 'delete');
     eventFunction(context, function (err, context) {
         context.logger.info('delete event end', 'before');
         if (err) {
@@ -23,7 +23,7 @@ function eventBefore(context, asyncDone) {
 
 function eventAfter(context, asyncDone) {
     context.logger.info('delete', 'event start after');
-    var eventFunction = exitProcessor.eventHandler('after', 'delete');
+    const eventFunction = exitProcessor.eventHandler('after', 'delete');
     eventFunction(context, function (err, context) {
         context.logger.info('delete event end', 'after');
         if (err) {
@@ -35,7 +35,7 @@ function eventAfter(context, asyncDone) {
 
 function eventPrecommit(context, asyncDone) {
     context.logger.info('delete', 'event start precommit');
-    var eventFunction = exitProcessor.eventHandler('precommit', 'delete');
+    const eventFunction = exitProcessor.eventHandler('precommit', 'delete');
     eventFunction(context, function (err, context) {
         context.logger.info('delete event end', 'precommit');
         if (err) {
@@ -47,7 +47,7 @@ function eventPrecommit(context, asyncDone) {
 
 function eventPostCommit(context, asyncDone) {
     context.logger.info('delete', 'event start postcommit');
-    var eventFunction = exitProcessor.eventHandler('postcommit', 'delete');
+    const eventFunction = exitProcessor.eventHandler('postcommit', 'delete');
     eventFunction(context, function (err, context) {
         context.logger.info('delete event end', 'postcommit');
         if (err) {
@@ -59,9 +59,9 @@ function eventPostCommit(context, asyncDone) {
 
 
 function executeDelete(context, asyncDone) {
-    var dbSeg = context.oData.dbSegmentLast;
-    var entityType = dbSeg.entityType;
-    var _delete = (entityType.modifications || {})['delete'] || {};
+    const dbSeg = context.oData.dbSegmentLast;
+    const entityType = dbSeg.entityType;
+    const _delete = (entityType.modifications || {})['delete'] || {};
     if (_delete.using) {
         exitProcessor.executeExit(_delete.using, 'using', 'delete', context, asyncDone);
     } else {
@@ -103,7 +103,7 @@ exports.process = function (context, asyncDone) {
         ],
         function (err, context) {
             if (err) {
-                var dbClient = context.db.client;
+                const dbClient = context.db.client;
                 if (dbClient) {
                     //ROLLBack the changes
                     return connect.dbRollback(context, dbClient, function (errDB) {

package/lib/processor/resourceProcessorDeleteLinks.js

@@ -1,17 +1,17 @@
 'use strict';
 
-var async = require('async');
-var connect = require('../db/connect');
-var sqlDeleteLinks = require('./../sql/createDeleteLinksStatements');
-var dataCollectorLinks = require('./../sql/dataCollectorLinks');
-var dataCollectorDeleteLinks = require('./../sql/dataCollectorDeleteLinks');
-var serializer = require('./../serializer/serializer');
-var exitProcessor = require('./exitProcessor');
-var utils = require('./../utils/utils');
+const async = require('async');
+const connect = require('../db/connect');
+const sqlDeleteLinks = require('./../sql/createDeleteLinksStatements');
+const dataCollectorLinks = require('./../sql/dataCollectorLinks');
+const dataCollectorDeleteLinks = require('./../sql/dataCollectorDeleteLinks');
+const serializer = require('./../serializer/serializer');
+const exitProcessor = require('./exitProcessor');
+const utils = require('./../utils/utils');
 
 
 function eventBefore(context, asyncDone) {
-	var eventFunction = exitProcessor.eventHandler('before', 'delete');
+	const eventFunction = exitProcessor.eventHandler('before', 'delete');
 
 	context.logger.info('delete', 'event start before');
     eventFunction(context, function (err, context) {
@@ -24,7 +24,7 @@ function eventBefore(context, asyncDone) {
 }
 
 function eventAfter(context, asyncDone) {
-	var eventFunction = exitProcessor.eventHandler('after', 'delete');
+	const eventFunction = exitProcessor.eventHandler('after', 'delete');
 
 	context.logger.info('delete', 'event start after');
     eventFunction(context, function (err, context) {
@@ -37,7 +37,7 @@ function eventAfter(context, asyncDone) {
 }
 
 function eventPrecommit(context, asyncDone) {
-	var eventFunction = exitProcessor.eventHandler('precommit', 'delete');
+	const eventFunction = exitProcessor.eventHandler('precommit', 'delete');
 
 	context.logger.info('delete', 'event start precommit');
     eventFunction(context, function (err, context) {
@@ -50,7 +50,7 @@ function eventPrecommit(context, asyncDone) {
 }
 
 function eventPostCommit(context, asyncDone) {
-	var eventFunction = exitProcessor.eventHandler('postcommit', 'delete');
+	const eventFunction = exitProcessor.eventHandler('postcommit', 'delete');
 
 	context.logger.info('delete', 'event start postcommit');
     eventFunction(context, function (err, context) {
@@ -63,9 +63,9 @@ function eventPostCommit(context, asyncDone) {
 }
 
 function insertTmpTableToRealTable(context, asyncDone) {
-    var dbSeg = context.oData.dbSegmentLast;
-    var entityType = dbSeg.entityType;
-    var create = (entityType.modifications || {} ).update || {};
+    const dbSeg = context.oData.dbSegmentLast;
+    const entityType = dbSeg.entityType;
+    const create = (entityType.modifications || {} ).update || {};
 
     if (create.using) {
         exitProcessor.executeExit(create.using, 'using', 'delete', context, asyncDone);
@@ -75,9 +75,9 @@ function insertTmpTableToRealTable(context, asyncDone) {
 }
 
 function executeDelete(context, asyncDone) {
-    var dbSeg = context.oData.dbSegmentLast;
-    var entityType = dbSeg.entityType;
-    var _delete = (entityType.modifications || {} )['delete'] || {};
+    const dbSeg = context.oData.dbSegmentLast;
+    const entityType = dbSeg.entityType;
+    const _delete = (entityType.modifications || {} )['delete'] || {};
     if (_delete.using) {
         exitProcessor.executeExit(_delete.using, 'using', 'delete', context, asyncDone);
     } else {
@@ -91,8 +91,8 @@ function setStatus(context, asyncDone) {
 }
 
 exports.process = function (context, asyncDone) {
-    var execArr;
-	var m2n = (context.oData.dbSegmentLast.getOver()!==undefined);
+    let execArr;
+	const m2n = (context.oData.dbSegmentLast.getOver()!==undefined);
 
 	context.logger.silly('resourceProcessorDeleteLinks', 'process');
 
@@ -167,7 +167,7 @@ exports.process = function (context, asyncDone) {
         execArr,
         function (err, context) {
             if (err) {
-                var dbClient = context.db.client;
+                const dbClient = context.db.client;
                 if (dbClient) {
                     //ROLLBack the changes
                     return connect.dbRollback(context, dbClient, function (errDB) {
@@ -188,8 +188,8 @@ exports.process = function (context, asyncDone) {
 exports.processInBatchCreateTables = function (context, asyncDone) {
     context.logger.silly('resourceProcessorDeleteLinks', 'processInBatchCreateTables');
 
-    var execArr;
-    var m2n = (context.oData.dbSegmentLast.getOver()!==undefined);
+    let execArr;
+    const m2n = (context.oData.dbSegmentLast.getOver()!==undefined);
     if (m2n) { // INSERT into 3rd table
         context.oData.dbSegmentLast.m2n = true;
         execArr = [
@@ -221,8 +221,8 @@ exports.processInBatchCreateTables = function (context, asyncDone) {
 exports.processInBatch = function (context, asyncDone) {
     context.logger.silly('resourceProcessorDeleteLinks', 'processInBatch');
 
-    var execArr;
-    var m2n = (context.oData.dbSegmentLast.getOver()!==undefined);
+    let execArr;
+    const m2n = (context.oData.dbSegmentLast.getOver()!==undefined);
     if (m2n) { // INSERT into 3rd table
         context.oData.dbSegmentLast.m2n = true;
         execArr = [

package/lib/processor/resourceProcessorGet.js

@@ -1,11 +1,11 @@
 'use strict';
 
-var async = require('async');
+const async = require('async');
 
-var utils = require('./../utils/utils');
-var dataCollectorGet = require('./../sql/dataCollectorGet');
-var serializer = require('./../serializer/serializer');
-var sqlGet = require('./../sql/createGetStatements');
+const utils = require('./../utils/utils');
+const dataCollectorGet = require('./../sql/dataCollectorGet');
+const serializer = require('./../serializer/serializer');
+const sqlGet = require('./../sql/createGetStatements');
 
 exports.process = function (context, asyncDone) {
     context.logger.silly('resourceprocessor', 'process_GET');

package/lib/processor/resourceProcessorPost.js

@@ -1,36 +1,36 @@
 'use strict';
 
-var async = require('async');
-var connect = require('../db/connect');
-var utils = require('./../utils/utils');
+const async = require('async');
+const connect = require('../db/connect');
+const utils = require('./../utils/utils');
 
-var contentIdHelper = require('./contentIdHelper');
-var contentTypeCheck = require('./../utils/checkContentType');
-var dataCollectorPost = require('./../sql/dataCollectorPost');
-var exitProcessor = require('./exitProcessor');
-var serializer = require('./../serializer/serializer');
-var sqlPost = require('./../sql/createPostStatements');
-var typeConverter = require('./../utils/typeConverter');
+const contentIdHelper = require('./contentIdHelper');
+const contentTypeCheck = require('./../utils/checkContentType');
+const dataCollectorPost = require('./../sql/dataCollectorPost');
+const exitProcessor = require('./exitProcessor');
+const serializer = require('./../serializer/serializer');
+const sqlPost = require('./../sql/createPostStatements');
+const typeConverter = require('./../utils/typeConverter');
 
 
-var AtomXmlSerializer = require('./../serializer/atomXmlToJsonSerializer');
-var BadRequestError = require('./../utils/errors/http/badRequest');
-var InternalError = require('./../utils/errors/internalError');
-var UnsupportedMediaType = require('./../utils/errors/http/unsupportedMediaType');
+const AtomXmlSerializer = require('./../serializer/atomXmlToJsonSerializer');
+const BadRequestError = require('./../utils/errors/http/badRequest');
+const InternalError = require('./../utils/errors/internalError');
+const UnsupportedMediaType = require('./../utils/errors/http/unsupportedMediaType');
 
 const model = require('../model/model.js');
 
 function movePayloadToDbSegment_RecordNV(context, asyncDone) {
 
-    var req = context.request;
-    var contentType = req.headers['content-type'];
+    const req = context.request;
+    const contentType = req.headers['content-type'];
 
     try {
         context.logger.silly('resourceProcessorPost', 'movePayloadToDbSegment');
 
         if (contentType) {
 
-            var isSupportedContentType =
+            const isSupportedContentType =
                 contentTypeCheck.isSupportedContentType(contentType);
 
             if (isSupportedContentType === false) {
@@ -55,15 +55,15 @@ function movePayloadToDbSegment_RecordNV(context, asyncDone) {
         }
         try {
 
-            var dbSeg = context.oData.dbSegmentLast;
+            const dbSeg = context.oData.dbSegmentLast;
 
-            var data = buffer.toString('utf-8');
+            const data = buffer.toString('utf-8');
 
             if (contentTypeCheck.isContentTypeXml(contentType) === true && utils.isXml(data) === true) {
 
-                var typeModel = dbSeg.entityType.propertiesMap;
+                const typeModel = dbSeg.entityType.propertiesMap;
 
-                var serializer = new AtomXmlSerializer(data, typeModel);
+                const serializer = new AtomXmlSerializer(data, typeModel);
 
                 return serializer
                     .serialize(function (err, innerContext) {
@@ -73,17 +73,17 @@ function movePayloadToDbSegment_RecordNV(context, asyncDone) {
                         }
 
                         if (!Array.isArray(innerContext.result)) {
-                            var err1 = new InternalError("Serialization result is not an array", context);
+                            const err1 = new InternalError("Serialization result is not an array", context);
                             return asyncDone(err1, context);
                         }
 
-                        var len = innerContext.result.length;
+                        const len = innerContext.result.length;
                         if (len !== 1) {
-                            var err2 = new InternalError("Serialization result length must be 1 but is " + len, context);
+                            const err2 = new InternalError("Serialization result length must be 1 but is " + len, context);
                             return asyncDone(err2, context);
                         }
 
-                        var result = innerContext.result[0];
+                        const result = innerContext.result[0];
 
                         dbSeg.setRecordFromPostPayload(context, result);
 
@@ -94,7 +94,7 @@ function movePayloadToDbSegment_RecordNV(context, asyncDone) {
             }
 
             if (contentTypeCheck.isContentTypeJson(contentType) === true) {
-                var json;
+                let json;
 
                 try {
                     json = JSON.parse(data);
@@ -119,15 +119,12 @@ function movePayloadToDbSegment_RecordNV(context, asyncDone) {
         } catch (err) {
             return asyncDone(err, context);
         }
-
-        return asyncDone(null, context);
     }
-
 }
 
 
 function modifyDbSegment(context, asyncDone) {
-    var dbSeg = context.oData.dbSegmentLast;
+    const dbSeg = context.oData.dbSegmentLast;
     //The dbSegments are build by URL so the last segment for a POST request is a collection.
     //But the returned value from a POST request is the newly created entity, so the last dbsegments
     //must be switched to by a collection to make the serialization work (the _KeyValues of the dbsegment are
@@ -138,9 +135,9 @@ function modifyDbSegment(context, asyncDone) {
 
 
 function insertTmpTableToRealTable(context, asyncDone) {
-    var dbSeg = context.oData.dbSegmentLast;
-    var entityType = dbSeg.entityType;
-    var create = (entityType.modifications || {}).create || {};
+    const dbSeg = context.oData.dbSegmentLast;
+    const entityType = dbSeg.entityType;
+    const create = (entityType.modifications || {}).create || {};
     if (create.using) {
         exitProcessor.executeExit(create.using, 'using', 'create', context, asyncDone);
     } else {
@@ -152,7 +149,7 @@ function insertTmpTableToRealTable(context, asyncDone) {
 
 function eventBefore(context, asyncDone) {
     context.logger.info('create', 'event start before');
-    var eventFunction = exitProcessor.eventHandler('before', 'create');
+    const eventFunction = exitProcessor.eventHandler('before', 'create');
     eventFunction(context, function (err, context) {
         context.logger.info('create event end', 'before');
         if (err) {
@@ -164,7 +161,7 @@ function eventBefore(context, asyncDone) {
 
 function eventAfter(context, asyncDone) {
     context.logger.info('create', 'event start after');
-    var eventFunction = exitProcessor.eventHandler('after', 'create');
+    const eventFunction = exitProcessor.eventHandler('after', 'create');
     eventFunction(context, function (err, context) {
         context.logger.info('create event end', 'after');
         if (err) {
@@ -176,7 +173,7 @@ function eventAfter(context, asyncDone) {
 
 function eventPrecommit(context, asyncDone) {
     context.logger.info('create', 'event start precommit');
-    var eventFunction = exitProcessor.eventHandler('precommit', 'create');
+    const eventFunction = exitProcessor.eventHandler('precommit', 'create');
     eventFunction(context, function (err, context) {
         context.logger.info('create event end', 'precommit');
         if (err) {
@@ -188,7 +185,7 @@ function eventPrecommit(context, asyncDone) {
 
 function eventPostCommit(context, asyncDone) {
     context.logger.info('create', 'event start postcommit');
-    var eventFunction = exitProcessor.eventHandler('postcommit', 'create');
+    const eventFunction = exitProcessor.eventHandler('postcommit', 'create');
     eventFunction(context, function (err, context) {
         context.logger.info('create event end', 'postcommit');
         if (err) {
@@ -204,12 +201,13 @@ function setStatus(context, asyncDone) {
 }
 
 function writeLocationHeader(context, asyncDone) {
-    var dbSeg = context.oData.dbSegmentLast;
+    const dbSeg = context.oData.dbSegmentLast;
+    let row;
     try {
-        var location = dbSeg.entityType.name;
-        var keysProperties = dbSeg.getKeysProperties();
-        var rows = dbSeg.getRowsWithGenKey();
-        var row = rows[0];
+        let location = dbSeg.entityType.name;
+        const keysProperties = dbSeg.getKeysProperties();
+        const rows = dbSeg.getRowsWithGenKey();
+        row = rows[0];
 
         location += '(';
         location += keysProperties.map(toValues).filter(v => v !== undefined).join(',');
@@ -226,9 +224,9 @@ function writeLocationHeader(context, asyncDone) {
     }
 
     function toValues(keyProperty, index, array) {
-        //old var value = typeConv.serializeDbValueToUriLiteral(row[index.toString()]/*keys[index]*/, keyProperty);
+        //old const value = typeConv.serializeDbValueToUriLiteral(row[index.toString()]/*keys[index]*/, keyProperty);
 
-        var value;
+        let value;
         if (dbSeg.entityType.kind === model.entityKind.inputParameters || dbSeg.entityType.kind === model.entityKind.calculationView) {
             // for create, update, delete on a calcview, input parameters are optional and are NOT rendered in location url
             //if (dbSeg.entityType._entityType.parameters && dbSeg.entityType._entityType.parameters.viaKey === true) {
@@ -291,7 +289,7 @@ exports.process = function (context, asyncDone) {
         ],
         function (err, context) {
             if (err) {
-                var dbClient = context.db.client;
+                const dbClient = context.db.client;
                 if (dbClient) {
                     //ROLLBack the changes
                     return connect.dbRollback(context, dbClient, function (errDB) {