manyfest 1.0.39 → 1.0.40

package/package.json

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

package/source/Manyfest-ObjectAddress-SetValue.js

@@ -66,6 +66,24 @@ class ManyfestObjectAddressSetValue
 				// The "Name" of the Object contained too the left of the bracket
 				let tmpBoxedPropertyName = pAddress.substring(0, tmpBracketStartIndex).trim();
 
+				// The "Reference" to the property within it, either an array element or object property
+				let tmpBoxedPropertyReference = pAddress.substring(tmpBracketStartIndex+1, tmpBracketStopIndex).trim();
+				// Attempt to parse the reference as a number, which will be used as an array element
+				let tmpBoxedPropertyNumber = parseInt(tmpBoxedPropertyReference, 10);
+				let tmpIndexIsNumeric = !isNaN(tmpBoxedPropertyNumber);
+
+				if (pObject[tmpBoxedPropertyName] == null)
+				{
+					if (tmpIndexIsNumeric)
+					{
+						pObject[tmpBoxedPropertyName] = [];
+					}
+					else
+					{
+						pObject[tmpBoxedPropertyName] = {};
+					}
+				}
+
 				// If the subproperty doesn't test as a proper Object, none of the rest of this is possible.
 				// This is a rare case where Arrays testing as Objects is useful
 				if (typeof(pObject[tmpBoxedPropertyName]) !== 'object')
@@ -73,11 +91,6 @@ class ManyfestObjectAddressSetValue
 					return false;
 				}
 
-				// The "Reference" to the property within it, either an array element or object property
-				let tmpBoxedPropertyReference = pAddress.substring(tmpBracketStartIndex+1, tmpBracketStopIndex).trim();
-				// Attempt to parse the reference as a number, which will be used as an array element
-				let tmpBoxedPropertyNumber = parseInt(tmpBoxedPropertyReference, 10);
-
 				// Guard: If the referrant is a number and the boxed property is not an array, or vice versa, return undefined.
 				//        This seems confusing to me at first read, so explaination:
 				//        Is the Boxed Object an Array?  TRUE
@@ -105,6 +118,7 @@ class ManyfestObjectAddressSetValue
 					}
 
 					// Return the value in the property
+					//TODO: For cases where we have chained [][] properties, this needs to recurse somehow
 					pObject[tmpBoxedPropertyName][tmpBoxedPropertyReference] = pValue;
 					return true;
 				}
@@ -156,6 +170,20 @@ class ManyfestObjectAddressSetValue
 				let tmpBoxedPropertyReference = tmpSubObjectName.substring(tmpBracketStartIndex+1, tmpBracketStopIndex).trim();
 
 				let tmpBoxedPropertyNumber = parseInt(tmpBoxedPropertyReference, 10);
+				let tmpIndexIsNumeric = !isNaN(tmpBoxedPropertyNumber);
+
+				//if (typeof(pObject[tmpBoxedPropertyName]) !== 'object')
+				if (pObject[tmpBoxedPropertyName] == null)
+				{
+					if (tmpIndexIsNumeric)
+					{
+						pObject[tmpBoxedPropertyName] = [];
+					}
+					else
+					{
+						pObject[tmpBoxedPropertyName] = {};
+					}
+				}
 
 				// Guard: If the referrant is a number and the boxed property is not an array, or vice versa, return undefined.
 				//        This seems confusing to me at first read, so explaination:
@@ -171,7 +199,7 @@ class ManyfestObjectAddressSetValue
 				//       BUT
 				//         StudentData.Sections.Algebra.Students is an array, so the ["JaneDoe"].Grade is not possible to access
 				// TODO: Should this be an error or something?  Should we keep a log of failures like this?
-				if (Array.isArray(pObject[tmpBoxedPropertyName]) == isNaN(tmpBoxedPropertyNumber))
+				if (Array.isArray(pObject[tmpBoxedPropertyName]) != tmpIndexIsNumeric)
 				{
 					return false;
 				}

package/test/Manyfest_Object_Write_tests.js

@@ -165,6 +165,52 @@ suite
 						fTestComplete();
 					}
 				);
+				test
+				(
+					'Indexed subobject are settable',
+					(fTestComplete)=>
+					{
+						let _Manyfest = new libManyfest({});
+						let _SimpleObject = {};
+						_Manyfest.setValueAtAddress(_SimpleObject, `_Object['SubObject'].Property`, '123');
+						Expect(_Manyfest.getValueAtAddress(_SimpleObject, `_Object.SubObject.Property`))
+							.to.equal('123');
+						Expect(_SimpleObject._Object.SubObject.Property)
+							.to.equal('123');
+						fTestComplete();
+					}
+				);
+				test
+				(
+					'Indexed subobject are settable unquoted',
+					(fTestComplete)=>
+					{
+						let _Manyfest = new libManyfest({});
+						let _SimpleObject = {};
+						_Manyfest.setValueAtAddress(_SimpleObject, `_Object[SubObject].Property`, '123');
+						Expect(_Manyfest.getValueAtAddress(_SimpleObject, `_Object.SubObject.Property`))
+							.to.equal('123');
+						Expect(_SimpleObject._Object.SubObject.Property)
+							.to.equal('123');
+						fTestComplete();
+					}
+				);
+				//TODO: known broken case for multiple bracketed properties
+				test.skip
+				(
+					'Indexed subobject are settable double brackets',
+					(fTestComplete)=>
+					{
+						let _Manyfest = new libManyfest({});
+						let _SimpleObject = {};
+						_Manyfest.setValueAtAddress(_SimpleObject, `_Object[SubObject]['Property']`, '123');
+						Expect(_Manyfest.getValueAtAddress(_SimpleObject, `_Object.SubObject.Property`))
+							.to.equal('123');
+						Expect(_SimpleObject._Object.SubObject.Property)
+							.to.equal('123');
+						fTestComplete();
+					}
+				);
 			}
 		);
 	}