node-red-contrib-mobius-flow-base 3.1.0 → 3.1.1

package/dist/nodes/mobius-discover-object-list.html

@@ -24,7 +24,10 @@ ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
 POSSIBILITY OF SUCH DAMAGE.
 -->
 
-<script type="text/x-red" data-template-name="mobius discover object list">
+<script
+  type="text/x-red"
+  data-template-name="mobius discover object list"
+>
   <div class="form-row">
     <label for="node-input-name"><i class="fa fa-tag"></i> Name</label>
     <input type="text" id="node-input-name" placeholder="Name">
@@ -138,7 +141,7 @@ POSSIBILITY OF SUCH DAMAGE.
     },
     oneditprepare: function () {
       var node = this;
-      let mobiusSrvc = $("#node-input-connection").val();
+      let mobiusSrvc = $('#node-input-connection').val();
 
       /** Create the subs container and populate it */
       $('#node-input-subs-container')
@@ -167,15 +170,21 @@ POSSIBILITY OF SUCH DAMAGE.
             }).appendTo(row);
 
             // *** MOBSEARCH *** //
-            var searchField = $('<a class="searchbutton" style="margin-left: 5px; vertical-align: middle"><i id="node-lookup-instance-icon" class="fa fa-search"></i></a>').appendTo(row);
+            var searchField = $(
+              '<a class="searchbutton" style="margin-left: 5px; vertical-align: middle"><i id="node-lookup-instance-icon" class="fa fa-search"></i></a>'
+            ).appendTo(row);
             // *** END MOBSEARCH *** //
 
             var typeField = $('<select/>', {
               class: 'node-input-sub-type',
               style: 'width:auto; margin-left: 5px; text-align: center;',
             }).appendTo(row);
-            typeField.append($('<option></option>').val('include').text('Include'));
-            typeField.append($('<option></option>').val('exclude').text('Exclude'));
+            typeField.append(
+              $('<option></option>').val('include').text('Include')
+            );
+            typeField.append(
+              $('<option></option>').val('exclude').text('Exclude')
+            );
 
             // Populate the new stat
             uriField.val(subscription.uri);
@@ -184,10 +193,10 @@ POSSIBILITY OF SUCH DAMAGE.
             // *** MOBSEARCH *** //
 
             if (!node.subscriptions[i]) {
-              node.subscriptions[i] = {uri: '', type: 'include'}
+              node.subscriptions[i] = { uri: '', type: 'include' };
             }
 
-            addMobsearchListButton(mobiusSrvc, searchField, i, mobsearchOpts)
+            addMobsearchListButton(mobiusSrvc, searchField, i, mobsearchOpts);
             // *** END MOBSEARCH *** //
           },
         });
@@ -198,7 +207,7 @@ POSSIBILITY OF SUCH DAMAGE.
       if (!node.subscriptions) {
         node.subscriptions = [(subscription = { uri: '', type: 'include' })];
       }
-      
+
       // *** MOBSEARCH *** //
 
       mobsearchOpts = {
@@ -244,7 +253,9 @@ POSSIBILITY OF SUCH DAMAGE.
       }
       var editorRow = $('#dialog-form>div.node-input-subs-container-row');
       height -=
-        4 * (parseInt(editorRow.css('marginTop')) + parseInt(editorRow.css('marginBottom')));
+        4 *
+        (parseInt(editorRow.css('marginTop')) +
+          parseInt(editorRow.css('marginBottom')));
 
       $('#node-input-subs-container').editableList('height', height);
     },
@@ -270,7 +281,10 @@ POSSIBILITY OF SUCH DAMAGE.
   });
 </script>
 
-<script type="text/x-red" data-help-name="mobius discover object list">
+<script
+  type="text/x-red"
+  data-help-name="mobius discover object list"
+>
     <p>Discover MOBiUS objects and return a filtered list of objects or URIs.</p>
 
     <h3>Configuration Page</h3>

package/dist/nodes/mobius-discover-object-list.js

