manyfest 1.0.36 → 1.0.38

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "manyfest",
-    "version": "1.0.36",
+    "version": "1.0.38",
     "description": "JSON Object Manifest for Data Description and Parsing",
     "main": "source/Manyfest.js",
     "scripts": {

package/source/Manyfest-ObjectAddress-GetValue.js

@@ -47,9 +47,19 @@ class ManyfestObjectAddressResolverGetValue
 	getValueAtAddress (pObject, pAddress, pParentAddress, pRootObject)
 	{
 		// Make sure pObject (the object we are meant to be recursing) is an object (which could be an array or object)
-		if (typeof(pObject) != 'object') return undefined;
+		if (typeof(pObject) != 'object')
+		{
+			return undefined;
+		}
+		if (pObject === null)
+		{
+			return undefined;
+		}
 		// Make sure pAddress (the address we are resolving) is a string
-		if (typeof(pAddress) != 'string') return undefined;
+		if (typeof(pAddress) != 'string')
+		{
+			return undefined;
+		}
 		// Stash the parent address for later resolution
 		let tmpParentAddress = "";
 		if (typeof(pParentAddress) == 'string')
@@ -335,7 +345,7 @@ class ManyfestObjectAddressResolverGetValue
 				}
 				else
 				{
-					return undefined;
+					return null;
 				}
 			}
 		}

package/source/Manyfest.js

@@ -332,6 +332,12 @@ class Manyfest extends libFableServiceProviderBase
 	// Get the value of an element at an address
 	getValueAtAddress (pObject, pAddress)
 	{
+		let tmpLintedAddress = pAddress.trim();
+		if (tmpLintedAddress == '')
+		{
+			this.logError(`(${this.scope}) Error getting value at address; address is an empty string.`, pObject);
+			return undefined;
+		}
 		let tmpValue = this.objectAddressGetValue.getValueAtAddress(pObject, pAddress);
 
 		if (typeof(tmpValue) == 'undefined')

package/test/Manyfest_Object_Read_tests.js

@@ -38,6 +38,26 @@ suite
 							.to.equal('Franken Berry / Count Chocula : Tevevision Commercial 1971');
 						Expect(tmpTitle)
 							.to.equal(_SampleDataArchiveOrgFrankenberry.metadata.title);
+						let tmpNoAddress = _Manyfest.getValueAtAddress(_SampleDataArchiveOrgFrankenberry, '');
+						fTestComplete();
+					}
+				);
+				test
+				(
+					'Javascript null values should not cause errors..',
+					(fTestComplete)=>
+					{
+						let _Manyfest = new libManyfest({ Scope:'Archive.org',	Descriptors: {'metadata.creator': {Name:'Creator', Hash:'Creator'}}});
+						let tmpNullSampleData = JSON.parse(JSON.stringify(_SampleDataArchiveOrgFrankenberry));
+						tmpNullSampleData.Noel = null;
+						let tmpNullValue = _Manyfest.getValueAtAddress(tmpNullSampleData, 'Noel');
+						let tmpNullValueOneDeep = _Manyfest.getValueAtAddress(tmpNullSampleData, 'Noel.SomeChildValue');
+						let tmpNullValueTwoDeep = _Manyfest.getValueAtAddress(tmpNullSampleData, 'Noel.SomeChildValue.SecondTier');
+						let tmpNullValueTwoDeepObject = _Manyfest.getValueAtAddress(tmpNullSampleData, 'Noel.SomeChildValue[SecondTier]');
+						let tmpNullValueTwoDeepArray = _Manyfest.getValueAtAddress(tmpNullSampleData, 'Noel.SomeChildValue.SecondTier[0]');
+						Expect(tmpNullValue).to.equal(null);
+						Expect(tmpNullValueOneDeep).to.equal(undefined);
+						Expect(tmpNullValueTwoDeep).to.equal(undefined);
 						fTestComplete();
 					}
 				);