meadow-endpoints 4.0.5 → 4.0.9

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "meadow-endpoints",
-  "version": "4.0.5",
+  "version": "4.0.9",
   "description": "Automatic API endpoints for Meadow data.",
   "main": "source/Meadow-Endpoints.js",
   "scripts": {
@@ -12,7 +12,8 @@
     "tests": "./node_modules/.bin/mocha -u tdd --exit -R spec --grep",
     "build": "./node_modules/.bin/gulp build",
     "docker-dev-build-image": "docker build ./ -f Dockerfile_LUXURYCode -t retold/meadow-endpoints:local",
-    "docker-dev-run": "docker run -it -d --name meadow-endpoints-dev -p 12343:8080 -p 12305:3306 -p 18086:8086 -v \"$PWD/.config:/home/coder/.config\"  -v \"$PWD:/home/coder/meadow-endpoints\" -u \"$(id -u):$(id -g)\" -e \"DOCKER_USER=$USER\" retold/meadow-endpoints:local"
+    "docker-dev-run": "docker run -it -d --name meadow-endpoints-dev -p 12343:8080 -p 12305:3306 -p 18086:8086 -v \"$PWD/.config:/home/coder/.config\"  -v \"$PWD:/home/coder/meadow-endpoints\" -u \"$(id -u):$(id -g)\" -e \"DOCKER_USER=$USER\" retold/meadow-endpoints:local",
+    "check": "npx -p typescript tsc --noEmit"
   },
   "mocha": {
     "diff": true,
@@ -46,31 +47,35 @@
   },
   "homepage": "https://github.com/stevenvelozo/meadow-endpoints",
   "devDependencies": {
-    "alasql": "^4.0.0",
-    "browserify": "^17.0.0",
-    "chai": "4.3.7",
-    "chance": "^1.1.11",
-    "gulp": "^4.0.2",
+    "alasql": "^4.17.0",
+    "better-sqlite3": "^12.6.2",
+    "browserify": "^17.0.1",
+    "chai": "6.2.2",
+    "chance": "^1.1.13",
+    "gulp": "^5.0.1",
     "gulp-babel": "^8.0.0",
     "gulp-sourcemaps": "^3.0.0",
     "gulp-terser": "^2.1.0",
     "gulp-util": "^3.0.8",
-    "mocha": "10.2.0",
-    "mysql2": "^3.3.1",
-    "nyc": "^15.1.0",
-    "orator-serviceserver-restify": "^1.0.4",
-    "papaparse": "^5.4.1",
-    "supertest": "^6.3.3",
+    "meadow-connection-sqlite": "file:../meadow-connection-sqlite",
+    "mocha": "11.7.5",
+    "mysql2": "^3.16.3",
+    "nyc": "^17.1.0",
+    "orator-serviceserver-restify": "^2.0.5",
+    "papaparse": "^5.5.3",
+    "supertest": "^7.2.2",
+    "typescript": "^5.9.3",
     "vinyl-buffer": "^1.0.1",
     "vinyl-source-stream": "^2.0.0",
-    "why-is-node-running": "^2.2.2"
+    "why-is-node-running": "^3.2.2"
   },
   "dependencies": {
+    "async": "3.2.6",
+    "fable": "^3.1.55",
     "JSONStream": "^1.3.5",
-    "async": "3.2.4",
-    "fable": "^3.0.37",
-    "meadow": "^2.0.4",
-    "orator": "^3.0.11",
-    "underscore": "^1.13.6"
+    "meadow": "^2.0.22",
+    "meadow-filter": "^1.0.10",
+    "orator": "^5.0.2",
+    "underscore": "^1.13.7"
   }
 }

package/source/Meadow-Endpoints.js

@@ -101,10 +101,17 @@ class MeadowEndpoints
 		});
 	}
 
+    /**
+     * @return {import('./controller/Meadow-Endpoints-Controller-Base.js')} The controller instance
+     */
 	get controller()
 	{
 		return this._Controller;
 	}
