@barchart/portfolio-api-common 1.2.42 → 1.2.46

package/lib/api/failures/PortfolioFailureType.js

@@ -131,6 +131,18 @@ module.exports = (() => {
 			return transactionCreateFailedTypeReserved;
 		}
 
+		/**
+		 * The transaction failed because a dividend would be reinvested on
+		 * a day for which no historical price is available.
+		 *
+		 * @public
+		 * @static
+		 * @returns {FailureType}
+		 */
+		static get TRANSACTION_CREATE_FAILED_REINVEST_PRICE_UNAVAILABLE() {
+			return transactionCreateFailedReinvestPriceUnavailable;
+		}
+
 		/**
 		 * Deleting any transaction except for the most recent requires
 		 * re-writing transaction history.
@@ -204,8 +216,9 @@ module.exports = (() => {
 	const transactionCreateFailedTypeInvalidForInstrument = new FailureType('TRANSACTION_CREATE_FAILED_TYPE_INVALID_FOR_INSTRUMENT', 'Unable to process transaction, {L|transactionType.description} transactions cannot be used with {L|instrumentType.description} positions.');
 	const transactionCreateFailedTypeInvalidForDirection = new FailureType('TRANSACTION_CREATE_FAILED_TYPE_INVALID_FOR_DIRECTION', 'Unable to process transaction, a {L|positionDirection.description} position would be created (i.e. you would have {L|positionDirection.sign} shares/units). {u|instrumentType.description} positions cannot have {L|positionDirection.description} positions.');
 	const transactionCreateFailedInvalidDirectionSwitch = new FailureType('TRANSACTION_CREATE_FAILED_INVALID_DIRECTION_SWITCH', 'Unable to process transaction, the transaction would switch the position from {L|currentDirection.description} to {L|proposedDirection.description} (i.e. {L|currentDirection.sign} to {L|proposedDirection.sign} shares/units). This is not allowed. Please close the current position (i.e. zero it out) and then enter a second transaction.');
-	const transactionCreateFailedInvalidInitialType = new FailureType('TRANSACTION_CREATE_FAILED_INITIAL_TYPE', 'Unable to process operation because the first transaction would to be a {U|transactionType.description}, which is not allowed -- since {U|transactionType.description} transactions cannot open a position.');
+	const transactionCreateFailedInvalidInitialType = new FailureType('TRANSACTION_CREATE_FAILED_INVALID_INITIAL_TYPE', 'Unable to process operation because the first transaction would to be a {U|transactionType.description}, which is not allowed -- since {U|transactionType.description} transactions cannot open a position.');
 	const transactionCreateFailedTypeReserved = new FailureType('TRANSACTION_CREATE_FAILED_TYPE_RESERVED', 'Unable to create {U|type.description} transaction, this type of transaction is managed by the system.');
+	const transactionCreateFailedReinvestPriceUnavailable = new FailureType('TRANSACTION_CREATE_FAILED_REINVEST_PRICE_UNAVAILABLE', 'Unable to create transaction, a dividend was paid on {L|day}; however no historical price is available for this day. To successfully create this transaction, please turn off dividend reinvestment for this position.');
 
 	const transactionDeleteFailedOutOfSequence = new FailureType('TRANSACTION_DELETE_FAILED_OUT_OF_SEQUENCE', 'Deleting any transaction, except for the most recent, will cause transaction history to be re-written. Please confirm your intent to re-write transaction history (which could take some time and alter the historical results for this position).');
 	const transactionDeleteFailedNoTransaction = new FailureType('TRANSACTION_DELETE_FAILED_NO_TRANSACTION', 'Unable to delete transaction. The referenced transaction does not exist.');

package/lib/data/TransactionValidator.js

@@ -20,7 +20,7 @@ module.exports = (() => {
 		}
 
 		/**
-		 * Given a set of transaction, ensures that all sequence numbers and dates
+		 * Given an array of transactions, ensures that all sequence numbers and dates
 		 * are properly ordered.
 		 *
 		 * @public
@@ -33,7 +33,7 @@ module.exports = (() => {
 		}
 
 		/**
-		 * Given a set of transaction, when transaction references are present, ensures
+		 * Given an array of transactions, when transaction references are present, ensures
 		 * that no transactions within the set reference the same transaction.
 		 *
 		 * @public
@@ -71,7 +71,7 @@ module.exports = (() => {
 		}
 
 		/**
-		 * Given a set of transaction, returns the index of the first transaction that with an invalid
+		 * Given an array of transactions, returns the index of the first transaction that with an invalid
 		 * sequence number or date.
 		 *
 		 * @public
@@ -92,7 +92,7 @@ module.exports = (() => {
 		 * @public
 		 * @param {InstrumentType} instrumentType
 		 * @param {Boolean=} userInitiated
-		 * @pararm {PositionDirection=} currentDirection
+		 * @param {PositionDirection=} currentDirection
 		 * @returns {Array.<TransactionType>}
 		 */
 		static getTransactionTypesFor(instrumentType, userInitiated, currentDirection) {
@@ -156,7 +156,9 @@ module.exports = (() => {
 		 * Determines if a transaction type is valid as the first transaction of
 		 * a position.
 		 *
+		 * @public
 		 * @param {TransactionType} transactionType
+		 * @returns {Boolean}
 		 */
 		static validateInitialTransactionType(transactionType) {
 			return transactionType.initial;

package/lib/serialization/TransactionSchema.js

@@ -1,5 +1,4 @@
-const assert = require('@barchart/common-js/lang/assert'),
-	is = require('@barchart/common-js/lang/is'),
+const is = require('@barchart/common-js/lang/is'),
 	Currency = require('@barchart/common-js/lang/Currency'),
 	DataType = require('@barchart/common-js/serialization/json/DataType'),
 	Enum = require('@barchart/common-js/lang/Enum'),
@@ -211,6 +210,8 @@ module.exports = (() => {
 		.withField('sequence', DataType.NUMBER, true)
 		.withField('type', DataType.forEnum(TransactionType, 'TransactionType'))
 		.withField('instrument.name', DataType.STRING, true)
+		.withField('instrument.exchange', DataType.STRING, true)
+		.withField('instrument.code', DataType.NUMBER, true)
 		.withField('instrument.type', DataType.forEnum(InstrumentType, 'InstrumentType'), true)
 		.withField('instrument.currency', DataType.forEnum(Currency, 'Currency'), true)
 		.withField('instrument.symbol.barchart', DataType.STRING, true)
@@ -257,6 +258,8 @@ module.exports = (() => {
 		.withField('sequence', DataType.NUMBER, true)
 		.withField('type', DataType.forEnum(TransactionType, 'TransactionType'))
 		.withField('instrument.name', DataType.STRING, true)
+		.withField('instrument.exchange', DataType.STRING, true)
+		.withField('instrument.code', DataType.NUMBER, true)
 		.withField('instrument.type', DataType.forEnum(InstrumentType, 'InstrumentType'), true)
 		.withField('instrument.currency', DataType.forEnum(Currency, 'Currency'), true)
 		.withField('instrument.symbol.barchart', DataType.STRING, true)

package/package.json

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