bruce-models 7.1.100 → 7.1.102

package/dist/bruce-models.umd.js

@@ -364,6 +364,10 @@
             EConcept["UI_SLIDE"] = "pvs";
             EConcept["UI_SLIDE_GROUP"] = "sg";
             EConcept["UI_TABLE_VIEW"] = "tv";
+            EConcept["WORKFLOW_ITEM"] = "wi";
+            EConcept["WORKFLOW_ITEM_STATUS"] = "wis";
+            EConcept["WORKFLOW_ITEM_TYPE"] = "wit";
+            EConcept["WORKFLOW_QUEUE"] = "wq";
         })(EConcept = AccountConcept.EConcept || (AccountConcept.EConcept = {}));
         let EAction;
         (function (EAction) {
@@ -579,14 +583,17 @@
             };
         };
         /**
-         * Returns true if the given Entity internal ID falls within the reported range string.
+         * Returns true if the given internal ID falls within the reported range string.
+         *
+         * Every concept whose records carry a numeric identity reports them this way, as runs rather than
+         * as one ID per record, so that a change touching thousands of rows is still a short message.
          *
          * @example
-         * isEntityInternalIdInRangeString("100-200,305,400-450", 150) // true
-         * isEntityInternalIdInRangeString("100-200,305,400-450", 301) // false
+         * isInternalIdInRangeString("100-200,305,400-450", 150) // true
+         * isInternalIdInRangeString("100-200,305,400-450", 301) // false
          */
-        RecordChangeFeed.isEntityInternalIdInRangeString = (rangeString, entityInternalId) => {
-            if (!rangeString || !Number.isInteger(entityInternalId)) {
+        RecordChangeFeed.isInternalIdInRangeString = (rangeString, internalId) => {
+            if (!rangeString || !Number.isInteger(internalId)) {
                 return false;
             }
             for (const part of rangeString.split(",")) {
@@ -602,12 +609,20 @@
                 }
                 const min = Math.min(start, end);
                 const max = Math.max(start, end);
-                if (entityInternalId >= min && entityInternalId <= max) {
+                if (internalId >= min && internalId <= max) {
                     return true;
                 }
             }
             return false;
         };
+        /**
+         * Returns true if the given Entity internal ID falls within the reported range string.
+         *
+         * The Entity-named form of {@link isInternalIdInRangeString}, kept because it is what the Entity
+         * callers already read as. Ranges are not an Entity idea: a Workflow Item reports its own IDs the
+         * same way.
+         */
+        RecordChangeFeed.isEntityInternalIdInRangeString = (rangeString, entityInternalId) => RecordChangeFeed.isInternalIdInRangeString(rangeString, entityInternalId);
         /**
          * Returns true if the given record ID appears in a comma-separated list string.
          *
@@ -815,8 +830,23 @@
                 }
                 const isEntityConcept = event.concept === ENTITY_CONCEPT;
                 if (!isEntityConcept && params.recordId !== undefined) {
-                    const list = typeof event.data === "string" ? event.data : "";
-                    if (!RecordChangeFeed.isRecordIdInListString(list, params.recordId)) {
+                    const data = typeof event.data === "string" ? event.data : "";
+                    const numericId = Number(params.recordId);
+                    const isNumericIdentity = Number.isInteger(numericId)
+                        && String(params.recordId).trim() === String(numericId);
+                    /*
+                     * A NUMERIC identity is reported as runs, so it is matched against the runs: a Workflow
+                     * Item asking about ID 7 has to see itself in "5-9" and not only in a list that spells
+                     * out every ID. Matching runs still finds a plain numeric ID, since one ID is a run of
+                     * one, so this is what every numeric concept wants.
+                     *
+                     * A STRING identity (a Bookmark, a Project View) has no runs to fall inside and arrives
+                     * as a plain list, which is matched as one.
+                     */
+                    const matched = isNumericIdentity
+                        ? RecordChangeFeed.isInternalIdInRangeString(data, numericId)
+                        : RecordChangeFeed.isRecordIdInListString(data, params.recordId);
+                    if (!matched) {
                         return false;
                     }
                 }
@@ -5552,9 +5582,21 @@
                     body["Scenario"] = scenario;
                 }
                 const res = yield api.POST("v3/entities/historic", body, exports.Api.PrepReqParams(req));
-                const entityIds = records.map(r => r["Entity.ID"]).filter((v, i, a) => v && a.indexOf(v) === i);
+                const seenIds = new Set();
+                const entityIds = records.map(r => r["Entity.ID"]).filter(v => {
+                    if (!v || seenIds.has(v))
+                        return false;
+                    seenIds.add(v);
+                    return true;
+                });
                 ClearCacheByEntityIds(api, entityIds);
-                for (const attrKey of records.map(r => r.AttrKey).filter((v, i, a) => v && a.indexOf(v) === i)) {
+                const seenKeys = new Set();
+                for (const attrKey of records.map(r => r.AttrKey).filter(v => {
+                    if (!v || seenKeys.has(v))
+                        return false;
+                    seenKeys.add(v);
+                    return true;
+                })) {
                     api.Cache.RemoveByContains(exports.Entity.GetHistoricContainsKey(attrKey));
                 }
                 return {
@@ -23685,7 +23727,7 @@
     }
 
     // This is updated with the package.json version on build.
-    const VERSION = "7.1.100";
+    const VERSION = "7.1.102";
 
     exports.VERSION = VERSION;
     exports.AbstractApi = AbstractApi;