@barchart/portfolio-api-common 1.2.75 → 1.2.79

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (3) hide show
  1. package/lib/processing/PositionContainer.js +11 -6
  2. package/package.json +1 -1
  3. package/test/SpecRunner.js +12 -7
package/lib/processing/PositionContainer.js CHANGED
@@ -6,6 +6,7 @@ const array = require('@barchart/common-js/lang/array'),
6
6
  Decimal = require('@barchart/common-js/lang/Decimal'),
7
7
  DisposableStack = require('@barchart/common-js/collections/specialized/DisposableStack'),
8
8
  Event = require('@barchart/common-js/messaging/Event'),
9
+ is = require('@barchart/common-js/lang/is'),
9
10
  Rate = require('@barchart/common-js/lang/Rate'),
10
11
  Tree = require('@barchart/common-js/collections/Tree');
11
12
 
@@ -371,15 +372,17 @@ module.exports = (() => {
371
372
  }
372
373
 
373
374
  setPositionLock(position) {
374
- assert.argumentIsRequired(position, 'position', Object);
375
- assert.argumentIsRequired(position.position, 'position.position', String);
375
+ if (position) {
376
+ assert.argumentIsRequired(position, 'position', Object);
377
+ assert.argumentIsRequired(position.position, 'position.position', String);
376
378
 
377
- const item = this._items.find((i) => i.position.position === position.position);
379
+ const item = this._items.find((i) => i.position.position === position.position);
378
380
 
379
- if (item) {
380
- const locked = is.object(position.system) && is.boolean(position.system.locked) && position.system.locked;
381
+ if (item) {
382
+ const locked = is.object(position.system) && is.boolean(position.system.locked) && position.system.locked;
381
383
 
382
- item.setPositionLock(locked);
384
+ item.setPositionLock(locked);
385
+ }
383
386
  }
384
387
  }
385
388
 
@@ -730,6 +733,8 @@ module.exports = (() => {
730
733
  });
731
734
  }
732
735
  }
736
+
737
+ recalculatePercentages.call(this);
733
738
  }));
734
739
  }
735
740
 
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "@barchart/portfolio-api-common",
3
- "version": "1.2.75",
3
+ "version": "1.2.79",
4
4
  "description": "Common classes used by the Portfolio system",
5
5
  "author": {
6
6
  "name": "Bryan Ingle",
package/test/SpecRunner.js CHANGED
@@ -1385,6 +1385,7 @@ const array = require('@barchart/common-js/lang/array'),
1385
1385
  Decimal = require('@barchart/common-js/lang/Decimal'),
1386
1386
  DisposableStack = require('@barchart/common-js/collections/specialized/DisposableStack'),
1387
1387
  Event = require('@barchart/common-js/messaging/Event'),
1388
+ is = require('@barchart/common-js/lang/is'),
1388
1389
  Rate = require('@barchart/common-js/lang/Rate'),
1389
1390
  Tree = require('@barchart/common-js/collections/Tree');
1390
1391
 
@@ -1750,15 +1751,17 @@ module.exports = (() => {
1750
1751
  }
1751
1752
 
1752
1753
  setPositionLock(position) {
1753
- assert.argumentIsRequired(position, 'position', Object);
1754
- assert.argumentIsRequired(position.position, 'position.position', String);
1754
+ if (position) {
1755
+ assert.argumentIsRequired(position, 'position', Object);
1756
+ assert.argumentIsRequired(position.position, 'position.position', String);
1755
1757
 
1756
- const item = this._items.find((i) => i.position.position === position.position);
1758
+ const item = this._items.find((i) => i.position.position === position.position);
1757
1759
 
1758
- if (item) {
1759
- const locked = is.object(position.system) && is.boolean(position.system.locked) && position.system.locked;
1760
+ if (item) {
1761
+ const locked = is.object(position.system) && is.boolean(position.system.locked) && position.system.locked;
1760
1762
 
1761
- item.setPositionLock(locked);
1763
+ item.setPositionLock(locked);
1764
+ }
1762
1765
  }
1763
1766
  }
1764
1767
 
@@ -2109,6 +2112,8 @@ module.exports = (() => {
2109
2112
  });
2110
2113
  }
2111
2114
  }
2115
+
2116
+ recalculatePercentages.call(this);
2112
2117
  }));
2113
2118
  }
2114
2119
 
@@ -2331,7 +2336,7 @@ module.exports = (() => {
2331
2336
  return PositionContainer;
2332
2337
  })();
2333
2338
 
2334
- },{"./../data/PositionSummaryFrame":3,"./PositionGroup":7,"./PositionItem":8,"./definitions/PositionLevelDefinition":9,"./definitions/PositionLevelType":10,"./definitions/PositionTreeDefinition":11,"@barchart/common-js/collections/Tree":15,"@barchart/common-js/collections/sorting/ComparatorBuilder":16,"@barchart/common-js/collections/sorting/comparators":17,"@barchart/common-js/collections/specialized/DisposableStack":18,"@barchart/common-js/lang/Currency":20,"@barchart/common-js/lang/Decimal":22,"@barchart/common-js/lang/Rate":26,"@barchart/common-js/lang/array":28,"@barchart/common-js/lang/assert":29,"@barchart/common-js/messaging/Event":35}],7:[function(require,module,exports){
2339
+ },{"./../data/PositionSummaryFrame":3,"./PositionGroup":7,"./PositionItem":8,"./definitions/PositionLevelDefinition":9,"./definitions/PositionLevelType":10,"./definitions/PositionTreeDefinition":11,"@barchart/common-js/collections/Tree":15,"@barchart/common-js/collections/sorting/ComparatorBuilder":16,"@barchart/common-js/collections/sorting/comparators":17,"@barchart/common-js/collections/specialized/DisposableStack":18,"@barchart/common-js/lang/Currency":20,"@barchart/common-js/lang/Decimal":22,"@barchart/common-js/lang/Rate":26,"@barchart/common-js/lang/array":28,"@barchart/common-js/lang/assert":29,"@barchart/common-js/lang/is":33,"@barchart/common-js/messaging/Event":35}],7:[function(require,module,exports){
2335
2340
  const array = require('@barchart/common-js/lang/array'),
2336
2341
  assert = require('@barchart/common-js/lang/assert'),
2337
2342
  Currency = require('@barchart/common-js/lang/Currency'),