+
+    /**
+     * @param {import('./controller/Meadow-Endpoints-Controller-Base.js')} pController - The controller instance
+     */
 	set controller(pController)
 	{
 		this._Controller = pController;
@@ -231,4 +238,4 @@ module.exports = MeadowEndpoints;
 module.exports.new = autoConstruct;
 
 module.exports.Meadow = libMeadow;
-module.exports.BaseController = libMeadowEndpointsControllerBase;
+module.exports.BaseController = libMeadowEndpointsControllerBase;

package/source/controller/Meadow-Endpoints-Controller-Base.js

@@ -11,25 +11,29 @@ const libMeadowEndpointsStreamRecordArray = require('./utility/Meadow-Endpoints-
 
 class MeadowEndpointControllerBase
 {
+    /**
+     * @param {import('../Meadow-Endpoints.js')} pMeadowEndpoints
+     */
 	constructor(pMeadowEndpoints)
 	{
 		this.DAL = pMeadowEndpoints.DAL;
-		this.ControllerOptions = pMeadowEndpoints._ControllerOptions
+		this.ControllerOptions = pMeadowEndpoints._ControllerOptions;
 
 		// Application Services
-		this._Settings = false;
-		this._LogController = false;
+		this._Settings = null;
+		this._LogController = null;
 
 		// Logic and Behavior
-		this._BehaviorInjectionController = false;
-		this._ErrorController = false;
+		this._BehaviorInjectionController = null;
+		this._ErrorController = null;
 
 		// Internal async utility functions
 		this.waterfall = this.DAL.fable.Utility.waterfall;
 		this.eachLimit = this.DAL.fable.Utility.eachLimit;
 		this.extend = this.DAL.fable.Utility.extend;
 
-		if ((typeof(pControllerOptions) != 'object') || pControllerOptions.hasOwnProperty('ControllerClass'))
+		//FIXME: not sure what this is supposed to do, it was broken before
+		//if ((typeof(this.ControllerOptions) != 'object') || this.ControllerOptions.hasOwnProperty('ControllerClass'))
 		{
 			this.initializeDefaultUnsetControllers(this);
 		}
@@ -40,6 +44,9 @@ class MeadowEndpointControllerBase
 		this._StreamRecordArray = new libMeadowEndpointsStreamRecordArray(this);
 	}
 
+    /**
+     * @param {import('./Meadow-Endpoints-Controller-Base.js')} pController
+     */
 	initializeDefaultUnsetControllers(pController)
 	{
 		// Application Services
@@ -158,4 +165,4 @@ module.exports.BaseBehaviorInjectionController = libBaseBehaviorInjectionControl
 
 module.exports.BaseFilterParser = libMeadowEndpointsFilterParser;
 module.exports.BaseSessionMarshaler = libMeadowEndpointsSessionMarshaler;
-module.exports.BaseStreamRecordArray = libMeadowEndpointsStreamRecordArray;
+module.exports.BaseStreamRecordArray = libMeadowEndpointsStreamRecordArray;

package/source/controller/components/Meadow-Endpoints-Controller-BehaviorInjection.js

@@ -4,6 +4,9 @@ const libUnderscore = require('underscore');
 
 class MeadowEndpointsControllerBehaviorInjectionBase
 {
+    /**
+     * @param {import('../Meadow-Endpoints-Controller-Base.js')} pController
+     */
     constructor(pController)
 	{
         this._Controller = pController;
@@ -127,4 +130,4 @@ class MeadowEndpointsControllerBehaviorInjectionBase
 	}
 }
 
-module.exports = MeadowEndpointsControllerBehaviorInjectionBase;
+module.exports = MeadowEndpointsControllerBehaviorInjectionBase;

package/source/controller/components/Meadow-Endpoints-Controller-Error.js

@@ -1,5 +1,8 @@
 class MeadowEndpointsControllerErrorBase
 {
+    /**
+     * @param {import('../Meadow-Endpoints-Controller-Base.js')} pController
+     */
     constructor(pController)
 	{
         this._Controller = pController;
@@ -115,4 +118,4 @@ class MeadowEndpointsControllerErrorBase
 	}
 }
 
-module.exports = MeadowEndpointsControllerErrorBase;
+module.exports = MeadowEndpointsControllerErrorBase;

package/source/controller/components/Meadow-Endpoints-Controller-Log.js

@@ -1,5 +1,8 @@
 class MeadowEndpointsControllerLogBase
 {
+    /**
+     * @param {import('../Meadow-Endpoints-Controller-Base.js')} pController
+     */
     constructor(pController)
 	{
         this._Controller = pController;
@@ -100,4 +103,4 @@ class MeadowEndpointsControllerLogBase
 	}
 }
 
-module.exports = MeadowEndpointsControllerLogBase;
+module.exports = MeadowEndpointsControllerLogBase;

package/source/controller/utility/Meadow-Endpoints-Filter-Parser.js

@@ -6,220 +6,29 @@
 * @author Steven Velozo <steven@velozo.com>
 * @module Meadow
 */
-/**
-* Parse GET-passed Filter Strings, turn the results into proper Meadow query stanzas
-
- Take the filter and return an array of filter instructions
- Basic instruction anatomy:
-       INSTRUCTION~FIELD~OPERATOR~VALUE
- FOP - Filter Open Paren
-       FOP~0~(~0
- FCP - Filter Close Paren
-       FCP~0~)~0
- FBV - Filter By Value (left-side AND connected)
-       FBV~Category~EQ~Books
-       Possible comparisons:
-       * EQ - Equals To (=)
-       * NE - Not Equals To (!=)
-       * GT - Greater Than (>)
-       * GE - Greater Than or Equals To (>=)
-       * LT - Less Than (<)
-       * LE - Less Than or Equals To (<=)
-       * LK - Like (Like)
-       * IN - Is NULL
-       * NN - Is NOT NULL
-       * INN - IN list
- FBVOR - Filter By Value (left-side OR connected)
- FBL - Filter By List (value list, separated by commas)
-       FBL~Category~EQ~Books,Movies
- FBD - Filter by Date (exclude time)
-       FBD~UpdateDate~EQ~2015-10-01
- FSF - Filter Sort Field
-       FSF~Category~ASC~0
-       FSF~Category~DESC~0
- FDST - Filter by Distinct (adds distinct keyword to Read and Count queries)
-       FDST~0~0~0~
 
- This means: FBV~Category~EQ~Books~FBV~PublishedYear~GT~2000~FSF~PublishedYear~DESC~0
-             Filters down to ALL BOOKS PUBLUSHED AFTER 2000 IN DESCENDING ORDER
-*/
+const { parse } = require('meadow-filter');
 
 class MeadowEndpointsFilterParser
 {
+    /**
+     * @param {import('../Meadow-Endpoints-Controller-Base.js')} pController - The controller instance to which this parser belongs.
+     */
     constructor(pController)
 	{
         this._Controller = pController;
     }
 
-    // Get the comparison operator for use in a query stanza
-    getFilterComparisonOperator(pFilterOperator)
-    {
-        let tmpOperator = '=';
-        switch(pFilterOperator)
-        {
-            case 'EQ':
-                tmpOperator = '=';
-                break;
-            case 'NE':
-                tmpOperator = '!=';
-                break;
-            case 'GT':
-                tmpOperator = '>';
-                break;
-            case 'GE':
-                tmpOperator = '>=';
-                break;
-            case 'LT':
-                tmpOperator = '<';
-                break;
-            case 'LE':
-                tmpOperator = '<=';
-                break;
-            case 'LK':
-                tmpOperator = 'LIKE';
-                break;
-            case 'NLK':
-                tmpOperator = 'NOT LIKE';
-                break;
-            case 'IN':
-                tmpOperator = 'IS NULL';
-                break;
-            case 'NN':
-                tmpOperator = 'IS NOT NULL';
-                break;
-            case 'INN':
-                tmpOperator = 'IN';
-                break;
-            case 'FOP':
-                tmpOperator = '(';
-                break;
-            case 'FCP':
-                tmpOperator = ')';
-                break;
-        }
-        return tmpOperator;
-    }
-
-    addFilterStanzaToQuery(pFilterStanza, pQuery)
-    {
-        if (!pFilterStanza.Instruction)
-        {
-            return false;
-        }
-
-        switch(pFilterStanza.Instruction)
-        {
-            case 'FBV':   // Filter by Value (left-side AND)
-                pQuery.addFilter(pFilterStanza.Field, pFilterStanza.Value, this.getFilterComparisonOperator(pFilterStanza.Operator), 'AND');
-                break;
-
-            case 'FBVOR': // Filter by Value (left-side OR)
-                pQuery.addFilter(pFilterStanza.Field, pFilterStanza.Value, this.getFilterComparisonOperator(pFilterStanza.Operator), 'OR');
-                break;
-
-            case 'FBL':   // Filter by List (left-side AND)
-                // Just split the value by comma for now.  May want to revisit better characters or techniques later.
-                pQuery.addFilter(pFilterStanza.Field, pFilterStanza.Value.split(','), this.getFilterComparisonOperator(pFilterStanza.Operator), 'AND');
-                break;
-
-            case 'FBLOR': // Filter by List (left-side OR)
-                // Just split the value by comma for now.  May want to revisit better characters or techniques later.
-                pQuery.addFilter(pFilterStanza.Field, pFilterStanza.Value.split(','), this.getFilterComparisonOperator(pFilterStanza.Operator), 'OR');
-                break;
-
-            case 'FBD': // Filter by Date (exclude time)
-                pQuery.addFilter(`DATE(${pFilterStanza.Field})`, pFilterStanza.Value.split(','), this.getFilterComparisonOperator(pFilterStanza.Operator), 'AND', pFilterStanza.Field);
-                break;
-
-            case 'FBDOR': // Filter by Date (exclude time)
-                pQuery.addFilter(`DATE(${pFilterStanza.Field})`, pFilterStanza.Value.split(','), this.getFilterComparisonOperator(pFilterStanza.Operator), 'OR', pFilterStanza.Field);
-                break;
-
-            case 'FSF':   // Filter Sort Field
-                const tmpSortDirection = (pFilterStanza.Operator === 'DESC') ? 'Descending' : 'Ascending';
-                pQuery.addSort({ Column: pFilterStanza.Field, Direction: tmpSortDirection });
-                break;
-
-            case 'FOP':   // Filter Open Paren
-                pQuery.addFilter('', '', '(');
-                break;
-
-            case 'FCP':   // Filter Close Paren
-                pQuery.addFilter('', '', ')');
-                break;
-
-            case 'FDST':   // Filter Distinct
-                // ensure we don't break if using an older foxhound version
-                if (pQuery.setDistinct)
-                {
-                    pQuery.setDistinct(true);
-                }
-                break;
-
-            default:
-                //console.log('Unparsable filter stanza.');
-                return false;
-                break;
-        }
-
-        // Be paranoid about the instruction
-        pFilterStanza.Instruction = false;
-        return true;
-    };
-
+    /**
+     * @param {string} pFilterString - The filter string to parse.
+     * @param {any} pQuery - The foxhound query object to populate.
+     *
+     * @return {boolean} - True if the filter was parsed successfully, false otherwise.
+     */
     parseFilter(pFilterString, pQuery)
     {
-        if (typeof(pFilterString) !== 'string')
-        {
-            return false;
-        }
-
-        const tmpFilterTerms = pFilterString.split('~');
-
-        if (tmpFilterTerms.length < 4)
-        {
-            return true;
-        }
-
-        let tmpFilterStanza = { Instruction: false };
-
-        for (let i = 0; i < tmpFilterTerms.length; i++)
-        {
-            switch(i % 4)
-            {
-                case 0:  // INSTRUCTION
-                    this.addFilterStanzaToQuery(tmpFilterStanza, pQuery);
-                    //console.log(i+' Instruction: '+tmpFilterTerms[i]);
-                    tmpFilterStanza = (
-                    {
-                        Instruction: tmpFilterTerms[i],
-                        Field: '',
-                        Operator: '',
-                        Value: ''
-                    });
-                    break;
-
-                case 1:  // FIELD
-                    //console.log(i+' Field:       '+tmpFilterTerms[i]);
-                    tmpFilterStanza.Field = tmpFilterTerms[i];
-                    break;
-
-                case 2:  // OPERATOR
-                    //console.log(i+' Operator:    '+tmpFilterTerms[i]);
-                    tmpFilterStanza.Operator = tmpFilterTerms[i];
-                    break;
-
-                case 3:  // VALUE
-                    //console.log(i+' Value:       '+tmpFilterTerms[i]);
-                    tmpFilterStanza.Value = tmpFilterTerms[i];
-                    break;
-            }
-        }
-
-        this.addFilterStanzaToQuery(tmpFilterStanza, pQuery);
-
-        return true;
-    };
+        return parse(pFilterString, pQuery);
+    }
 }
 
-module.exports = MeadowEndpointsFilterParser;
+module.exports = MeadowEndpointsFilterParser;

package/source/controller/utility/Meadow-Endpoints-Session-Marshaler.js

@@ -1,5 +1,8 @@
 class MeadowEndpointsSessionMarshaler
 {
+    /**
+     * @param {import('../Meadow-Endpoints-Controller-Base.js')} pController
+     */
     constructor(pController)
 	{
         this._Controller = pController;
@@ -8,11 +11,12 @@ class MeadowEndpointsSessionMarshaler
     getSessionData(pRequest)
     {
         let tmpSession = Object.assign({}, this._Controller.settings.MeadowEndpointsDefaultSessionObject);
+        let tmpHeaderSessionString;
 
         switch (this._Controller.settings.MeadowEndpointsSessionDataSource || 'Request')
         {
             default:
-                this._LogController.warn(`Unknown session source configured: ${_SessionDataSource} - defaulting to Request for backward compatibility`);
+                this._Controller.log.warn(`Unknown session source configured: ${this._Controller.settings.MeadowEndpointsSessionDataSource} - defaulting to Request for backward compatibility`);
             case 'Request':
                 // noop - already set by orator-session
                 tmpSession = this._Controller.extend(tmpSession, pRequest.UserSession);
@@ -22,17 +26,17 @@ class MeadowEndpointsSessionMarshaler
             case 'Header':
                 try
                 {
-                    const tmpHeaderSessionString = pRequest.headers['x-trusted-session'];
+                    tmpHeaderSessionString = pRequest.headers['x-trusted-session'];
                     if (!tmpHeaderSessionString)
                     {
                         break;
                     }
-                    tmpHeaderSession = JSON.parse(tmpHeaderSessionString);
-                    tmpSession = this._Controller.extend(tmpSession, pRequest.tmpHeaderSession);
+                    const tmpHeaderSession = JSON.parse(tmpHeaderSessionString);
+                    tmpSession = this._Controller.extend(tmpSession, tmpHeaderSession);
                 }
                 catch (pError)
                 {
-                    this._LogController.error(`Meadow Endpoints attempted to process a Header Session String with value [${tmpHeaderSessionString}] and failed -- likely culprit is bad JSON.`)
+                    this._Controller.log.error(`Meadow Endpoints attempted to process a Header Session String with value [${tmpHeaderSessionString}] and failed -- likely culprit is bad JSON.`)
                 }
                 break;
         }
@@ -45,4 +49,4 @@ class MeadowEndpointsSessionMarshaler
     }
 }
 
-module.exports = MeadowEndpointsSessionMarshaler;
+module.exports = MeadowEndpointsSessionMarshaler;

package/source/controller/utility/Meadow-Endpoints-Stream-RecordArray.js

@@ -6,6 +6,9 @@ const JSONStream = require('JSONStream');
 
 class MeadowEndpointsStreamRecordArray
 {
+    /**
+     * @param {import('../Meadow-Endpoints-Controller-Base.js')} pController
+     */
     constructor(pController)
 	{
         this._Controller = pController;
@@ -53,7 +56,7 @@ class MeadowEndpointsStreamRecordArray
 		libAsyncEachSeries(this.chunk(pRecords, 1000), (pRecordChunk, fNext) =>
 		{
 			pRecordChunk.forEach(recordJsonMarshaller.write);
-			setImmediate(fNext);
+			setTimeout(fNext, 0);
 		},
 		(error) =>
 		{
@@ -63,4 +66,4 @@ class MeadowEndpointsStreamRecordArray
 	}
 }
 
-module.exports = MeadowEndpointsStreamRecordArray;
+module.exports = MeadowEndpointsStreamRecordArray;

package/source/endpoints/delete/Meadow-Endpoint-Undelete.js

@@ -23,7 +23,7 @@ const doAPIEndpointUndelete = function(pRequest, pResponse, fNext)
 	// TODO: Decide if we want to keep this pattern similar to Delete, or, if we want to change it to allow bulk undeletes.
 	if (tmpIDRecord < 1)
 	{
-		return fStageComplete(this.ErrorHandler.getError('Record undelete failure - a valid record ID is required.', 500));
+		return fNext(this.ErrorHandler.getError('Record undelete failure - a valid record ID is required.', 500));
 	}
 
 	tmpRequestState.RecordCount = {Count:0};
@@ -105,4 +105,4 @@ const doAPIEndpointUndelete = function(pRequest, pResponse, fNext)
 	);
 };
 
-module.exports = doAPIEndpointUndelete;
+module.exports = doAPIEndpointUndelete;

package/source/endpoints/read/Meadow-Endpoint-ReadDistinctList.js

@@ -16,7 +16,9 @@ const doAPIEndpointReadDistinct = function(pRequest, pResponse, fNext)
 		{
 			tmpRequestState.Query = this.DAL.query.setDistinct(true);
 
+			/** @type {number | boolean} */
 			let tmpCap = false;
+			/** @type {number | boolean} */
 			let tmpBegin = false;
 			if (typeof(pRequest.params.Begin) === 'string' ||
 				typeof(pRequest.params.Begin) === 'number')
@@ -89,4 +91,4 @@ const doAPIEndpointReadDistinct = function(pRequest, pResponse, fNext)
 	});
 };
 
-module.exports = doAPIEndpointReadDistinct;
+module.exports = doAPIEndpointReadDistinct;

package/source/endpoints/read/Meadow-Endpoint-ReadLiteList.js

@@ -16,7 +16,9 @@ const doAPIEndpointReadLite = function(pRequest, pResponse, fNext)
 				tmpRequestState.Query = this.DAL.query;
 				// TODO: Limit the query to the columns we need for the templated expression
 
+				/** @type {number | boolean} */
 				var tmpCap = false;
+				/** @type {number | boolean} */
 				var tmpBegin = false;
 				if (typeof(pRequest.params.Begin) === 'string' ||
 					typeof(pRequest.params.Begin) === 'number')
@@ -82,4 +84,4 @@ const doAPIEndpointReadLite = function(pRequest, pResponse, fNext)
 	);
 };
 
