@barchart/portfolio-api-common 1.0.264 → 1.0.268

package/lib/processing/PositionContainer.js

@@ -146,6 +146,8 @@ module.exports = (() => {
 			}, { });
 
 			Object.keys(this._portfolios).forEach(key => updateEmptyPortfolioGroups.call(this, this._portfolios[key]));
+
+			recalculatePercentages.call(this);
 		}
 
 		/**
@@ -324,6 +326,8 @@ module.exports = (() => {
 					this._positionSymbolAddedEvent.fire(addedBarchartSymbol);
 				}
 			});
+
+			recalculatePercentages.call(this);
 		}
 
 		/**
@@ -337,6 +341,8 @@ module.exports = (() => {
 			assert.argumentIsRequired(position.position, 'position.position', String);
 
 			removePositionItem.call(this, this._items.find((item) => item.position.position === position.position));
+
+			recalculatePercentages.call(this);
 		}
 
 		/**
@@ -374,18 +380,47 @@ module.exports = (() => {
 		 * level that contains the position(s) for the symbol.
 		 *
 		 * @public
-		 * @param {String} symbol
 		 * @param {Object} quote
 		 */
-		setPositionQuote(symbol, quote) {
-			assert.argumentIsRequired(symbol, 'symbol', String);
+		setPositionQuote(quote) {
 			assert.argumentIsRequired(quote, 'quote', Object);
 
-			if (this._symbols.hasOwnProperty(symbol)) {
-				this._symbols[symbol].forEach(item => item.setQuote(quote));
+			const symbol = quote.symbol;
+
+			if (symbol) {
+				if (this._symbols.hasOwnProperty(symbol)) {
+					this._symbols[symbol].forEach(item => item.setQuote(quote));
+				}
+
+				recalculatePercentages.call(this);
 			}
 		}
 
+		/**
+		 * Performs a batch update of quotes; causing updates to any grouping
+		 * level that contains the position(s) for the symbol(s).
+		 *
+		 * @public
+		 * @param {Object} quote
+		 */
+		setPositionQuotes(quotes) {
+			assert.argumentIsArray(quotes, 'quotes');
+
+			this.startTransaction(() => {
+				quotes.forEach((quote) => {
+					const symbol = quote.symbol;
+
+					if (symbol) {
+						if (this._symbols.hasOwnProperty(symbol)) {
+							this._symbols[symbol].forEach(item => item.setQuote(quote));
+						}
+					}
+				});
+
+				recalculatePercentages.call(this);
+			});
+		}
+
 		/**
 		 * Returns all forex symbols that are required to do currency translations.
 		 *
@@ -411,23 +446,27 @@ module.exports = (() => {
 		 * any grouping level that contains that requires translation.
 		 *
 		 * @public
-		 * @param {String} symbol
 		 * @param {Object} quote
 		 */
-		setForexQuote(symbol, quote) {
-			assert.argumentIsRequired(symbol, 'symbol', String);
+		setForexQuote(quote) {
 			assert.argumentIsRequired(quote, 'quote', Object);
 
-			const rate = Rate.fromPair(quote.lastPrice, symbol);
-			const index = this._forexQuotes.findIndex(existing => existing.formatPair() === rate.formatPair());
+			const symbol = quote.symbol;
 
-			if (index < 0) {
-				this._forexQuotes.push(rate);
-			} else {
-				this._forexQuotes[index] = rate;
-			}
+			if (symbol) {
+				const rate = Rate.fromPair(quote.lastPrice, symbol);
+				const index = this._forexQuotes.findIndex(existing => existing.formatPair() === rate.formatPair());
+
+				if (index < 0) {
+					this._forexQuotes.push(rate);
+				} else {
+					this._forexQuotes[ index ] = rate;
+				}
+
+				Object.keys(this._trees).forEach(key => this._trees[ key ].walk(group => group.setForexRates(this._forexQuotes), true, false));
 
-			Object.keys(this._trees).forEach(key => this._trees[key].walk(group => group.setForexRate(rate), true, false));
+				recalculatePercentages.call(this);
+			}
 		}
 
 		/**
@@ -739,12 +778,6 @@ module.exports = (() => {
 				}
 			}
 		}));
-
-		addGroupBinding.call(this, group, group.registerMarketPercentChangeHandler(() => {
-			if (!groupTree.getIsRoot()) {
-				groupTree.getParent().walk((childGroup) => childGroup.refreshMarketPercent());
-			}
-		}));
 	}
 
 	function createGroups(parentTree, items, treeDefinition, levelDefinitions, overrideRequiredGroups) {
@@ -752,6 +785,8 @@ module.exports = (() => {
 			return;
 		}
 
+		const rates = this._forexQuotes;
+
 		const levelDefinition = levelDefinitions[0];
 
 		const populatedObjects = array.groupBy(items, levelDefinition.keySelector);
@@ -759,7 +794,7 @@ module.exports = (() => {
 			const items = populatedObjects[key];
 			const first = items[0];
 
-			list.push(new PositionGroup(this, levelDefinition, items, levelDefinition.currencySelector(first), key, levelDefinition.descriptionSelector(first), levelDefinition.aggregateCash));
+			list.push(new PositionGroup(levelDefinition, items, rates, levelDefinition.currencySelector(first), key, levelDefinition.descriptionSelector(first), levelDefinition.aggregateCash));
 
 			return list;
 		}, [ ]);
@@ -772,7 +807,7 @@ module.exports = (() => {
 			});
 
 		const empty = missingGroups.map((group) => {
-			return new PositionGroup(this, levelDefinition, [ ], group.currency, group.key, group.description);
+			return new PositionGroup(levelDefinition, [ ], rates, group.currency, group.key, group.description);
 		});
 
 		const compositeGroups = populatedGroups.concat(empty);
@@ -812,6 +847,9 @@ module.exports = (() => {
 
 			this._nodes[group.id] = childTree;
 
+			group.setParentGroup(this.getParentGroup(group));
+			group.setPortfolioGroup(this.getParentGroupForPortfolio(group));
+
 			initializeGroupObservers.call(this, childTree, treeDefinition);
 
 			createGroups.call(this, childTree, group.items, treeDefinition, array.dropLeft(levelDefinitions));
@@ -952,5 +990,11 @@ module.exports = (() => {
 		groupNodeToSever.walk(group => delete this._nodes[group.id], false, true);
 	}
 
+	function recalculatePercentages() {
+		Object.keys(this._trees).forEach((key) => {
+			this._trees[key].walk(group => group.refreshMarketPercent(), false, false);
+		});
+	}
+
 	return PositionContainer;
 })();

package/lib/processing/PositionGroup.js

@@ -33,13 +33,17 @@ module.exports = (() => {
 	 * @param {Boolean=} aggregateCash
 	 */
 	class PositionGroup {
-		constructor(container, definition, items, currency, key, description, aggregateCash) {
+		constructor(definition, items, rates, currency, key, description, aggregateCash) {
 			this._id = counter++;
 
 			this._definition = definition;
-			this._container = container;
 
 			this._items = items;
+			this._rates = rates;
+
+			this._parentGroup = null;
+			this._portfolioGroup = null;
+
 			this._currency = currency || Currency.CAD;
 			this._bypassCurrencyTranslation = false;
 
@@ -53,7 +57,6 @@ module.exports = (() => {
 			this._suspended = false;
 			this._showClosedPositions = false;
 
-			this._marketPercentChangeEvent = new Event(this);
 			this._groupExcludedChangeEvent = new Event(this);
 			this._showClosedPositionsChangeEvent = new Event(this);
 
@@ -266,6 +269,40 @@ module.exports = (() => {
 			return this._excluded;
 		}
 
+		/**
+		 * Sets the immediate parent group (allowing for calculation of relative
+		 * percentages).
+		 *
+		 * @public
+		 * @param {PortfolioGroup} group
+		 */
+		setParentGroup(group) {
+			assert.argumentIsOptional(group, 'group', PositionGroup, 'PositionGroup');
+
+			if (this._parentGroup !== null) {
+				throw new Error('The parent group has already been set.');
+			}
+
+			this._parentGroup = group;
+		}
+
+		/**
+		 * Sets the nearest parent group for a portfolio (allowing for calculation
+		 * of relative percentages).
+		 *
+		 * @public
+		 * @param {PortfolioGroup} group
+		 */
+		setPortfolioGroup(group) {
+			assert.argumentIsOptional(group, 'group', PositionGroup, 'PositionGroup');
+
+			if (this._portfolioGroup !== null) {
+				throw new Error('The portfolio group has already been set.');
+			}
+
+			this._portfolioGroup = group;
+		}
+
 		/**
 		 * Adds a new {@link PositionItem} to the group.
 		 *
@@ -311,9 +348,11 @@ module.exports = (() => {
 		 * Causes aggregated data to be recalculated using a new exchange rate.
 		 *
 		 * @public
-		 * @param {Rate} rate
+		 * @param {Array.<Rate>} rate
 		 */
-		setForexRate(rate) {
+		setForexRates(rates) {
+			this._rates = rates;
+
 			if (!this._bypassCurrencyTranslation) {
 				this.refresh();
 			}
@@ -401,10 +440,8 @@ module.exports = (() => {
 				return;
 			}
 
-			const rates = this._container.getForexQuotes();
-
-			calculateStaticData(this, rates);
-			calculatePriceData(this, rates, null, true);
+			calculateStaticData(this, this._rates);
+			calculatePriceData(this, this._rates, null, true);
 		}
 
 		/**
@@ -414,7 +451,7 @@ module.exports = (() => {
 		 * @public
 		 */
 		refreshMarketPercent() {
-			calculateMarketPercent(this, this._container.getForexQuotes(), true);
+			calculateMarketPercent(this, this._rates, this._parentGroup, this._portfolioGroup);
 		}
 
 		/**
@@ -427,17 +464,6 @@ module.exports = (() => {
 			return this._items.length === 0;
 		}
 
-		/**
-		 * Adds an observer for change in the market percentage of the group.
-		 *
-		 * @public
-		 * @param {Function} handler
-		 * @return {Disposable}
-		 */
-		registerMarketPercentChangeHandler(handler) {
-			return this._marketPercentChangeEvent.register(handler);
-		}
-
 		/**
 		 * Adds an observer for changes to the exclusion of the group
 		 * from higher level aggregations.
@@ -492,10 +518,10 @@ module.exports = (() => {
 				this._dataFormat.currentPrice = null;
 			}
 
-			calculatePriceData(this, this._container.getForexQuotes(), sender, false);
+			calculatePriceData(this, this._rates, sender, false);
 		});
 
-		let fundamentalBinding = item.registerFundamentalDataChangeHandler((data, sender) => {
+		let fundamentalBinding = item.registerFundamentalDataChangeHandler((data) => {
 			if (this._single) {
 				this._dataFormat.fundamental = data;
 			} else {
@@ -546,13 +572,13 @@ module.exports = (() => {
 		let newsBinding = Disposable.getEmpty();
 
 		if (this._single) {
-			newsBinding = item.registerNewsExistsChangeHandler((exists, sender) => {
+			newsBinding = item.registerNewsExistsChangeHandler((exists) => {
 				this._dataActual.newsExists = exists;
 				this._dataFormat.newsExists = exists;
 			});
 		}
 
-		this._disposeStack.push(item.registerPortfolioChangeHandler((portfolio, sender) => {
+		this._disposeStack.push(item.registerPortfolioChangeHandler((portfolio) => {
 			const descriptionSelector = this._definition.descriptionSelector;
 
 			this._description = descriptionSelector(this._items[0]);
@@ -801,12 +827,11 @@ module.exports = (() => {
 
 		format.total = formatCurrency(actual.total, currency);
 		format.totalNegative = actual.total.getIsNegative();
-		
-		calculateMarketPercent(group, rates, false);
+
 		calculateUnrealizedPercent(group);
 	}
 
-	function calculateMarketPercent(group, rates, silent) {
+	function calculateMarketPercent(group, rates, parentGroup, portfolioGroup) {
 		if (group.suspended) {
 			return;
 		}
@@ -815,12 +840,10 @@ module.exports = (() => {
 		const format = group._dataFormat;
 		const excluded = group._excluded;
 
-		const portfolioParent = group._container.getParentGroupForPortfolio(group);
-
 		const calculatePercent = (parent) => {
 			let marketPercent;
 
-			if (parent !== null && !excluded) {
+			if (parent && !excluded) {
 				const parentData = parent._dataActual;
 
 				if (parentData.marketAbsolute !== null && !parentData.marketAbsolute.getIsZero()) {
@@ -843,14 +866,15 @@ module.exports = (() => {
 			return marketPercent;
 		};
 
-		actual.marketPercent = calculatePercent(group._container.getParentGroup(group));
+		actual.marketPercent = calculatePercent(parentGroup);
 		format.marketPercent = formatPercent(actual.marketPercent, 2);
 
-		actual.marketPercentPortfolio = calculatePercent(group._container.getParentGroupForPortfolio(group));
-		format.marketPercentPortfolio = formatPercent(actual.marketPercentPortfolio, 2);
-
-		if (!silent) {
-			group._marketPercentChangeEvent.fire(group);
+		if (parentGroup === portfolioGroup) {
+			actual.marketPercentPortfolio = actual.marketPercent;
+			format.marketPercentPortfolio = format.marketPercent;
+		} else {
+			actual.marketPercentPortfolio = calculatePercent(portfolioGroup);
+			format.marketPercentPortfolio = formatPercent(actual.marketPercentPortfolio, 2);
 		}
 	}
 

package/package.json

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

package/test/SpecRunner.js

@@ -1302,6 +1302,8 @@ module.exports = (() => {
 			}, { });
 
 			Object.keys(this._portfolios).forEach(key => updateEmptyPortfolioGroups.call(this, this._portfolios[key]));
+
+			recalculatePercentages.call(this);
 		}
 
 		/**
@@ -1480,6 +1482,8 @@ module.exports = (() => {
 					this._positionSymbolAddedEvent.fire(addedBarchartSymbol);
 				}
 			});
+
+			recalculatePercentages.call(this);
 		}
 
 		/**
@@ -1493,6 +1497,8 @@ module.exports = (() => {
 			assert.argumentIsRequired(position.position, 'position.position', String);
 
 			removePositionItem.call(this, this._items.find((item) => item.position.position === position.position));
+
+			recalculatePercentages.call(this);
 		}
 
 		/**
@@ -1530,18 +1536,47 @@ module.exports = (() => {
 		 * level that contains the position(s) for the symbol.
 		 *
 		 * @public
-		 * @param {String} symbol
 		 * @param {Object} quote
 		 */
-		setPositionQuote(symbol, quote) {
-			assert.argumentIsRequired(symbol, 'symbol', String);
+		setPositionQuote(quote) {
 			assert.argumentIsRequired(quote, 'quote', Object);
 
-			if (this._symbols.hasOwnProperty(symbol)) {
-				this._symbols[symbol].forEach(item => item.setQuote(quote));
+			const symbol = quote.symbol;
+
+			if (symbol) {
+				if (this._symbols.hasOwnProperty(symbol)) {
+					this._symbols[symbol].forEach(item => item.setQuote(quote));
+				}
+
+				recalculatePercentages.call(this);
 			}
 		}
 
+		/**
+		 * Performs a batch update of quotes; causing updates to any grouping
+		 * level that contains the position(s) for the symbol(s).
+		 *
+		 * @public
+		 * @param {Object} quote
+		 */
+		setPositionQuotes(quotes) {
+			assert.argumentIsArray(quotes, 'quotes');
+
+			this.startTransaction(() => {
+				quotes.forEach((quote) => {
+					const symbol = quote.symbol;
+
+					if (symbol) {
+						if (this._symbols.hasOwnProperty(symbol)) {
+							this._symbols[symbol].forEach(item => item.setQuote(quote));
+						}
+					}
+				});
+
+				recalculatePercentages.call(this);
+			});
+		}
+
 		/**
 		 * Returns all forex symbols that are required to do currency translations.
 		 *
@@ -1567,23 +1602,27 @@ module.exports = (() => {
 		 * any grouping level that contains that requires translation.
 		 *
 		 * @public
-		 * @param {String} symbol
 		 * @param {Object} quote
 		 */
-		setForexQuote(symbol, quote) {
-			assert.argumentIsRequired(symbol, 'symbol', String);
+		setForexQuote(quote) {
 			assert.argumentIsRequired(quote, 'quote', Object);
 
-			const rate = Rate.fromPair(quote.lastPrice, symbol);
-			const index = this._forexQuotes.findIndex(existing => existing.formatPair() === rate.formatPair());
+			const symbol = quote.symbol;
 
-			if (index < 0) {
-				this._forexQuotes.push(rate);
-			} else {
-				this._forexQuotes[index] = rate;
-			}
+			if (symbol) {
+				const rate = Rate.fromPair(quote.lastPrice, symbol);
+				const index = this._forexQuotes.findIndex(existing => existing.formatPair() === rate.formatPair());
 
-			Object.keys(this._trees).forEach(key => this._trees[key].walk(group => group.setForexRate(rate), true, false));
+				if (index < 0) {
+					this._forexQuotes.push(rate);
+				} else {
+					this._forexQuotes[ index ] = rate;
+				}
+
+				Object.keys(this._trees).forEach(key => this._trees[ key ].walk(group => group.setForexRates(this._forexQuotes), true, false));
+
+				recalculatePercentages.call(this);
+			}
 		}
 
 		/**
@@ -1895,12 +1934,6 @@ module.exports = (() => {
 				}
 			}
 		}));
-
-		addGroupBinding.call(this, group, group.registerMarketPercentChangeHandler(() => {
-			if (!groupTree.getIsRoot()) {
-				groupTree.getParent().walk((childGroup) => childGroup.refreshMarketPercent());
-			}
-		}));
 	}
 
 	function createGroups(parentTree, items, treeDefinition, levelDefinitions, overrideRequiredGroups) {
@@ -1908,6 +1941,8 @@ module.exports = (() => {
 			return;
 		}
 
+		const rates = this._forexQuotes;
+
 		const levelDefinition = levelDefinitions[0];
 
 		const populatedObjects = array.groupBy(items, levelDefinition.keySelector);
@@ -1915,7 +1950,7 @@ module.exports = (() => {
 			const items = populatedObjects[key];
 			const first = items[0];
 
-			list.push(new PositionGroup(this, levelDefinition, items, levelDefinition.currencySelector(first), key, levelDefinition.descriptionSelector(first), levelDefinition.aggregateCash));
+			list.push(new PositionGroup(levelDefinition, items, rates, levelDefinition.currencySelector(first), key, levelDefinition.descriptionSelector(first), levelDefinition.aggregateCash));
 
 			return list;
 		}, [ ]);
@@ -1928,7 +1963,7 @@ module.exports = (() => {
 			});
 
 		const empty = missingGroups.map((group) => {
-			return new PositionGroup(this, levelDefinition, [ ], group.currency, group.key, group.description);
+			return new PositionGroup(levelDefinition, [ ], rates, group.currency, group.key, group.description);
 		});
 
 		const compositeGroups = populatedGroups.concat(empty);
@@ -1968,6 +2003,9 @@ module.exports = (() => {
 
 			this._nodes[group.id] = childTree;
 
+			group.setParentGroup(this.getParentGroup(group));
+			group.setPortfolioGroup(this.getParentGroupForPortfolio(group));
+
 			initializeGroupObservers.call(this, childTree, treeDefinition);
 
 			createGroups.call(this, childTree, group.items, treeDefinition, array.dropLeft(levelDefinitions));
@@ -2108,6 +2146,12 @@ module.exports = (() => {
 		groupNodeToSever.walk(group => delete this._nodes[group.id], false, true);
 	}
 
+	function recalculatePercentages() {
+		Object.keys(this._trees).forEach((key) => {
+			this._trees[key].walk(group => group.refreshMarketPercent(), false, false);
+		});
+	}
+
 	return PositionContainer;
 })();
 
@@ -2147,13 +2191,17 @@ module.exports = (() => {
 	 * @param {Boolean=} aggregateCash
 	 */
 	class PositionGroup {
-		constructor(container, definition, items, currency, key, description, aggregateCash) {
+		constructor(definition, items, rates, currency, key, description, aggregateCash) {
 			this._id = counter++;
 
 			this._definition = definition;
-			this._container = container;
 
 			this._items = items;
+			this._rates = rates;
+
+			this._parentGroup = null;
+			this._portfolioGroup = null;
+
 			this._currency = currency || Currency.CAD;
 			this._bypassCurrencyTranslation = false;
 
@@ -2167,7 +2215,6 @@ module.exports = (() => {
 			this._suspended = false;
 			this._showClosedPositions = false;
 
-			this._marketPercentChangeEvent = new Event(this);
 			this._groupExcludedChangeEvent = new Event(this);
 			this._showClosedPositionsChangeEvent = new Event(this);
 
@@ -2380,6 +2427,40 @@ module.exports = (() => {
 			return this._excluded;
 		}
 
+		/**
+		 * Sets the immediate parent group (allowing for calculation of relative
+		 * percentages).
+		 *
+		 * @public
+		 * @param {PortfolioGroup} group
+		 */
+		setParentGroup(group) {
+			assert.argumentIsOptional(group, 'group', PositionGroup, 'PositionGroup');
+
+			if (this._parentGroup !== null) {
+				throw new Error('The parent group has already been set.');
+			}
+
+			this._parentGroup = group;
+		}
+
+		/**
+		 * Sets the nearest parent group for a portfolio (allowing for calculation
+		 * of relative percentages).
+		 *
+		 * @public
+		 * @param {PortfolioGroup} group
+		 */
+		setPortfolioGroup(group) {
+			assert.argumentIsOptional(group, 'group', PositionGroup, 'PositionGroup');
+
+			if (this._portfolioGroup !== null) {
+				throw new Error('The portfolio group has already been set.');
+			}
+
+			this._portfolioGroup = group;
+		}
+
 		/**
 		 * Adds a new {@link PositionItem} to the group.
 		 *
@@ -2425,9 +2506,11 @@ module.exports = (() => {
 		 * Causes aggregated data to be recalculated using a new exchange rate.
 		 *
 		 * @public
-		 * @param {Rate} rate
+		 * @param {Array.<Rate>} rate
 		 */
-		setForexRate(rate) {
+		setForexRates(rates) {
+			this._rates = rates;
+
 			if (!this._bypassCurrencyTranslation) {
 				this.refresh();
 			}
@@ -2515,10 +2598,8 @@ module.exports = (() => {
 				return;
 			}
 
-			const rates = this._container.getForexQuotes();
-
-			calculateStaticData(this, rates);
-			calculatePriceData(this, rates, null, true);
+			calculateStaticData(this, this._rates);
+			calculatePriceData(this, this._rates, null, true);
 		}
 
 		/**
@@ -2528,7 +2609,7 @@ module.exports = (() => {
 		 * @public
 		 */
 		refreshMarketPercent() {
-			calculateMarketPercent(this, this._container.getForexQuotes(), true);
+			calculateMarketPercent(this, this._rates, this._parentGroup, this._portfolioGroup);
 		}
 
 		/**
@@ -2541,17 +2622,6 @@ module.exports = (() => {
 			return this._items.length === 0;
 		}
 
-		/**
-		 * Adds an observer for change in the market percentage of the group.
-		 *
-		 * @public
-		 * @param {Function} handler
-		 * @return {Disposable}
-		 */
-		registerMarketPercentChangeHandler(handler) {
-			return this._marketPercentChangeEvent.register(handler);
-		}
-
 		/**
 		 * Adds an observer for changes to the exclusion of the group
 		 * from higher level aggregations.
@@ -2606,10 +2676,10 @@ module.exports = (() => {
 				this._dataFormat.currentPrice = null;
 			}
 
-			calculatePriceData(this, this._container.getForexQuotes(), sender, false);
+			calculatePriceData(this, this._rates, sender, false);
 		});
 
-		let fundamentalBinding = item.registerFundamentalDataChangeHandler((data, sender) => {
+		let fundamentalBinding = item.registerFundamentalDataChangeHandler((data) => {
 			if (this._single) {
 				this._dataFormat.fundamental = data;
 			} else {
@@ -2660,13 +2730,13 @@ module.exports = (() => {
 		let newsBinding = Disposable.getEmpty();
 
 		if (this._single) {
-			newsBinding = item.registerNewsExistsChangeHandler((exists, sender) => {
+			newsBinding = item.registerNewsExistsChangeHandler((exists) => {
 				this._dataActual.newsExists = exists;
 				this._dataFormat.newsExists = exists;
 			});
 		}
 
-		this._disposeStack.push(item.registerPortfolioChangeHandler((portfolio, sender) => {
+		this._disposeStack.push(item.registerPortfolioChangeHandler((portfolio) => {
 			const descriptionSelector = this._definition.descriptionSelector;
 
 			this._description = descriptionSelector(this._items[0]);
@@ -2915,12 +2985,11 @@ module.exports = (() => {
 
 		format.total = formatCurrency(actual.total, currency);
 		format.totalNegative = actual.total.getIsNegative();
-		
-		calculateMarketPercent(group, rates, false);
+
 		calculateUnrealizedPercent(group);
 	}
 
-	function calculateMarketPercent(group, rates, silent) {
+	function calculateMarketPercent(group, rates, parentGroup, portfolioGroup) {
 		if (group.suspended) {
 			return;
 		}
@@ -2929,12 +2998,10 @@ module.exports = (() => {
 		const format = group._dataFormat;
 		const excluded = group._excluded;
 
-		const portfolioParent = group._container.getParentGroupForPortfolio(group);
-
 		const calculatePercent = (parent) => {
 			let marketPercent;
 
-			if (parent !== null && !excluded) {
+			if (parent && !excluded) {
 				const parentData = parent._dataActual;
 
 				if (parentData.marketAbsolute !== null && !parentData.marketAbsolute.getIsZero()) {
@@ -2957,14 +3024,15 @@ module.exports = (() => {
 			return marketPercent;
 		};
 
-		actual.marketPercent = calculatePercent(group._container.getParentGroup(group));
+		actual.marketPercent = calculatePercent(parentGroup);
 		format.marketPercent = formatPercent(actual.marketPercent, 2);
 
-		actual.marketPercentPortfolio = calculatePercent(group._container.getParentGroupForPortfolio(group));
-		format.marketPercentPortfolio = formatPercent(actual.marketPercentPortfolio, 2);
-
-		if (!silent) {
-			group._marketPercentChangeEvent.fire(group);
+		if (parentGroup === portfolioGroup) {
+			actual.marketPercentPortfolio = actual.marketPercent;
+			format.marketPercentPortfolio = format.marketPercent;
+		} else {
+			actual.marketPercentPortfolio = calculatePercent(portfolioGroup);
+			format.marketPercentPortfolio = formatPercent(actual.marketPercentPortfolio, 2);
 		}
 	}