chore: isLicensedDeviceSystemWide API

Author: abose

src/services/manage-licenses.js

@@ -170,7 +170,7 @@ define(function (require, exports, module) {
             $valid.show();
 
             // Show reapply button if license is valid but not applied system-wide
-            const isLicensed = await NodeUtils.isLicensedDevice();
+            const isLicensed = await NodeUtils.isLicensedDeviceSystemWide();
             if (!isLicensed) {
                 $reapplyContainer.show();
             }

src/utils/NodeUtils.js

@@ -29,9 +29,12 @@
 
 define(function (require, exports, module) {
     const Strings             = require("strings"),
+        PreferencesManager = require("preferences/PreferencesManager"),
         NodeConnector = require('NodeConnector');
     const UTILS_NODE_CONNECTOR = "ph_utils";
 
+    const PREF_STATE_LICENSED_DEVICE_CHECK = "LICENSED_DEVICE_CHECK";
+
     let utilsConnector;
     if(Phoenix.isNativeApp) {
         // node not available in browser builds!
@@ -203,6 +206,7 @@ define(function (require, exports, module) {
             throw new Error("addDeviceLicense not available in browser");
         }
         try {
+            PreferencesManager.stateManager.set(PREF_STATE_LICENSED_DEVICE_CHECK, true);
             await utilsConnector.execPeer("addDeviceLicense");
             return true;
         } catch (err) {
@@ -223,6 +227,7 @@ define(function (require, exports, module) {
             throw new Error("removeDeviceLicense not available in browser");
         }
         try {
+            PreferencesManager.stateManager.set(PREF_STATE_LICENSED_DEVICE_CHECK, false);
             await utilsConnector.execPeer("removeDeviceLicense");
             return true;
         } catch (err) {
@@ -232,13 +237,12 @@ define(function (require, exports, module) {
     }
 
     /**
-     * Checks if the current machine is licensed.
-     * This validates that the system-wide license file exists,
-     * contains valid JSON, and has `licensedDevice: true`.
+     * Checks if the current machine is configured to check for system-wide device license for all users at app start.
+     * This validates that the system-wide license file exists, contains valid JSON, and has `licensedDevice: true`.
      *
      * @returns {Promise<boolean>} - Resolves with `true` if the device is licensed, `false` otherwise.
      */
-    async function isLicensedDevice() {
+    async function isLicensedDeviceSystemWide() {
         if (!Phoenix.isNativeApp) {
             console.error("isLicensedDevice not available in browser");
             return false;
@@ -251,6 +255,16 @@ define(function (require, exports, module) {
         return false;
     }
 
+    /**
+     * Checks if app is configured to check for device licenses at app start at system or user level.
+     *
+     * @returns {Promise<boolean>} - Resolves with `true` if the device is licensed, `false` otherwise.
+     */
+    async function isLicensedDevice() {
+        const userCheck = PreferencesManager.stateManager.get(PREF_STATE_LICENSED_DEVICE_CHECK);
+        const systemCheck = await isLicensedDeviceSystemWide();
+        return userCheck || systemCheck;
+    }
 
     if(NodeConnector.isNodeAvailable()) {
         // todo we need to update the strings if a user extension adds its translations. Since we dont support
@@ -277,6 +291,8 @@ define(function (require, exports, module) {
         console.error("Error setting Phoenix.isTestWindowGitHubActions", e);
     }
 
+    PreferencesManager.stateManager.definePreference(PREF_STATE_LICENSED_DEVICE_CHECK, "boolean", false);
+
     // private apis
     exports._loadNodeExtensionModule = _loadNodeExtensionModule;
     exports._npmInstallInFolder = _npmInstallInFolder;
@@ -293,6 +309,7 @@ define(function (require, exports, module) {
     exports.addDeviceLicense = addDeviceLicense;
     exports.removeDeviceLicense = removeDeviceLicense;
     exports.isLicensedDevice = isLicensedDevice;
+    exports.isLicensedDeviceSystemWide = isLicensedDeviceSystemWide;
 
     /**
      * checks if Node connector is ready