@ouroboros/browser 0.5.5 → 0.5.6

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@ouroboros/browser",
-	"version": "0.5.5",
+	"version": "0.5.6",
 	"description": "Package to handle common browser functionality.",
 	"main": "index.js",
 	"types": "index.d.ts",

package/pageVisibility.d.ts

@@ -17,6 +17,7 @@ import Subscribe, { SubscribeCallback, SubscribeReturn } from '@ouroboros/subscr
  * @extends Subscribe
  */
 declare class PageVisibility extends Subscribe {
+    #private;
     /**
      * Constructor
      *
@@ -27,6 +28,17 @@ declare class PageVisibility extends Subscribe {
      * @returns PageVisibility
      */
     constructor();
+    /**
+     * Set
+     *
+     * Overrides Subscribe.set to make sure no one can call it without a failure
+     *
+     * @name set
+     * @access public
+     * @param data The new data to set and then send
+     * @returns void
+     */
+    set(data: any): boolean;
     /**
      * Subscribe
      *

package/pageVisibility.js

@@ -35,22 +35,6 @@ else {
         }
     }
 }
-/**
- * Track
- *
- * The actual function passed to the event so that we only have one event
- * listener
- *
- * @name track
- * @access private
- */
-function track() {
-    // Set the data so the instance notifies subscribers
-    pageVisibility.set({
-        property: document[properties.property],
-        state: document[properties.state]
-    });
-}
 /**
  * Page Visibility
  *
@@ -76,6 +60,19 @@ class PageVisibility extends Subscribe {
             state: document[properties.state]
         } : null);
     }
+    /**
+     * Set
+     *
+     * Overrides Subscribe.set to make sure no one can call it without a failure
+     *
+     * @name set
+     * @access public
+     * @param data The new data to set and then send
+     * @returns void
+     */
+    set(data) {
+        throw new Error('Calling PageVisibility.set() is forbidden');
+    }
     /**
      * Subscribe
      *
@@ -90,11 +87,27 @@ class PageVisibility extends Subscribe {
         // If we have no current subscribers
         if (apiAvailable && this.subscribeCallbacks.length === 0) {
             // Add the event listener
-            document.addEventListener(properties.event, track);
+            document.addEventListener(properties.event, this.#track);
         }
         // Call the parent subscribe and return
         return super.subscribe(callback);
     }
+    /**
+     * Track
+     *
+     * The actual function passed to the event so that we only have one event
+     * listener
+     *
+     * @name track
+     * @access private
+     */
+    #track = () => {
+        // Set the data so the instance notifies subscribers
+        super.set({
+            property: document[properties.property],
+            state: document[properties.state]
+        });
+    };
     /**
      * Unsubscribe
      *
@@ -115,7 +128,7 @@ class PageVisibility extends Subscribe {
         // If it was successful and we have no more callbacks
         if (bRet && this.subscribeCallbacks.length === 0) {
             // Remove the event listener
-            document.removeEventListener(properties.event, track);
+            document.removeEventListener(properties.event, this.#track);
         }
         // Return
         return bRet;

package/src/pageVisibility.ts

@@ -45,24 +45,6 @@ else {
 	}
 }
 
-/**
- * Track
- *
- * The actual function passed to the event so that we only have one event
- * listener
- *
- * @name track
- * @access private
- */
-function track(): void {
-
-	// Set the data so the instance notifies subscribers
-	pageVisibility.set({
-		property: document[properties.property as keyof Document],
-		state: document[properties.state as keyof Document]
-	});
-}
-
 /**
  * Page Visibility
  *
@@ -91,6 +73,20 @@ class PageVisibility extends Subscribe {
 		} : null);
 	}
 
+	/**
+	 * Set
+	 *
+	 * Overrides Subscribe.set to make sure no one can call it without a failure
+	 *
+	 * @name set
+	 * @access public
+	 * @param data The new data to set and then send
+	 * @returns void
+	 */
+	set(data: any): boolean {
+		throw new Error('Calling PageVisibility.set() is forbidden');
+	}
+
 	/**
 	 * Subscribe
 	 *
@@ -107,13 +103,31 @@ class PageVisibility extends Subscribe {
 		if(apiAvailable && this.subscribeCallbacks.length === 0) {
 
 			// Add the event listener
-			document.addEventListener(properties.event, track);
+			document.addEventListener(properties.event, this.#track);
 		}
 
 		// Call the parent subscribe and return
 		return super.subscribe(callback);
 	}
 
+	/**
+	 * Track
+	 *
+	 * The actual function passed to the event so that we only have one event
+	 * listener
+	 *
+	 * @name track
+	 * @access private
+	 */
+	#track = () => {
+
+		// Set the data so the instance notifies subscribers
+		super.set({
+			property: document[properties.property as keyof Document],
+			state: document[properties.state as keyof Document]
+		});
+	}
+
 	/**
 	 * Unsubscribe
 	 *
@@ -138,7 +152,7 @@ class PageVisibility extends Subscribe {
 		if(bRet && this.subscribeCallbacks.length === 0) {
 
 			// Remove the event listener
-			document.removeEventListener(properties.event, track);
+			document.removeEventListener(properties.event, this.#track);
 		}
 
 		// Return