-module.exports = doAPIEndpointReadLite;
+module.exports = doAPIEndpointReadLite;

package/source/endpoints/read/Meadow-Endpoint-ReadSelectList.js

@@ -12,7 +12,9 @@ const doAPIEndpointReadSelectList = function(pRequest, pResponse, fNext)
 			{
 				tmpRequestState.Query = this.DAL.query;
 
+				/** @type {number | boolean} */
 				var tmpCap = false;
+				/** @type {number | boolean} */
 				var tmpBegin = false;
 				if (typeof(pRequest.params.Begin) === 'string' ||
 					typeof(pRequest.params.Begin) === 'number')
@@ -86,4 +88,4 @@ const doAPIEndpointReadSelectList = function(pRequest, pResponse, fNext)
 	);
 };
 
-module.exports = doAPIEndpointReadSelectList;
+module.exports = doAPIEndpointReadSelectList;

package/source/endpoints/read/Meadow-Endpoint-Reads.js

@@ -12,7 +12,9 @@ const doAPIEndpointReads = function(pRequest, pResponse, fNext)
 		{
 			tmpRequestState.Query = this.DAL.query;
 
+			/** @type {number | boolean} */
 			var tmpCap = false;
+			/** @type {number | boolean} */
 			var tmpBegin = false;
 			if (typeof(pRequest.params.Begin) === 'string' ||
 				typeof(pRequest.params.Begin) === 'number')
@@ -72,4 +74,4 @@ const doAPIEndpointReads = function(pRequest, pResponse, fNext)
 	});
 };
 
