@barchart/portfolio-api-common 1.0.193 → 1.0.197

package/lib/api/failures/PortfolioFailureType.js

@@ -0,0 +1,33 @@
+const FailureType = require('@barchart/common-js/api/failures/FailureType');
+
+module.exports = (() => {
+	'use strict';
+
+	/**
+	 * A static container for portfolio-specific {@link FailureType} items.
+	 *
+	 * @public
+	 */
+	class PortfolioFailureType {
+		constructor() {
+		}
+
+		/**
+		 * The transaction would occur before an existing transaction.
+		 *
+		 * @static
+		 * @returns {FailureType}
+		 */
+		static get TRANSACTION_CREATE_FAILED_OUT_OF_SEQUENCE() {
+			return transactionCreateFailedOutOfSequence;
+		}
+
+		toString() {
+			return '[PortfolioFailureType]';
+		}
+	}
+
+	const transactionCreateFailedOutOfSequence = new FailureType('TRANSACTION_CREATE_FAILED_OUT_OF_SEQUENCE', 'Unable to create transaction, the transaction date is out-of-sequence (i.e. it would occur before an existing transaction). Please confirm your intent to re-write transaction history.');
+
+	return PortfolioFailureType;
+})();

package/lib/processing/PositionContainer.js

@@ -24,6 +24,11 @@ module.exports = (() => {
 
 	const DEFAULT_CURRENCY = Currency.CAD;
 
+	const REQUIRED_CURRENCIES = [
+		Currency.CAD,
+		Currency.USD
+	];
+
 	/**
 	 * A container for positions which groups the positions into one or more
 	 * trees for aggregation and display purposes. For example, positions could be
@@ -114,7 +119,9 @@ module.exports = (() => {
 				return map;
 			}, { });
 
-			this._forexSymbols = Object.keys(this._currencies).reduce((symbols, code) => {
+			const forexCurrencyCodes = array.unique(Object.keys(this._currencies).concat(REQUIRED_CURRENCIES.map(c => c.code)));
+
+			this._forexSymbols = forexCurrencyCodes.reduce((symbols, code) => {
 				if (code !== DEFAULT_CURRENCY.code) {
 					symbols.push(`^${DEFAULT_CURRENCY.code}${code}`);
 				}
@@ -290,7 +297,7 @@ module.exports = (() => {
 
 				const addedBarchartSymbol = extractSymbolForBarchart(item.position);
 
-				if (!existingBarchartSymbols.some(existingBarchartSymbol => existingBarchartSymbol === addedBarchartSymbol)) {
+				if (addedBarchartSymbol !== null && !existingBarchartSymbols.some(existingBarchartSymbol => existingBarchartSymbol === addedBarchartSymbol)) {
 					this._positionSymbolAddedEvent.fire(addedBarchartSymbol);
 				}
 			});
@@ -562,7 +569,7 @@ module.exports = (() => {
 		 * @param {Function} handler
 		 * @returns {Disposable}
 		 */
-		registerPositionSymbolAddedHandler(handler) {
+		registerPositionSymbolRemovedHandler(handler) {
 			return this._positionSymbolRemovedEvent.register(handler);
 		}
 

package/package.json

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

package/test/SpecRunner.js

@@ -740,6 +740,11 @@ module.exports = (() => {
 
 	const DEFAULT_CURRENCY = Currency.CAD;
 
+	const REQUIRED_CURRENCIES = [
+		Currency.CAD,
+		Currency.USD
+	];
+
 	/**
 	 * A container for positions which groups the positions into one or more
 	 * trees for aggregation and display purposes. For example, positions could be
@@ -830,7 +835,9 @@ module.exports = (() => {
 				return map;
 			}, { });
 
-			this._forexSymbols = Object.keys(this._currencies).reduce((symbols, code) => {
+			const forexCurrencyCodes = array.unique(Object.keys(this._currencies).concat(REQUIRED_CURRENCIES.map(c => c.code)));
+
+			this._forexSymbols = forexCurrencyCodes.reduce((symbols, code) => {
 				if (code !== DEFAULT_CURRENCY.code) {
 					symbols.push(`^${DEFAULT_CURRENCY.code}${code}`);
 				}
@@ -1006,7 +1013,7 @@ module.exports = (() => {
 
 				const addedBarchartSymbol = extractSymbolForBarchart(item.position);
 
-				if (!existingBarchartSymbols.some(existingBarchartSymbol => existingBarchartSymbol === addedBarchartSymbol)) {
+				if (addedBarchartSymbol !== null && !existingBarchartSymbols.some(existingBarchartSymbol => existingBarchartSymbol === addedBarchartSymbol)) {
 					this._positionSymbolAddedEvent.fire(addedBarchartSymbol);
 				}
 			});
@@ -1278,7 +1285,7 @@ module.exports = (() => {
 		 * @param {Function} handler
 		 * @returns {Disposable}
 		 */
-		registerPositionSymbolAddedHandler(handler) {
+		registerPositionSymbolRemovedHandler(handler) {
 			return this._positionSymbolRemovedEvent.register(handler);
 		}