@@ -126,7 +126,11 @@ module.exports = function (RED) {
         let objectList = [];
         try {
           const hid = node.hid === '' ? node.serviceConn.hubId : node.hid;
-          const profiles = await discoverProfiles(node.serviceConn, hid, node.subscriptions);
+          const profiles = await discoverProfiles(
+            node.serviceConn,
+            hid,
+            node.subscriptions
+          );
 
           for await (let entry of profiles.entries()) {
             let discoverParams = {};
@@ -150,15 +154,28 @@ module.exports = function (RED) {
               };
             }
             try {
-              let objects = await filterObjectsByRIDValue(node.serviceConn, discoverParams);
+              let objects = await filterObjectsByRIDValue(
+                node.serviceConn,
+                discoverParams
+              );
               for (let lp = 0; lp < objects.value.length; lp++) {
                 objects.value[lp][
                   'uri'
                 ] = `${serviceURI}/${objects.value[lp].pid}/${objects.value[lp].ins}`;
-                objectList.push(objects.value[lp]);
+                if (
+                  isIncluded(
+                    `${objects.value[lp]['uri']}`,
+                    node.subscriptions
+                  ) &&
+                  !isExcluded(`${objects.value[lp]['uri']}`, node.subscriptions)
+                ) {
+                  objectList.push(objects.value[lp]);
+                }
               }
             } catch (err) {
-              node.warn(`Discover object failed for service (${serviceURI}) - ${err.message}`);
+              node.warn(
+                `Discover object failed for service (${serviceURI}) - ${err.message}`
+              );
             }
           }
 
@@ -185,10 +202,7 @@ module.exports = function (RED) {
             let profiles = await discover(serviceConn, `${hid}/${sid}`);
             let pids = [];
             for (const pid of profiles.data) {
-              if (
-                isIncluded(`${hid}/${sid}/${pid}/#`, subscriptions) &&
-                !isExcluded(`${hid}/${sid}/${pid}/#`, subscriptions)
-              ) {
+              if (isIncluded(`${hid}/${sid}/${pid}/#`, subscriptions)) {
                 pids.push(pid);
               }
             }
@@ -198,7 +212,9 @@ module.exports = function (RED) {
           }
         } else {
           RED.log.warn(
-            `Failed to discover objects for service ${service.id.split('/')[1]} as it is offline.`
+            `Failed to discover objects for service ${
+              service.id.split('/')[1]
+            } as it is offline.`
           );
         }
       }
@@ -240,7 +256,10 @@ module.exports = function (RED) {
   function isIncluded(uri, subscriptions) {
     let include = false;
     for (const subscription of subscriptions) {
-      if (subscription.type === 'include' && validateSubscription(subscription.uri)) {
+      if (
+        subscription.type === 'include' &&
+        validateSubscription(subscription.uri)
+      ) {
         if (matchTopic(subscription.uri, uri)) {
           include = true;
         }
@@ -252,7 +271,10 @@ module.exports = function (RED) {
   function isExcluded(uri, subscriptions) {
     let exclude = false;
     for (const subscription of subscriptions) {
-      if (subscription.type === 'exclude' && validateSubscription(subscription.uri)) {
+      if (
+        subscription.type === 'exclude' &&
+        validateSubscription(subscription.uri)
+      ) {
         if (matchTopic(subscription.uri, uri)) {
           exclude = true;
         }
@@ -280,12 +302,23 @@ module.exports = function (RED) {
   function matchTopic(filter, topic) {
     const filterArray = filter.split('/');
     const topicArray = topic.split('/');
-    const length = filterArray.length < topicArray.length ? filterArray.length : topicArray.length;
+    const length =
+      filterArray.length < topicArray.length
+        ? filterArray.length
+        : topicArray.length;
 
     for (let i = 0; i < length; ++i) {
       const left = filterArray[i];
       const right = topicArray[i];
-      if (!(left === right || left === '+' || left === '#' || right === '+' || right === '#')) {
+      if (
+        !(
+          left === right ||
+          left === '+' ||
+          left === '#' ||
+          right === '+' ||
+          right === '#'
+        )
+      ) {
         return false;
       }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "node-red-contrib-mobius-flow-base",
-  "version": "3.1.0",
+  "version": "3.1.1",
   "description": "Node-RED base nodes for Mobius Flow",
   "main": "index.js",
   "types": "dist/index.d.ts",