-module.exports = doAPIEndpointReads;
+module.exports = doAPIEndpointReads;

package/source/endpoints/read/Meadow-Endpoint-ReadsBy.js

@@ -13,7 +13,9 @@ const doAPIEndpointReadsBy = function(pRequest, pResponse, fNext)
 			{
 				tmpRequestState.Query = this.DAL.query;
 
+				/** @type {number | boolean} */
 				var tmpCap = false;
+				/** @type {number | boolean} */
 				var tmpBegin = false;
 				if (typeof(pRequest.params.Begin) === 'string' ||
 					typeof(pRequest.params.Begin) === 'number')
@@ -97,4 +99,4 @@ const doAPIEndpointReadsBy = function(pRequest, pResponse, fNext)
 	);
 };
 
-module.exports = doAPIEndpointReadsBy;
+module.exports = doAPIEndpointReadsBy;

package/source/endpoints/update/Meadow-Operation-Update.js

@@ -100,7 +100,15 @@ const doUpdate = function(pRecordToModify, pRequest, pRequestState, pResponse, f
 	{
 		if (pError)
 		{
-			tmpRequestState.Record.Error = pError;
+			// Ensure we have a record object to attach the error to
+			if (tmpRequestState.Record)
+			{
+				tmpRequestState.Record.Error = pError;
+			}
+			else
+			{
+				tmpRequestState.Record = { Error: pError };
+			}
 
 			tmpRequestState.ParentRequestState.RecordUpdateError = true;
 			tmpRequestState.ParentRequestState.RecordUpdateErrorObject = pError;