Revert multi protocol (#1298)

* Revert "Fixing the theme based on the zcl package chosen (#1295)"

This reverts commit 5cc7614.

* Revert "Using getAllSessionPartitionInfoForSession instead of getSessionPartitionInfo for more stability (#1294)"

This reverts commit c9ca9f7.

* Revert "Fixing the ZAP GUI for blank configurations and opening category based configurations (#1289)"

This reverts commit fba7659.

* Revert "ZAP: Multi-protocol Configurator Integration (#1276)"

This reverts commit 06d1428.

Author: paulr34

apack.json

@@ -4,7 +4,7 @@
   "description": "Graphical configuration tool for application and libraries based on Zigbee Cluster Library.",
   "path": [".", "node_modules/.bin/", "ZAP.app/Contents/MacOS"],
   "requiredFeatureLevel": "apack.core:9",
-  "featureLevel": 101,
+  "featureLevel": 100,
   "uc.triggerExtension": "zap",
   "executable": {
     "zap:win32.x86_64": {

docs/zap-schema.svg

@@ -58,7 +58,6 @@
     "package-metadata": "node src-script/zap-package-metadata.js",
     "zap": "node src-script/zap-start.js --logToStdout --gen ./test/gen-template/zigbee/gen-templates.json",
     "zapall": "node src-script/zap-start.js --logToStdout --zcl ./test/resource/meta/zcl.json --zcl ./zcl-builtin/silabs/zcl.json  --zcl ./zcl-builtin/matter/zcl.json --gen ./test/gen-template/zigbee/gen-templates.json --gen ./test/gen-template/matter/gen-test.json --gen ./test/resource/meta/gen-test.json",
-    "zapmultiprotocol": "node src-script/zap-start.js --logToStdout --zcl ./zcl-builtin/silabs/zcl.json  --zcl ./zcl-builtin/matter/zcl-with-test-extensions.json --gen ./test/gen-template/zigbee/gen-templates.json --gen ./test/gen-template/matter/gen-test.json",
     "zapzigbee": "node src-script/zap-start.js --logToStdout --zcl ./zcl-builtin/silabs/zcl.json --gen ./test/gen-template/zigbee/gen-templates.json",
     "zapmatter": "node src-script/zap-start.js --logToStdoutc --zcl ./zcl-builtin/matter/zcl.json --gen ./test/gen-template/matter/gen-test.json",
     "zapmeta": "node src-script/zap-start.js --logToStdout --zcl ./test/resource/meta/zcl.json --gen ./test/resource/meta/gen-test.json --in ./test/resource/test-meta.zap",
@@ -67,7 +66,6 @@
     "zigbeezap-devserver": "node src-script/zap-start.js server --stateDirectory ~/.zap/zigbee-server/ --allowCors --logToStdout --gen ./test/gen-template/zigbee/gen-templates.json --reuseZapInstance",
     "matterzap-devserver": "node src-script/zap-start.js server --stateDirectory ~/.zap/matter-server/ --allowCors --logToStdout --zcl ./zcl-builtin/matter/zcl.json --gen ./test/resource/meta/gen-test.json --reuseZapInstance",
     "matterzap-devserver2": "node src-script/zap-start.js server --stateDirectory --zcl ./zcl-builtin/matter/zcl.json --allowCors --logToStdout --gen ./test/gen-template/matter3/t.json --reuseZapInstance",
-    "zapall-devserver": "node src-script/zap-start.js server --stateDirectory --zcl ./zcl-builtin/silabs/zcl.json  --zcl ./zcl-builtin/matter/zcl.json --allowCors --logToStdout --gen ./test/gen-template/zigbee/gen-templates.json --gen ./test/gen-template/matter/gen-test.json --gen --reuseZapInstance",
     "server": "node src-script/zap-start.js server --stateDirectory ~/.zap/server/ --allowCors --logToStdout --zcl ./zcl-builtin/silabs/zcl.json  --zcl ./zcl-builtin/matter/zcl.json --gen ./test/gen-template/zigbee/gen-templates.json  --gen ./test/gen-template/matter/gen-test.json ",
     "stop": "node src-script/zap-start.js stop --reuseZapInstance",
     "status": "node src-script/zap-start.js status --reuseZapInstance",

package.json

@@ -384,23 +384,6 @@ exports.map = {
       name: x.NAME,
       caption: x.DESCRIPTION,
       class: x.CLASS,
-      packageRef: x.PACKAGE_REF,
-    }
-  },
-
-  deviceTypeExtended: (x) => {
-    if (x == null) return undefined
-    return {
-      id: x.DEVICE_TYPE_ID,
-      code: x.CODE,
-      profileId: x.PROFILE_ID,
-      domain: x.DOMAIN,
-      label: x.NAME,
-      name: x.NAME,
-      caption: x.DESCRIPTION,
-      class: x.CLASS,
-      packageRef: x.PACKAGE_REF,
-      category: x.CATEGORY,
     }
   },
 
@@ -469,25 +452,6 @@ exports.map = {
       parentEndpointIdentifier: x.PARENT_ENDPOINT_IDENTIFIER,
     }
   },
-  endpointExtended: (x) => {
-    if (x == null) return undefined
-    return {
-      id: x.ENDPOINT_ID,
-      endpointRef: x.ENDPOINT_ID,
-      sessionRef: x.SESSION_REF,
-      endpointIdentifier: x.ENDPOINT_IDENTIFIER,
-      endpointId: x.ENDPOINT_IDENTIFIER,
-      endpointTypeRef: x.ENDPOINT_TYPE_REF,
-      profileId: x.PROFILE,
-      networkId: x.NETWORK_IDENTIFIER,
-      endpointVersion: x.DEVICE_VERSION, // Left for backwards compatibility
-      deviceVersion: x.DEVICE_VERSION,
-      deviceIdentifier: x.DEVICE_IDENTIFIER,
-      parentRef: x.PARENT_ENDPOINT_REF,
-      parentEndpointIdentifier: x.PARENT_ENDPOINT_IDENTIFIER,
-      category: x.CATEGORY, // Category of the device type coming from the zcl package it belongs to.
-    }
-  },
   endpointType: (x) => {
     if (x == null) return undefined
     return {
@@ -679,8 +643,6 @@ exports.map = {
       sessionRef: x.SESSION_REF,
       required: x.REQUIRED,
       type: x.TYPE,
-      sessionPartitionId: x.SESSION_PARTITION_ID,
-      category: x.CATEGORY,
     }
   },
   sessionLog: (x) => {
@@ -702,16 +664,6 @@ exports.map = {
       newNotification: x.NEW_NOTIFICATION == 1,
     }
   },
-
-  sessionPartition: (x) => {
-    if (x == null) return undefined
-    return {
-      sessionPartitionId: x.SESSION_PARTITION_ID,
-      sessionRef: x.SESSION_REF,
-      sessionPartitionNumber: x.SESSION_PARTITION_NUMBER,
-    }
-  },
-
   user: (x) => {
     if (x == null) return undefined
     return {

src-electron/db/db-mapping.js

@@ -1102,12 +1102,8 @@ INNER JOIN
   CLUSTER
 ON
   ENDPOINT_TYPE_CLUSTER.CLUSTER_REF = CLUSTER.CLUSTER_ID
-INNER JOIN
-  SESSION_PARTITION
-ON
-ENDPOINT_TYPE.SESSION_PARTITION_REF = SESSION_PARTITION.SESSION_PARTITION_ID
 WHERE
-  SESSION_PARTITION.SESSION_REF = ${sessionId}
+  ENDPOINT_TYPE.SESSION_REF = ${sessionId}
 AND
   ENDPOINT_TYPE_CLUSTER.ENABLED=1
 AND

src-electron/db/query-attribute.js

@@ -218,12 +218,8 @@ async function selectAllUserClustersWithTokenAttributes(
     CLUSTER
   ON
     ENDPOINT_TYPE_CLUSTER.CLUSTER_REF = CLUSTER.CLUSTER_ID
-  INNER JOIN
-    SESSION_PARTITION
-  ON
-  ENDPOINT_TYPE.SESSION_PARTITION_REF = SESSION_PARTITION.SESSION_PARTITION_ID
   WHERE
-    SESSION_PARTITION.SESSION_REF = ?
+    ENDPOINT_TYPE.SESSION_REF = ?
   AND
     ENDPOINT_TYPE_CLUSTER.ENABLED=1
   AND

src-electron/db/query-cluster.js

@@ -27,7 +27,6 @@ const dbEnum = require('../../src-shared/db-enum.js')
 const queryZcl = require('./query-zcl.js')
 const queryUpgrade = require('../sdk/matter.js')
 const queryDeviceType = require('./query-device-type')
-const querySession = require('./query-session')
 const queryCommand = require('./query-command.js')
 const restApi = require('../../src-shared/rest-api.js')
 const _ = require('lodash')
@@ -577,18 +576,16 @@ async function selectCountOfEndpointsWithGivenEndpointIdentifier(
 /**
  * Promises to add an endpoint type.
  *
+ * @export
  * @param {*} db
- * @param {*} sessionPartitionInfo
+ * @param {*} sessionId
  * @param {*} name
  * @param {*} deviceTypeRef
- * @param {*} deviceTypeIdentifier
- * @param {*} deviceTypeVersion
- * @param {*} doTransaction
  * @returns Promise to update endpoints.
  */
 async function insertEndpointType(
   db,
-  sessionPartitionInfo,
+  sessionId,
   name,
   deviceTypeRef,
   deviceTypeIdentifier,
@@ -607,8 +604,8 @@ async function insertEndpointType(
   // Insert endpoint type
   let newEndpointTypeId = await dbApi.dbInsert(
     db,
-    'INSERT OR REPLACE INTO ENDPOINT_TYPE ( SESSION_PARTITION_REF, NAME ) VALUES ( ?, ?)',
-    [sessionPartitionInfo.sessionPartitionId, name]
+    'INSERT OR REPLACE INTO ENDPOINT_TYPE ( SESSION_REF, NAME ) VALUES ( ?, ?)',
+    [sessionId, name]
   )
 
   // Creating endpoint type and device type ref combinations along with order of insertion
@@ -638,7 +635,7 @@ async function insertEndpointType(
       db,
       'ERROR',
       isErrorStringPresent ? err.split('Error:')[1] : err,
-      sessionPartitionInfo.sessionRef,
+      sessionId,
       1,
       1
     )
@@ -659,7 +656,7 @@ async function insertEndpointType(
   for (const dtRef of deviceTypeRefs) {
     await setEndpointDefaults(
       db,
-      sessionPartitionInfo.sessionRef,
+      sessionId,
       newEndpointTypeId,
       dtRef,
       doTransaction
@@ -682,8 +679,7 @@ async function duplicateEndpointType(db, endpointTypeId) {
     db,
     `
     SELECT
-      SESSION_PARTITION.SESSION_REF,
-      SESSION_PARTITION.SESSION_PARTITION_ID,
+      ENDPOINT_TYPE.SESSION_REF,
       ENDPOINT_TYPE.NAME,
       ENDPOINT_TYPE_DEVICE.DEVICE_TYPE_REF,
       ENDPOINT_TYPE_DEVICE.DEVICE_IDENTIFIER,
@@ -694,10 +690,6 @@ async function duplicateEndpointType(db, endpointTypeId) {
       ENDPOINT_TYPE_DEVICE
     ON
       ENDPOINT_TYPE.ENDPOINT_TYPE_ID = ENDPOINT_TYPE_DEVICE.ENDPOINT_TYPE_REF
-    INNER JOIN
-      SESSION_PARTITION
-    ON
-      ENDPOINT_TYPE.SESSION_PARTITION_REF = SESSION_PARTITION.SESSION_PARTITION_ID
     WHERE
       ENDPOINT_TYPE_DEVICE.ENDPOINT_TYPE_REF = ?`,
     [endpointTypeId]
@@ -707,12 +699,9 @@ async function duplicateEndpointType(db, endpointTypeId) {
     // Enter into the endpoint_type table
     newEndpointTypeId = await dbApi.dbInsert(
       db,
-      `INSERT INTO ENDPOINT_TYPE (SESSION_PARTITION_REF, NAME)
+      `INSERT INTO ENDPOINT_TYPE (SESSION_REF, NAME)
       VALUES (?, ?)`,
-      [
-        endpointTypeDeviceInfo[0].SESSION_PARTITION_ID,
-        endpointTypeDeviceInfo[0].NAME,
-      ]
+      [endpointTypeDeviceInfo[0].SESSION_REF, endpointTypeDeviceInfo[0].NAME]
     )
 
     // Enter into the endpoint_type_device table to establish the endpoint_type
@@ -769,12 +758,8 @@ async function updateEndpointType(db, sessionId, endpointTypeId, changesArray) {
     INNER JOIN
       ENDPOINT_TYPE_DEVICE
     ON ENDPOINT_TYPE.ENDPOINT_TYPE_ID = ENDPOINT_TYPE_DEVICE.ENDPOINT_TYPE_REF
-    INNER JOIN
-      SESSION_PARTITION
-    ON
-      ENDPOINT_TYPE.SESSION_PARTITION_REF = SESSION_PARTITION.SESSION_PARTITION_ID
     WHERE
-      ENDPOINT_TYPE.ENDPOINT_TYPE_ID = ? AND SESSION_PARTITION.SESSION_REF = ?
+      ENDPOINT_TYPE_ID = ? AND SESSION_REF = ?
     ORDER BY
       ENDPOINT_TYPE_DEVICE.DEVICE_TYPE_ORDER`,
     [endpointTypeId, sessionId]
@@ -888,10 +873,7 @@ async function updateEndpointType(db, sessionId, endpointTypeId, changesArray) {
 /**
  * Promise to set the default attributes and clusters for a endpoint type.
  * @param {*} db
- * @param {*} sessionId
  * @param {*} endpointTypeId
- * @param {*} deviceTypeRef
- * @param {*} doTransaction
  */
 async function setEndpointDefaults(
   db,
@@ -911,20 +893,7 @@ async function setEndpointDefaults(
   if (pkgs == null || pkgs.length < 1)
     throw new Error('Could not locate package id for a given session.')
 
-  let deviceTypeInfo =
-    await querySession.selectDeviceTypePackageInfoFromDeviceTypeId(
-      db,
-      deviceTypeRef
-    )
-  let endpointTypeCategory =
-    deviceTypeInfo.length > 0 ? deviceTypeInfo[0].category : null
   let packageId = pkgs[0].id
-  for (let i = 0; i < pkgs.length; i++) {
-    if (pkgs[i].category == endpointTypeCategory) {
-      packageId = pkgs[i].id
-      break
-    }
-  }
   let clusters = await queryDeviceType.selectDeviceTypeClustersByDeviceTypeRef(
     db,
     deviceTypeRef
@@ -1247,16 +1216,7 @@ async function resolveNonOptionalAndReportableAttributes(
 async function selectEndpointTypeCount(db, sessionId) {
   let x = await dbApi.dbGet(
     db,
-    `SELECT
-      COUNT(ENDPOINT_TYPE_ID) AS CNT
-    FROM
-      ENDPOINT_TYPE
-    INNER JOIN
-      SESSION_PARTITION
-    ON
-      SESSION_PARTITION.SESSION_PARTITION_ID = ENDPOINT_TYPE.SESSION_PARTITION_REF
-    WHERE
-      SESSION_PARTITION.SESSION_REF = ?`,
+    'SELECT COUNT(ENDPOINT_TYPE_ID) AS CNT FROM ENDPOINT_TYPE WHERE SESSION_REF = ?',
     [sessionId]
   )
   return x['CNT']
@@ -1283,11 +1243,7 @@ SELECT
   COUNT(ENDPOINT_TYPE_ID)
 FROM
   ENDPOINT_TYPE
-INNER JOIN
-  SESSION_PARTITION
-ON
-  SESSION_PARTITION.SESSION_PARTITION_ID = ENDPOINT_TYPE.SESSION_PARTITION_REF
-WHERE SESSION_PARTITION.SESSION_REF = ?
+WHERE SESSION_REF = ?
   AND ENDPOINT_TYPE_ID IN
       (SELECT ENDPOINT_TYPE_REF
        FROM ENDPOINT_TYPE_CLUSTER
@@ -1448,12 +1404,8 @@ JOIN
   ATTRIBUTE AS A ON ETA.ATTRIBUTE_REF = A.ATTRIBUTE_ID
 JOIN
   ENDPOINT_TYPE AS ET ON ETA.ENDPOINT_TYPE_REF = ET.ENDPOINT_TYPE_ID
-INNER JOIN
-  SESSION_PARTITION
-ON
-  ET.SESSION_PARTITION_REF = SESSION_PARTITION.SESSION_PARTITION_ID
 WHERE
-  SESSION_PARTITION.SESSION_REF = ? AND ETA.INCLUDED = 1
+  ET.SESSION_REF = ? AND ETA.INCLUDED = 1
 ORDER BY
   CLUSTER_CODE, ATTRIBUTE_CODE
   `,

src-electron/db/query-config.js

@@ -26,15 +26,15 @@ const dbMapping = require('./db-mapping')
 /**
  * Retrieves all the device types in the database.
  *
+ * @export
  * @param {*} db
- * @param {*} packageId
  * @returns Promise that resolves with the rows of device types.
  */
 async function selectAllDeviceTypes(db, packageId) {
   return dbApi
     .dbAll(
       db,
-      'SELECT DEVICE_TYPE_ID, DOMAIN, CODE, PROFILE_ID, NAME, DESCRIPTION, CLASS, PACKAGE_REF FROM DEVICE_TYPE WHERE PACKAGE_REF = ? ORDER BY DOMAIN, CODE',
+      'SELECT DEVICE_TYPE_ID, DOMAIN, CODE, PROFILE_ID, NAME, DESCRIPTION, CLASS FROM DEVICE_TYPE WHERE PACKAGE_REF = ? ORDER BY DOMAIN, CODE',
       [packageId]
     )
     .then((rows) => rows.map(dbMapping.map.deviceType))
@@ -51,7 +51,7 @@ async function selectDeviceTypeById(db, id) {
   return dbApi
     .dbGet(
       db,
-      'SELECT DEVICE_TYPE_ID, DOMAIN, CODE, PROFILE_ID, NAME, DESCRIPTION, CLASS, PACKAGE_REF FROM DEVICE_TYPE WHERE DEVICE_TYPE_ID = ?',
+      'SELECT DEVICE_TYPE_ID, DOMAIN, CODE, PROFILE_ID, NAME, DESCRIPTION, CLASS FROM DEVICE_TYPE WHERE DEVICE_TYPE_ID = ?',
       [id]
     )
     .then(dbMapping.map.deviceType)