@angular-wave/angular.ts 0.0.4 → 0.0.6

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@angular-wave/angular.ts",
   "license": "MIT",
-  "version": "0.0.4",
+  "version": "0.0.6",
   "type": "module",
   "main": "dist/angular-ts.cjs.js",
   "module": "dist/angular-ts.esm.js",
@@ -24,6 +24,7 @@
     "@playwright/test": "^1.44.0",
     "@rollup/plugin-commonjs": "latest",
     "@rollup/plugin-node-resolve": "latest",
+    "@rollup/plugin-terser": "^0.4.4",
     "@types/jasmine": "latest",
     "@types/node": "latest",
     "eslint": "latest",
@@ -40,8 +41,5 @@
     "sinon": "latest",
     "typescript": "latest",
     "vite": "latest"
-  },
-  "dependencies": {
-   
   }
 }

package/rollup.config.js

@@ -1,6 +1,7 @@
 import resolve from '@rollup/plugin-node-resolve';
 import commonjs from '@rollup/plugin-commonjs';
 import pkg from './package.json' assert { type: 'json' };
+import terser from '@rollup/plugin-terser';
 
 export default [
   // browser-friendly UMD build
@@ -11,7 +12,7 @@ export default [
       file: pkg.browser,
       format: 'umd',
     },
-    plugins: [resolve(), commonjs()],
+    plugins: [resolve(), commonjs(), terser()],
   },
 
   // CommonJS (for Node) and ES module (for bundlers) build.
@@ -27,5 +28,6 @@ export default [
       { file: pkg.main, format: 'cjs' },
       { file: pkg.module, format: 'es' },
     ],
+    plugins: [terser()],
   },
 ];

package/src/core/utils.js

@@ -617,6 +617,14 @@ export function includes(array, obj) {
   return Array.prototype.indexOf.call(array, obj) !== -1;
 }
 
