@barchart/portfolio-api-common 1.0.146 → 1.0.150

package/lib/processing/PositionContainer.js

@@ -229,11 +229,11 @@ module.exports = (() => {
 
 					const initializeGroupObservers = (group, groupTree) => {
 						addGroupBinding(group, group.registerGroupExcludedChangeHandler((excluded, sender) => {
-							groupTree.climb((parentGroup, parentTree) => {
-								if (parentGroup !== null) {
+							groupTree.climb((parentGroup) => {
+								if (parentGroup) {
 									let excludedItems = [];
 
-									currentTree.walk((childGroup, childTree) => {
+									currentTree.walk((childGroup) => {
 										if (childGroup.excluded) {
 											excludedItems = excludedItems.concat(childGroup.items);
 										}
@@ -242,6 +242,32 @@ module.exports = (() => {
 									parentGroup.setExcludedItems(array.unique(excludedItems));
 								}
 							}, false);
+
+							if (treeDefinition.exclusionDependencies.length > 0) {
+								const dependantTrees = treeDefinition.exclusionDependencies.reduce((trees, name) => {
+									if (this._trees.hasOwnProperty(name)) {
+										trees.push(this._trees[name]);
+									}
+
+									return trees;
+								}, [ ]);
+
+								if (dependantTrees.length > 0) {
+									let excludedItems = [ ];
+
+									tree.walk((childGroup) => {
+										if (childGroup.excluded) {
+											excludedItems = excludedItems.concat(childGroup.items);
+										}
+									}, false, false);
+
+									dependantTrees.forEach((dependantTrees) => {
+										dependantTrees.walk((childGroup) => {
+											childGroup.setExcludedItems(excludedItems);
+										}, false, false);
+									});
+								}
+							}
 						}));
 					};
 

package/lib/processing/definitions/PositionTreeDefinition.js

@@ -13,17 +13,23 @@ module.exports = (() => {
 	 * @param {Array.<PositionLevelDefinition>} definitions
 	 */
 	class PositionTreeDefinitions {
-		constructor(name, definitions) {
+		constructor(name, definitions, exclusionDependencies) {
 			assert.argumentIsRequired(name, 'name', String);
 			assert.argumentIsArray(definitions, 'definitions', PositionLevelDefinition, 'PositionLevelDefinition');
 
+			if (exclusionDependencies) {
+				assert.argumentIsArray(exclusionDependencies, 'exclusionDependencies', String);
+			}
+
 			this._name = name;
 			this._definitions = definitions;
+			this._exclusionDependencies = exclusionDependencies || [ ];
 		}
 
 		/**
 		 * The name of the tree.
 		 *
+		 * @public
 		 * @returns {String}
 		 */
 		get name() {
@@ -43,6 +49,17 @@ module.exports = (() => {
 			return this._definitions;
 		}
 
+		/**
+		 * Returns the names of other trees which should be impacted when a
+		 * group (from the current tree) is excluded.
+		 *
+		 * @public
+		 * @return {Array.<String>}
+		 */
+		get exclusionDependencies() {
+			return this._exclusionDependencies;
+		}
+
 		toString() {
 			return '[PositionTreeDefinitions]';
 		}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@barchart/portfolio-api-common",
-  "version": "1.0.146",
+  "version": "1.0.150",
   "description": "Common classes used by the Portfolio system",
   "author": {
     "name": "Bryan Ingle",

package/test/SpecRunner.js

@@ -945,11 +945,11 @@ module.exports = (() => {
 
 					const initializeGroupObservers = (group, groupTree) => {
 						addGroupBinding(group, group.registerGroupExcludedChangeHandler((excluded, sender) => {
-							groupTree.climb((parentGroup, parentTree) => {
-								if (parentGroup !== null) {
+							groupTree.climb((parentGroup) => {
+								if (parentGroup) {
 									let excludedItems = [];
 
-									currentTree.walk((childGroup, childTree) => {
+									currentTree.walk((childGroup) => {
 										if (childGroup.excluded) {
 											excludedItems = excludedItems.concat(childGroup.items);
 										}
@@ -958,6 +958,32 @@ module.exports = (() => {
 									parentGroup.setExcludedItems(array.unique(excludedItems));
 								}
 							}, false);
+
+							if (treeDefinition.exclusionDependencies.length > 0) {
+								const dependantTrees = treeDefinition.exclusionDependencies.reduce((trees, name) => {
+									if (this._trees.hasOwnProperty(name)) {
+										trees.push(this._trees[name]);
+									}
+
+									return trees;
+								}, [ ]);
+
+								if (dependantTrees.length > 0) {
+									let excludedItems = [ ];
+
+									tree.walk((childGroup) => {
+										if (childGroup.excluded) {
+											excludedItems = excludedItems.concat(childGroup.items);
+										}
+									}, false, false);
+
+									dependantTrees.forEach((dependantTrees) => {
+										dependantTrees.walk((childGroup) => {
+											childGroup.setExcludedItems(excludedItems);
+										}, false, false);
+									});
+								}
+							}
 						}));
 					};
 
@@ -2407,17 +2433,23 @@ module.exports = (() => {
 	 * @param {Array.<PositionLevelDefinition>} definitions
 	 */
 	class PositionTreeDefinitions {
-		constructor(name, definitions) {
+		constructor(name, definitions, exclusionDependencies) {
 			assert.argumentIsRequired(name, 'name', String);
 			assert.argumentIsArray(definitions, 'definitions', PositionLevelDefinition, 'PositionLevelDefinition');
 
+			if (exclusionDependencies) {
+				assert.argumentIsArray(exclusionDependencies, 'exclusionDependencies', String);
+			}
+
 			this._name = name;
 			this._definitions = definitions;
+			this._exclusionDependencies = exclusionDependencies || [ ];
 		}
 
 		/**
 		 * The name of the tree.
 		 *
+		 * @public
 		 * @returns {String}
 		 */
 		get name() {
@@ -2437,6 +2469,17 @@ module.exports = (() => {
 			return this._definitions;
 		}
 
+		/**
+		 * Returns the names of other trees which should be impacted when a
+		 * group (from the current tree) is excluded.
+		 *
+		 * @public
+		 * @return {Array.<String>}
+		 */
+		get exclusionDependencies() {
+			return this._exclusionDependencies;
+		}
+
 		toString() {
 			return '[PositionTreeDefinitions]';
 		}