@sap/xsodata 8.3.0 → 8.3.1

package/CHANGELOG.md

@@ -8,6 +8,10 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/).
 
 ## Unreleased
 
+## [8.3.1] - 2026-02-12
+
+* FOSS related security updates
+
 ## [8.3.0] - 2026-01-09
 
 * FOSS related security updates

package/index.js

@@ -2,8 +2,8 @@
 
 
 //Includes
-var xsodata = require('./lib/xsodata');
-var configuration = require('./lib/configuration');
+const xsodata = require('./lib/xsodata');
+const configuration = require('./lib/configuration');
 
 //API
 module.exports.modes = configuration.modes;

package/lib/configuration.js

@@ -148,6 +148,6 @@ exports.RequestOptions = function RequestOptions(init) {
 
 function check (target) {
     if ((target.mode !== exports.modes.development ) && (target.mode !== exports.modes.productive )) {
-        throw('Invalid mode. See xsodata.modes for possible values.');
+        throw(new Error('Invalid mode. See xsodata.modes for possible values.'));
     }
 }

package/lib/db/connect.js

@@ -93,7 +93,7 @@ function setHanaCloudContext(context, asyncDone) {
 }
 
 function isHanaCloudDb(context, version) {
-    const regex = /^([0-9])+\..*$/;
+    const regex = /^(\d)+\..*$/;
     let result = version.match(regex);
 
     if (result && result[1]) {

package/lib/db/dbSegment.js

@@ -28,6 +28,13 @@ exports.DBS_Navigation = 6; // expanded navigation || selected navigation
 
 exports.DbSegment = DbSegment;
 
+const isSpecialCharacter = (createBy, createAt, modifyBy, modifyAt, property) => {
+    return (createBy && property === 'CREATED_BY') ||
+        (createAt && property === 'CREATED_AT') ||
+        (modifyBy && property === 'MODIFIED_BY') ||
+        (modifyAt && property === 'MODIFIED_AT');
+};
+
 /**
  * Constructs a dbSegment. A dbSegment stores all information to load data from one actual database table
  *
@@ -189,9 +196,9 @@ DbSegment.prototype.isPrincipal = function () {
  * @returns {undefined|{key: string, value: string}}
  */
 DbSegment.prototype.getKeyValue = function (key) {
-    for (let i = 0; i < this._KeyValues.length; i++) {
-        if (this._KeyValues[i].name === key) {
-            return this._KeyValues[i];
+    for (const keyValue of this._KeyValues) {
+        if (keyValue.name === key) {
+            return keyValue;
         }
     }
     return undefined;
@@ -211,23 +218,16 @@ DbSegment.prototype.hasKeyValues = function () {
  */
 DbSegment.prototype.setRecordFromPostPayload = function (context, record) {
     const converterArray = typeConverter.converterFunctions.jsonPayloadToDbName;
-    // const converterArray1 = typeConverter.converterFunctions.uriPayloadToDbName;
     const supportNullable = context.gModel.isNullSupported();
-
     const createBy = this.entityType.getAddAdmindata('create', 'by');
     const createAt = this.entityType.getAddAdmindata('create', 'at');
     const modifyBy = this.entityType.getAddAdmindata('modify', 'by');
     const modifyAt = this.entityType.getAddAdmindata('modify', 'at');
-
     //Loop through the properties of the entityType.
-    //Object.keys(this.entityType.propertiesMap).forEach(function (property) {
     for (let [key, prop] of Object.entries(this.entityType.propertiesMap)) {
         //Load property from record
         const value = record[key];
-
         let dbValue;
-
-
         if (value === undefined) {
             // Use columns default value
             if (prop.KIND === EntityType.entityKind.inputParameters) {
@@ -272,12 +272,9 @@ DbSegment.prototype.setRecordFromPostPayload = function (context, record) {
             const converter = converterArray[typeConverter.dbTypeNameToODataTypeName[dbType]];
             dbValue = converter(value, dbType);
         }
-
         this._recordMapFromPayload[key] = dbValue;
         this._recordNV.push({ name: key, value: dbValue });
     }
-
-
     //set KeyValues
     let max = this.entityType.keys.names.length;
     for (let i = 0; i < max; i++) {
@@ -310,7 +307,6 @@ DbSegment.prototype.setRecordFromPutPayload = function (context, record) {
     const modifyBy = this.entityType.getAddAdmindata('modify', 'by');
     const modifyAt = this.entityType.getAddAdmindata('modify', 'at');
 
-
     //Check if all required properties are send to server
     Object.keys(this.entityType.propertiesMap).forEach(function (property) {
         let dbValue;
@@ -336,10 +332,7 @@ DbSegment.prototype.setRecordFromPutPayload = function (context, record) {
 
                     if (!(
                         // skip special properties
-                        (createBy && property === 'CREATED_BY') ||
-                        (createAt && property === 'CREATED_AT') ||
-                        (modifyBy && property === 'MODIFIED_BY') ||
-                        (modifyAt && property === 'MODIFIED_AT') ||
+                        isSpecialCharacter(createBy, createAt, modifyBy, modifyAt, property) ||
                         // skip optional calc view parameters
                         (this.entityType.propertiesMap[property] && this.entityType.propertiesMap[property].MANDATORY === 0)
                     )) {
@@ -450,8 +443,8 @@ DbSegment.prototype.setLinks = function () {
 DbSegment.prototype.getQKeyProperties = function () {
     let ret = [];
 
-    for (let i = 0; i < this.entityType.keys.names.length; i++) {
-        ret.push(new sqlStatement.TableColumn(this._Alias, this.entityType.keys.names[i]));
+    for (const element of this.entityType.keys.names) {
+        ret.push(new sqlStatement.TableColumn(this._Alias, element));
     }
     return ret;
 };
@@ -534,8 +527,8 @@ DbSegment.prototype.validateInputParameters = function () {
         }
     }
 
-    for (let i = 0; i < this._InputParams.length; i++) {
-        let keyName = this._InputParams[i].key;
+    for (const element of this._InputParams) {
+        let keyName = element.key;
         if (!this.entityType.inputParameters[keyName]) {
             throw new InternalError('Inputparameter ' + keyName + ' unknown');
         }
@@ -545,13 +538,12 @@ DbSegment.prototype.validateInputParameters = function () {
 DbSegment.prototype.validateKeyParameters = function () {
     const keyNames = this.entityType.keyNamesOrdered;
 
-    for (let i = 0; i < keyNames.length; i++) {
-        const keyName = keyNames[i];
-        const property = this.entityType.propertiesMap[keyName];
+    for (const element of keyNames) {
+        const property = this.entityType.propertiesMap[element];
 
         if (property.KIND !== EntityType.entityKind.inputParameters) {
-            if (!this._InputParamsMap.has(keyName)) {
-                throw new InternalError('Missing value for input parameter ' + keyName + '.');
+            if (!this._InputParamsMap.has(element)) {
+                throw new InternalError('Missing value for input parameter ' + element + '.');
             }
         }
     }
@@ -636,9 +628,9 @@ DbSegment.prototype.hasAliasedKeyPropertiesOnCalcView = function() {
 };
 
 DbSegment.prototype.getAliasedKeyPropertyOnCalcView = function(propertyName) {
-    for (let i = 0; i < this._aliasedKeyPropertiesOnCalcView.length; i++) {
-        if (this._aliasedKeyPropertiesOnCalcView[i].property === propertyName) {
-            return this._aliasedKeyPropertiesOnCalcView[i];
+    for (const prop of this._aliasedKeyPropertiesOnCalcView) {
+        if (prop.property === propertyName) {
+            return prop;
         }
     }
     return null;
@@ -702,15 +694,13 @@ DbSegment.prototype.getKeyProperties0123ForOrderBy = function (noTable) {
 
 DbSegment.prototype.getKeyProperties0123ForOrderByCalcView = function (noTable) {
     let ret = [];
-    let keyName;
     let property;
 	let j = -1;
 
     let keyNames = this.entityType.keyNamesOrdered;
 
-    for (let i = 0; i < keyNames.length; i++) {
+    for (const keyName of keyNames) {
 
-        keyName = keyNames[i];
         property = this.entityType.propertiesMap[keyName];
 
 		if (property.KIND !== EntityType.entityKind.inputParameters) {
@@ -765,8 +755,7 @@ DbSegment.prototype.getKeyPropertiesNotSelectedForCreate = function () {
     let propertiesMap = this.entityType.propertiesMap;
 
 
-    for (let i = 0; i < keyNames.length; i++) {
-        let keyName = keyNames[i];
+    for (const keyName of keyNames) {
         if ((this._getSortedSelectProperties().indexOf(keyName) === -1) && (this._SelectedNavigations.indexOf(keyName) === -1)) {
             let property = propertiesMap[keyName];
             let typeString = createPropertyTypeString(property, this.entityType);
@@ -782,11 +771,9 @@ DbSegment.prototype.getKeyPropertiesNotSelectedForCreate = function () {
 DbSegment.prototype.getPropertiesForCreate = function () {
     let ret = [];
 
-    //let properties = this.entityType.properties;
     let properties = this._getSortedSelectProperties();
-    for (let i = 0; i < properties.length; i++) {
-        let property = this.entityType.propertiesMap[properties[i]];
-
+    for (const prop of properties) {
+        let property = this.entityType.propertiesMap[prop];
         ret.push(
             createSqlCreateProperty(property, this.entityType)
         );
@@ -823,9 +810,9 @@ DbSegment.prototype.getOverPropertiesForCreate = function () {
         properties.push(keyProp);
     }
 
-    for (let i = 0; i < properties.length; i++) {
+    for (const property of properties) {
         ret.push(
-            createSqlCreateProperty(properties[i], properties[i]._entityType)
+            createSqlCreateProperty(property, property._entityType)
         );
     }
     return ret;
@@ -856,24 +843,12 @@ function createPropertyTypeString(property, entityType) {
         checkLength(property);
         const scale = property.SCALE ? property.SCALE.toString() : 0;
         typeString += '(' + property.LENGTH.toString() + ',' + scale + ')';
-    } else if (typeString === 'VARCHAR') {
-        checkLength(property);
-        typeString += '(' + property.LENGTH.toString() + ')';
-    } else if (typeString === 'NVARCHAR') {
-        checkLength(property);
-        typeString += '(' + property.LENGTH.toString() + ')';
-    } else if (typeString === 'ALPHANUM') {
-        checkLength(property);
-        typeString += '(' + property.LENGTH.toString() + ')';
-    } else if (typeString === 'CHAR') {
+    } else if (typeString === 'VARCHAR' || typeString === 'NVARCHAR' ||
+        typeString === 'ALPHANUM' || typeString === 'CHAR' ||
+        typeString === 'NCHAR' || typeString === 'VARBINARY') {
+        
         checkLength(property);
         typeString += '(' + property.LENGTH.toString() + ')';
-    } else if (typeString === 'NCHAR') {
-        checkLength(property);
-        typeString += '(' + property.LENGTH.toString() + ')';
-    } else if (typeString === 'VARBINARY') {
-        checkLength(property);
-        typeString += '(' + property.LENGTH.toString() + ')'; // ON
     } else if (typeString === 'SHORTTEXT') {
         typeString = 'NVARCHAR(10)';
     }
@@ -896,19 +871,19 @@ DbSegment.prototype.getPropertiesForSelectCollectInputParameters = function (noT
     let selectProperty;
     let propertyType;
 
-    for (let i = 0; i < properties.length; i++) {
-        property = this.entityType.propertiesMap[properties[i]];
+    for (const prop of properties) {
+        property = this.entityType.propertiesMap[prop];
         selectProperty = null;
 
         if (property.KIND === EntityType.entityKind.inputParameters) {
             let paramValue = this.getParameterValue(property.COLUMN_NAME);
-            selectProperty = new sqlStatement.ParameterSelectProperty(noTable ? null : this._Alias, properties[i], null, paramValue);
+            selectProperty = new sqlStatement.ParameterSelectProperty(noTable ? null : this._Alias, prop, null, paramValue);
             selectProperty._property = property;
             selectProperty._dbType = property.aggregate ? null : property.DATA_TYPE_NAME;
             retInput.push(selectProperty);
         } else {
             propertyType = property.aggregate ? null : property.DATA_TYPE_NAME;
-            selectProperty = new sqlStatement.SelectProperty(noTable ? null : this._Alias, properties[i], propertyType, null);
+            selectProperty = new sqlStatement.SelectProperty(noTable ? null : this._Alias, prop, propertyType, null);
             ret.push(selectProperty);
         }
     }
@@ -932,16 +907,16 @@ DbSegment.prototype.getPropertiesForSelect = function (noTable) {
         selectProperty,
         propertyType;
 
-    for (let i = 0; i < properties.length; i++) {
-        property = this.entityType.propertiesMap[properties[i]];
+    for (const prop of properties) {
+        property = this.entityType.propertiesMap[prop];
         selectProperty = null;
 
         if (property.KIND === EntityType.entityKind.inputParameters) {
             let paramValue = this.getParameterValue(property.COLUMN_NAME);
-            selectProperty = new sqlStatement.ParameterSelectProperty(noTable ? null : this._Alias, properties[i], null, paramValue);
+            selectProperty = new sqlStatement.ParameterSelectProperty(noTable ? null : this._Alias, prop, null, paramValue);
         } else {
             propertyType = property.aggregate ? null : property.DATA_TYPE_NAME;
-            selectProperty = new sqlStatement.SelectProperty(noTable ? null : this._Alias, properties[i], propertyType, null);
+            selectProperty = new sqlStatement.SelectProperty(noTable ? null : this._Alias, prop, propertyType, null);
         }
 
         ret.push(selectProperty);
@@ -961,11 +936,11 @@ DbSegment.prototype.getSelectFragmentsFromAllProperties = function (noTable) {
     let ret = [];
     let properties = this._getSortedSelectProperties();
 
-    for (let i = 0; i < properties.length; i++) {
-        const property = this.entityType.propertiesMap[properties[i]];
+    for (const prop of properties) {
+        const property = this.entityType.propertiesMap[prop];
         // this include property KIND === inputParameters
         const propertyType = property.aggregate ? null : property.DATA_TYPE_NAME;
-        ret.push(new sqlStatement.SelectProperty(noTable ? null : this._Alias, properties[i], propertyType, null));
+        ret.push(new sqlStatement.SelectProperty(noTable ? null : this._Alias, prop, propertyType, null));
     }
     return ret;
 };
@@ -980,16 +955,16 @@ DbSegment.prototype.getSelectFragmentsForOverProperties = function (rId) {
     let over = this.getOver();
     let properties = [];
 
-    for (let i = 0; i < over.principal.length; i++) {
-        properties.push(over.principal[i]);
+    for (const property of over.principal) {
+        properties.push(property);
     }
 
-    for (let i = 0; i < over.dependent.length; i++) {
-        properties.push(over.dependent[i]);
+    for (const property of over.dependent) {
+        properties.push(property);
     }
 
-    for (let i = 0; i < properties.length; i++) {
-        selectProperty = new sqlStatement.SelectProperty(rId, properties[i], null);
+    for (const property of properties) {
+        selectProperty = new sqlStatement.SelectProperty(rId, property, null);
         ret.push(
             selectProperty
         );
@@ -1130,19 +1105,16 @@ function concatJoinProperties(selectedProps, joinProps, table) {
 DbSegment.prototype.getNavPropertiesForCreate = function () {
     let ret = [];
 
-    //let properties = this.entityType.properties;
     let properties = this._SelectedNavigations;
-    for (let i = 0; i < properties.length; i++) {
-        let navPropertyName = properties[i];
-
+    for (const navPropertyName of properties) {
         let navDbSeg = this._relevantNavigationSegments[navPropertyName];
         if (!navDbSeg) {
             continue;
         }
         let propertyNames = navDbSeg.getFrom().joinproperties;
 
-        for (let ii = 0; ii < propertyNames.length; ii++) {
-            let property = this.entityType.propertiesMap[propertyNames[ii]];
+        for (const prop of propertyNames) {
+            let property = this.entityType.propertiesMap[prop];
             let propertyName = property.COLUMN_NAME;
             if (this._getSortedSelectProperties().indexOf(propertyName) > -1) {
                 continue;
@@ -1176,10 +1148,8 @@ DbSegment.prototype.getNavPropertiesForCreate = function () {
 DbSegment.prototype.getNavPropertiesForSelect = function (noTable) {
     let ret = [];
 
-    //let properties = this.entityType.properties;
     let properties = this._SelectedNavigations;
-    for (let i = 0; i < properties.length; i++) {
-        let navPropertyName = properties[i];
+    for (const navPropertyName of properties) {
         if (this._getSortedSelectProperties().indexOf(navPropertyName) > -1) {
             continue;
         }
@@ -1190,15 +1160,15 @@ DbSegment.prototype.getNavPropertiesForSelect = function (noTable) {
         let propertyNames = navDbSeg.getFrom().joinproperties;
 
 
-        for (let ii = 0; ii < propertyNames.length; ii++) {
-            let property = this.entityType.propertiesMap[propertyNames[ii]];
+        for (const prop of propertyNames) {
+            let property = this.entityType.propertiesMap[prop];
             let propertyName = property.COLUMN_NAME;
             if (this._getSortedSelectProperties().indexOf(propertyName) > -1) {
                 continue;
             }
 
             ret.push(
-                new sqlStatement.SelectProperty(noTable ? null : this._Alias, propertyNames[ii], null)
+                new sqlStatement.SelectProperty(noTable ? null : this._Alias, prop, null)
             );
         }
     }
@@ -1214,13 +1184,11 @@ DbSegment.prototype.getNavPropertiesForSelect = function (noTable) {
 DbSegment.prototype.getNonKeyNonSelectedProperties4Expansion = function () {
     //
     let ret = [];
-    for (let i = 0; i < this._ExpandedNavigations.length; i++) {
-        let exp = this._relevantNavigationSegments[this._ExpandedNavigations[i]];
+    for (const expandedNavigation of this._ExpandedNavigations) {
+        let exp = this._relevantNavigationSegments[expandedNavigation];
         let from = exp._end.from;
 
-        for (let ii = 0; ii < from.joinproperties.length; ii++) {
-            let joinProperty = from.joinproperties[ii];
-
+        for (const joinProperty of from.joinproperties) {
             //Non Key
             if (this.entityType.keys.names.indexOf(joinProperty) === -1) {
                 if (this._getSortedSelectProperties().indexOf(joinProperty) === -1) {
@@ -1241,14 +1209,9 @@ DbSegment.prototype.getNonKeyNonSelectedProperties4Expansion = function () {
  * @returns {sqlStatement.TableColumn[]}
  */
 DbSegment.prototype.getQNonKeyProperties = function () {
-
     let ret = [];
-    let propertyName;
-
-
-    for (let i = 0; i < this._getSortedSelectProperties().length; i++) {
-        propertyName = this._getSortedSelectProperties()[i];
 
+    for (const propertyName of this._getSortedSelectProperties()) {
         if (this.entityType.keys.names.indexOf(propertyName) === -1) {
             ret.push(new sqlStatement.TableColumn(this._Alias, propertyName));
         }
@@ -1265,13 +1228,10 @@ DbSegment.prototype.getQNonKeyProperties = function () {
 DbSegment.prototype.getQNonKeyPropertiesForSelect = function () {
     let ret = [];
     let propertiesMap = this.entityType.propertiesMap;
-    let propertyName;
     let property;
     let propertyType;
 
-    for (let i = 0; i < this._getSortedSelectProperties().length; i++) {
-        propertyName = this._getSortedSelectProperties()[i];
-
+    for (const propertyName of this._getSortedSelectProperties()) {
         if (this.entityType.keys.names.indexOf(propertyName) === -1) {
             property = propertiesMap[propertyName];
             propertyType = property.aggregate ? null : property.DATA_TYPE_NAME;
@@ -1314,12 +1274,12 @@ DbSegment.prototype.getQForeignKeyProperties = function () {
  */
 DbSegment.prototype.getQKeyWithValues = function (alias) {
     let ret = [];
-    for (let i = 0; i < this._KeyValues.length; i++) {
-        let etProperty = this.entityType.propertiesMap[this._KeyValues[i].name];
+    for (const keyValue of this._KeyValues) {
+        let etProperty = this.entityType.propertiesMap[keyValue.name];
         if (etProperty.KIND !== EntityType.entityKind.inputParameters) {
             ret.push({
-                key: new typedObjects.Property(this._KeyValues[i].name, alias ? alias : this._Alias, etProperty.DATA_TYPE_NAME),
-                value: this._KeyValues[i].value
+                key: new typedObjects.Property(keyValue.name, alias ? alias : this._Alias, etProperty.DATA_TYPE_NAME),
+                value: keyValue.value
             });
         }
     }
@@ -1329,11 +1289,11 @@ DbSegment.prototype.getQKeyWithValues = function (alias) {
 DbSegment.prototype.getQKeyWithValuesDB = function (alias) {
     let ret = [];
 
-    for (let i = 0; i < this._KeyValues.length; i++) {
+    for (const keyValue of this._KeyValues) {
 
         ret.push({
-            key: new typedObjects.Property(this._KeyValues[i].name, alias ? alias : this._Alias, this._KeyValues[i].dbType), //OK
-            value: new typedObjects.DbValue(this._KeyValues[i].value, this._KeyValues[i].dbType) // Ok
+            key: new typedObjects.Property(keyValue.name, alias ? alias : this._Alias, keyValue.dbType), //OK
+            value: new typedObjects.DbValue(keyValue.value, keyValue.dbType) // Ok
         });
     }
     return ret;
@@ -1364,7 +1324,6 @@ DbSegment.prototype.getQForeignKeyWithValues = function (alias) {
 
 DbSegment.prototype.getOverPropertiesWithValues = function () {
     let ret = [], dbSeg, refDBSeg;
-    //var a = (alias ? alias : this._Alias);
 
     if (this.previousDBSegment) {
         refDBSeg = this.previousDBSegment;
@@ -1395,19 +1354,6 @@ DbSegment.prototype.getOverPropertiesWithValues = function () {
     return ret;
 };
 
-/*
-DbSegment.prototype.dump = function (logger, addText) {
-    let text = addText || 'DbSegment: ';
-    let filter = function (key, value) {
-        if (key === '__metadata' || key === 'previousDBSegment') {
-            return undefined;
-        }
-        return value;
-    };
-    logger.debug(text + JSON.stringify(this, filter, 4));
-};
-*/
-
 /**
  * Add a  navigationDbSegment to the expand tree
  *
@@ -1542,7 +1488,6 @@ DbSegment.prototype.getRowsWithGenKey = function getRowsWithGenKey() {
     if (genKey) {
         if (!this._rowsWithGenKey) {
             this._rowsWithGenKey = this.sql.rows.map(function (row, index) {
-                // let cloneRow = xsODataUtils.clone(row);
                 let cloneRow = {};
                 for (let key of Object.keys(row)) {
                     cloneRow[key] = row[key];

package/lib/db/dbVersionChecks.js

@@ -28,21 +28,15 @@ module.exports.shouldNotCleanTmpTables = (context) => {
         fixed in release 2.00.030.00 ---> 2.00.030.00.1522209842 HANA 2.0 SPS 03; revision SPS number 0
         */
 
-    if ((utils.compareDBVersion(version, '1.00.122.16.1520578817', utils.SAME_3RD_SPS) === utils.VERSION_A_COVERS_VERSION_B) ||
+    return (utils.compareDBVersion(version, '1.00.122.16.1520578817', utils.SAME_3RD_SPS) === utils.VERSION_A_COVERS_VERSION_B) ||
         (utils.compareDBVersion(version, '1.00.123.00.0000000000', utils.SAME_2ND) === utils.VERSION_A_COVERS_VERSION_B) ||
         (utils.compareDBVersion(version, '1.01.000.00.0000000000', utils.SAME_1ST_HANA_REL) === utils.VERSION_A_COVERS_VERSION_B) ||
-
         (utils.compareDBVersion(version, '2.00.012.04.1517989041', utils.SAME_3RD_SPS) === utils.VERSION_A_COVERS_VERSION_B) ||
-
         (utils.compareDBVersion(version, '2.00.024.00.1520347815', utils.SAME_3RD_SPS) === utils.VERSION_A_COVERS_VERSION_B) ||
-
         (utils.compareDBVersion(version, '2.00.030.00.1522209842', utils.SAME_3RD_SPS) === utils.VERSION_A_COVERS_VERSION_B) ||
         (utils.compareDBVersion(version, '2.00.031.00.0000000000', utils.SAME_2ND) === utils.VERSION_A_COVERS_VERSION_B) ||
         (utils.compareDBVersion(version, '2.01.000.00.0000000000', utils.SAME_1ST_HANA_REL) === utils.VERSION_A_COVERS_VERSION_B) ||
-        (utils.compareDBVersion(version, '3.00.000.00.0000000000', utils.NO_FIX_PART) === utils.VERSION_A_COVERS_VERSION_B)) {
+        (utils.compareDBVersion(version, '3.00.000.00.0000000000', utils.NO_FIX_PART) === utils.VERSION_A_COVERS_VERSION_B);
 
-        return true;// skip cleanup
-    }
-    return false; // do cleanup
 };
 

package/lib/handlerConfiguration.js

@@ -4,8 +4,8 @@
  * @module configuration
  */
 
-var utils = require('./utils/utils');
-var configuration = require('./configuration');
+const utils = require('./utils/utils');
+const configuration = require('./configuration');
 
 /**
  * Holds the runtime configuration for the OData handler.