@barchart/portfolio-api-common 1.2.86 → 1.2.87

package/lib/processing/PositionContainer.js

@@ -283,8 +283,17 @@ module.exports = (() => {
 			}
 
 			const existingBarchartSymbols = this.getPositionSymbols(false);
+			const existingPositionItem = this._items.find(item => item.position.position === position.position);
 
-			removePositionItem.call(this, this._items.find(item => item.position.position === position.position));
+			let currentQuote = null;
+			let previousQuote = null;
+
+			if (existingPositionItem) {
+				currentQuote = existingPositionItem.quote || null;
+				previousQuote = existingPositionItem.previousQuote || null;
+			}
+
+			removePositionItem.call(this, existingPositionItem);
 
 			summaries.forEach((summary) => {
 				addSummaryCurrent(this._summariesCurrent, summary, this._currentSummaryFrame, this._currentSummaryRange);
@@ -296,6 +305,14 @@ module.exports = (() => {
 			addBarchartSymbol(this._symbols, item);
 			addDisplaySymbol(this._symbolsDisplay, item);
 
+			if (previousQuote !== null) {
+				item.setQuote(previousQuote);
+			}
+
+			if (currentQuote !== null) {
+				item.setQuote(currentQuote);
+			}
+
 			this._items.push(item);
 
 			const createGroupOrInjectItem = (parentTree, treeDefinition, levelDefinitions) => {
@@ -915,7 +932,7 @@ module.exports = (() => {
 		}
 	}
 
-	function createPositionItem(position) {
+	function createPositionItem(position, currentQuote, previousQuote) {
 		const portfolio = this._portfolios[position.portfolio];
 
 		let returnRef;

package/lib/processing/PositionItem.js

@@ -41,6 +41,7 @@ module.exports = (() => {
 			this._data.basis = null;
 
 			this._currentQuote = null;
+			this._previousQuote = null;
 			this._currentPrice = null;
 
 			this._data.currentPrice = null;
@@ -154,7 +155,7 @@ module.exports = (() => {
 		}
 
 		/**
-		 * The current quote for the symbol of the encapsulated position.
+		 * The most recent quote for the symbol of the encapsulated position.
 		 *
 		 * @public
 		 * @returns {null|Object}
@@ -163,6 +164,16 @@ module.exports = (() => {
 			return this._currentQuote;
 		}
 
+		/**
+		 * The second most recent quote for the symbol of the encapsulated position.
+		 *
+		 * @public
+		 * @returns {null|Object}
+		 */
+		get previousQuote() {
+			return this._previousQuote;
+		}
+
 		updatePortfolio(portfolio) {
 			assert.argumentIsRequired(portfolio, 'portfolio', Object);
 			assert.argumentIsRequired(portfolio.portfolio, 'portfolio.portfolio', String);
@@ -200,6 +211,7 @@ module.exports = (() => {
 				this._currentPricePrevious = this._currentPrice;
 				this._currentPrice = quote.lastPrice;
 
+				this._previousQuote = this._currentQuote;
 				this._currentQuote = quote;
 
 				this._quoteChangedEvent.fire(this._currentQuote);

package/package.json

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

package/test/SpecRunner.js

@@ -1662,8 +1662,17 @@ module.exports = (() => {
 			}
 
 			const existingBarchartSymbols = this.getPositionSymbols(false);
+			const existingPositionItem = this._items.find(item => item.position.position === position.position);
 
-			removePositionItem.call(this, this._items.find(item => item.position.position === position.position));
+			let currentQuote = null;
+			let previousQuote = null;
+
+			if (existingPositionItem) {
+				currentQuote = existingPositionItem.quote || null;
+				previousQuote = existingPositionItem.previousQuote || null;
+			}
+
+			removePositionItem.call(this, existingPositionItem);
 
 			summaries.forEach((summary) => {
 				addSummaryCurrent(this._summariesCurrent, summary, this._currentSummaryFrame, this._currentSummaryRange);
@@ -1675,6 +1684,14 @@ module.exports = (() => {
 			addBarchartSymbol(this._symbols, item);
 			addDisplaySymbol(this._symbolsDisplay, item);
 
+			if (previousQuote !== null) {
+				item.setQuote(previousQuote);
+			}
+
+			if (currentQuote !== null) {
+				item.setQuote(currentQuote);
+			}
+
 			this._items.push(item);
 
 			const createGroupOrInjectItem = (parentTree, treeDefinition, levelDefinitions) => {
@@ -2294,7 +2311,7 @@ module.exports = (() => {
 		}
 	}
 
-	function createPositionItem(position) {
+	function createPositionItem(position, currentQuote, previousQuote) {
 		const portfolio = this._portfolios[position.portfolio];
 
 		let returnRef;
@@ -3277,6 +3294,7 @@ module.exports = (() => {
 			this._data.basis = null;
 
 			this._currentQuote = null;
+			this._previousQuote = null;
 			this._currentPrice = null;
 
 			this._data.currentPrice = null;
@@ -3390,7 +3408,7 @@ module.exports = (() => {
 		}
 
 		/**
-		 * The current quote for the symbol of the encapsulated position.
+		 * The most recent quote for the symbol of the encapsulated position.
 		 *
 		 * @public
 		 * @returns {null|Object}
@@ -3399,6 +3417,16 @@ module.exports = (() => {
 			return this._currentQuote;
 		}
 
+		/**
+		 * The second most recent quote for the symbol of the encapsulated position.
+		 *
+		 * @public
+		 * @returns {null|Object}
+		 */
+		get previousQuote() {
+			return this._previousQuote;
+		}
+
 		updatePortfolio(portfolio) {
 			assert.argumentIsRequired(portfolio, 'portfolio', Object);
 			assert.argumentIsRequired(portfolio.portfolio, 'portfolio.portfolio', String);
@@ -3436,6 +3464,7 @@ module.exports = (() => {
 				this._currentPricePrevious = this._currentPrice;
 				this._currentPrice = quote.lastPrice;
 
+				this._previousQuote = this._currentQuote;
 				this._currentQuote = quote;
 
 				this._quoteChangedEvent.fire(this._currentQuote);