+/**
+ * Removes the first occurrence of a specified value from an array.
+ *
+ * @template T
+ * @param {Array<T>} array - The array from which to remove the value.
+ * @param {T} value - The value to remove.
+ * @returns {number} - The index of the removed value, or -1 if the value was not found.
+ */
 export function arrayRemove(array, value) {
   const index = array.indexOf(value);
   if (index >= 0) {

package/src/directive/switch.js

@@ -1,66 +1,6 @@
 import { forEach } from "../core/utils";
 import { getBlockNodes } from "../jqLite";
 
-/**
- * @ngdoc directive
- * @name ngSwitch
- * @restrict EA
- *
- * @description
- * The `ngSwitch` directive is used to conditionally swap DOM structure on your template based on a scope expression.
- * Elements within `ngSwitch` but without `ngSwitchWhen` or `ngSwitchDefault` directives will be preserved at the location
- * as specified in the template.
- *
- * The directive itself works similar to ngInclude, however, instead of downloading template code (or loading it
- * from the template cache), `ngSwitch` simply chooses one of the nested elements and makes it visible based on which element
- * matches the value obtained from the evaluated expression. In other words, you define a container element
- * (where you place the directive), place an expression on the **`on="..."` attribute**
- * (or the **`ng-switch="..."` attribute**), define any inner elements inside of the directive and place
- * a when attribute per element. The when attribute is used to inform ngSwitch which element to display when the on
- * expression is evaluated. If a matching expression is not found via a when attribute then an element with the default
- * attribute is displayed.
- *
- * <div class="alert alert-info">
- * Be aware that the attribute values to match against cannot be expressions. They are interpreted
- * as literal string values to match against.
- * For example, **`ng-switch-when="someVal"`** will match against the string `"someVal"` not against the
- * value of the expression `$scope.someVal`.
- * </div>
-
- * @animations
- * | Animation                        | Occurs                              |
- * |----------------------------------|-------------------------------------|
- * | {@link ng.$animate#enter enter}  | after the ngSwitch contents change and the matched child element is placed inside the container |
- * | {@link ng.$animate#leave leave}  | after the ngSwitch contents change and just before the former contents are removed from the DOM |
- *
- * @usage
- *
- * ```
- * <ANY ng-switch="expression">
- *   <ANY ng-switch-when="matchValue1">...</ANY>
- *   <ANY ng-switch-when="matchValue2">...</ANY>
- *   <ANY ng-switch-default>...</ANY>
- * </ANY>
- * ```
- *
- *
- * @scope
- * @priority 1200
- * @param {*} ngSwitch|on expression to match against <code>ng-switch-when</code>.
- * On child elements add:
- *
- * * `ngSwitchWhen`: the case statement to match against. If match then this
- *   case will be displayed. If the same match appears multiple times, all the
- *   elements will be displayed. It is possible to associate multiple values to
- *   the same `ngSwitchWhen` by defining the optional attribute
- *   `ngSwitchWhenSeparator`. The separator will be used to split the value of
- *   the `ngSwitchWhen` attribute into multiple tokens, and the element will show
- *   if any of the `ngSwitch` evaluates to any of these tokens.
- * * `ngSwitchDefault`: the default case when no other case match. If there
- *   are multiple default cases, all of them will be displayed when no other
- *   case match.
- *
- */
 export const ngSwitchDirective = [
   "$animate",
   "$compile",
@@ -70,11 +10,13 @@ export const ngSwitchDirective = [
     // asks for $scope to fool the BC controller module
     controller: [
       "$scope",
-      function NgSwitchController() {
-        this.cases = {};
+      class {
+        constructor() {
+          this.cases = {};
+        }
       },
     ],
-    link(scope, element, attr, ngSwitchController) {
+    link(scope, _element, attr, ngSwitchController) {
       const watchExpr = attr.ngSwitch || attr.on;
       let selectedTranscludes = [];
       const selectedElements = [];

package/src/directive/switch.md

@@ -0,0 +1,66 @@
+/\*\*
+
+- @ngdoc directive
+- @name ngSwitch
+- @restrict EA
+-
+- @description
+- The `ngSwitch` directive is used to conditionally swap DOM structure on your template based on a scope expression.
+- Elements within `ngSwitch` but without `ngSwitchWhen` or `ngSwitchDefault` directives will be preserved at the location
+- as specified in the template.
+-
+- The directive itself works similar to ngInclude, however, instead of downloading template code (or loading it
+- from the template cache), `ngSwitch` simply chooses one of the nested elements and makes it visible based on which element
+- matches the value obtained from the evaluated expression. In other words, you define a container element
+- (where you place the directive), place an expression on the **`on="..."` attribute**
+- (or the **`ng-switch="..."` attribute**), define any inner elements inside of the directive and place
+- a when attribute per element. The when attribute is used to inform ngSwitch which element to display when the on
+- expression is evaluated. If a matching expression is not found via a when attribute then an element with the default
+- attribute is displayed.
+-
+- <div class="alert alert-info">
+- Be aware that the attribute values to match against cannot be expressions. They are interpreted
+- as literal string values to match against.
+- For example, **`ng-switch-when="someVal"`** will match against the string `"someVal"` not against the
+- value of the expression `$scope.someVal`.
+- </div>
+
+- @animations
+- | Animation | Occurs |
+- |----------------------------------|-------------------------------------|
+- | {@link ng.$animate#enter enter} | after the ngSwitch contents change and the matched child element is placed inside the container |
+- | {@link ng.$animate#leave leave} | after the ngSwitch contents change and just before the former contents are removed from the DOM |
+-
+- @usage
+-
+- ```
+
+  ```
+
+- <ANY ng-switch="expression">
+- <ANY ng-switch-when="matchValue1">...</ANY>
+- <ANY ng-switch-when="matchValue2">...</ANY>
+- <ANY ng-switch-default>...</ANY>
+- </ANY>
+- ```
+
+  ```
+
+-
+-
+- @scope
+- @priority 1200
+- @param {\*} ngSwitch|on expression to match against <code>ng-switch-when</code>.
+- On child elements add:
+-
+- - `ngSwitchWhen`: the case statement to match against. If match then this
+- case will be displayed. If the same match appears multiple times, all the
+- elements will be displayed. It is possible to associate multiple values to
+- the same `ngSwitchWhen` by defining the optional attribute
+- `ngSwitchWhenSeparator`. The separator will be used to split the value of
+- the `ngSwitchWhen` attribute into multiple tokens, and the element will show
+- if any of the `ngSwitch` evaluates to any of these tokens.
+- - `ngSwitchDefault`: the default case when no other case match. If there
+- are multiple default cases, all of them will be displayed when no other
+- case match.
+- \*/

package/src/jqLite.js

@@ -94,17 +94,8 @@ import { CACHE, EXPANDO } from "./core/cache";
  * @returns {Object} jQuery object.
  */
 
-JQLite.cache = CACHE;
-
 let jqId = 1;
 
-/**
- * !!! This is an undocumented "private" function !!!
- * @param {JQLite|Element} node
- * @returns
- */
-JQLite._data = (node) => JQLite.cache.get(node[EXPANDO]) || {};
-
 function jqNextId() {
   return ++jqId;
 }
@@ -289,6 +280,11 @@ export function JQLite(element) {
 }
 export var jqLite = JQLite;
 
+/**
+ * @param {Element} element
+ * @param {boolean} [onlyDescendants]
+ * @returns {void}
+ */
 export function dealoc(element, onlyDescendants) {
   if (!element) return;
   if (!onlyDescendants && elementAcceptsData(element))
@@ -307,13 +303,13 @@ export function dealoc(element, onlyDescendants) {
  */
 function removeIfEmptyData(element) {
   const expandoId = element[EXPANDO];
-  const { events, data } = JQLite.cache.get(expandoId);
+  const { events, data } = CACHE.get(expandoId);
 
   if (
     (!data || !Object.keys(data).length) &&
     (!events || !Object.keys(events).length)
   ) {
-    JQLite.cache.delete(expandoId);
+    CACHE.delete(expandoId);
     element[EXPANDO] = undefined; // don't delete DOM expandos. IE and Chrome don't like it
   }
 }
@@ -341,8 +337,8 @@ function jqLiteOff(element, type, fn, unsupported) {
   } else {
     const removeHandler = function (type) {
       const listenerFns = events[type];
-      if (isDefined(fn)) {
-        arrayRemove(listenerFns || [], fn);
+      if (isDefined(fn) && isArray(listenerFns)) {
+        arrayRemove(listenerFns, fn);
       }
       if (!(isDefined(fn) && listenerFns && listenerFns.length > 0)) {
         element.removeEventListener(type, handle);
@@ -370,7 +366,7 @@ function jqLiteOff(element, type, fn, unsupported) {
  */
 function jqLiteRemoveData(element, name) {
   const expandoId = element[EXPANDO];
-  const expandoStore = expandoId && JQLite.cache.get(expandoId);
+  const expandoStore = expandoId && CACHE.get(expandoId);
 
   if (expandoStore) {
     if (name) {
@@ -391,7 +387,7 @@ function jqLiteRemoveData(element, name) {
  */
 function jqLiteExpandoStore(element, createIfNecessary = false) {
   let expandoId = element[EXPANDO];
-  let expandoStore = expandoId && JQLite.cache.get(expandoId);
+  let expandoStore = expandoId && CACHE.get(expandoId);
 
   if (createIfNecessary && !expandoStore) {
     element[EXPANDO] = expandoId = jqNextId();
@@ -400,7 +396,7 @@ function jqLiteExpandoStore(element, createIfNecessary = false) {
       data: {},
       handle: null,
     };
-    JQLite.cache.set(expandoId, expandoStore);
+    CACHE.set(expandoId, expandoStore);
   }
 
   return expandoStore;
@@ -417,15 +413,12 @@ function jqLiteData(element, key, value) {
     const data = expandoStore && expandoStore.data;
 
     if (isSimpleSetter) {
-      // data('key', value)
       data[kebabToCamel(key)] = value;
     } else {
       if (massGetter) {
-        // data()
         return data;
       }
       if (isSimpleGetter) {
-        // data('key')
         // don't force creation of expandoStore if it doesn't exist yet
         return data && data[kebabToCamel(key)];
       }
@@ -572,7 +565,7 @@ export function getBooleanAttrName(element, name) {
 
 export function jqLiteCleanData(nodes) {
   for (let i = 0, ii = nodes.length; i < ii; i++) {
-    var events = (jqLite._data(nodes[i]) || {}).events;
+    var events = (CACHE.get(nodes[i][EXPANDO]) || {}).events;
     if (events && events.$destroy) {
       jqLite(nodes[i]).triggerHandler("$destroy");
     }
@@ -707,8 +700,6 @@ forEach(
       let key;
       const nodeCount = this.length;
 
-      // jqLiteHasClass has only two arguments, but is a getter-only fn, so we need to special-case it
-      // in a way that survives minification.
       // jqLiteEmpty takes no arguments but is a setter.
       if (
         fn !== jqLiteEmpty &&
@@ -718,7 +709,6 @@ forEach(
           // we are a write, but the object properties are the key/values
           for (i = 0; i < nodeCount; i++) {
             if (fn === jqLiteData) {
-              // data() takes the whole object in jQuery
               fn(this[i], arg1);
             } else {
               for (key in arg1) {

package/test/jqlite.spec.js

@@ -228,18 +228,6 @@ describe("jqLite", () => {
     });
   });
 
-  describe("_data", () => {
-    it("should provide access to the events present on the element", () => {
-      const element = jqLite("<i>foo</i>");
-      // TODO: REMOVE angular becomes TESTED
-      angular.element = jqLite;
-      expect(angular.element._data(element[0]).events).toBeUndefined();
-
-      element.on("click", () => {});
-      expect(angular.element._data(element[0]).events.click).toBeDefined();
-    });
-  });
-
   describe("inheritedData", () => {
     it("should retrieve data attached to the current element", () => {
       const element = jqLite("<i>foo</i>");
@@ -541,7 +529,7 @@ describe("jqLite", () => {
 
     it("should not break on cleanData(), if element has no data", () => {
       const selected = jqLite([a, b, c]);
-      spyOn(jqLite, "_data").and.returnValue(undefined);
+      spyOn(CACHE, "get").and.returnValue(undefined);
       expect(() => {
         jqLite.cleanData(selected);
       }).not.toThrow();

package/test/ng/compile.spec.js

@@ -13903,7 +13903,7 @@ describe("$compile", () => {
             firstRepeatedElem = element.children(".ng-scope").eq(0);
 
             expect(firstRepeatedElem.data("$scope")).toBeDefined();
-            privateData = JQLite._data(firstRepeatedElem[0]);
+            privateData = CACHE.get(firstRepeatedElem[0]);
             expect(privateData.events).toBeDefined();
 
             expect(privateData.events.click).toBeDefined();
@@ -13919,7 +13919,7 @@ describe("$compile", () => {
 
             expect(destroyCount).toBe(2);
             expect(firstRepeatedElem.data("$scope")).not.toBeDefined();
-            privateData = JQLite._data(firstRepeatedElem[0]);
+            privateData = CACHE.get(firstRepeatedElem[0]);
             expect(privateData && privateData.events).not.toBeDefined();
           }