@angular-wave/angular.ts 0.0.45 → 0.0.47

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@angular-wave/angular.ts",
   "license": "MIT",
-  "version": "0.0.45",
+  "version": "0.0.47",
   "type": "module",
   "main": "dist/angular-ts.esm.js",
   "browser": "dist/angular-ts.umd.js",

package/src/angular.spec.js

@@ -21,7 +21,7 @@ import {
   toJson,
   fromJson,
   nextUid,
-  nodeName_,
+  getNodeName,
   snakeCase,
 } from "./shared/utils";
 import { dealoc, JQLite, startingTag } from "./shared/jqlite/jqlite";
@@ -1827,14 +1827,14 @@ describe("angular", () => {
     });
   });
 
-  describe("nodeName_", () => {
+  describe("getNodeName", () => {
     it('should correctly detect node name with "namespace" when xmlns is defined', () => {
       const div = JQLite(
         '<div xmlns:ngtest="http://angularjs.org/">' +
           '<ngtest:foo ngtest:attr="bar"></ngtest:foo>' +
           "</div>",
       )[0];
-      expect(nodeName_(div.childNodes[0])).toBe("ngtest:foo");
+      expect(getNodeName(div.childNodes[0])).toBe("ngtest:foo");
       expect(div.childNodes[0].getAttribute("ngtest:attr")).toBe("bar");
     });
 
@@ -1844,13 +1844,13 @@ describe("angular", () => {
           '<ngtest:foo ngtest:attr="bar"></ng-test>' +
           "</div>",
       )[0];
-      expect(nodeName_(div.childNodes[0])).toBe("ngtest:foo");
+      expect(getNodeName(div.childNodes[0])).toBe("ngtest:foo");
       expect(div.childNodes[0].getAttribute("ngtest:attr")).toBe("bar");
     });
 
     it("should return undefined for elements without the .nodeName property", () => {
       // some elements, like SVGElementInstance don't have .nodeName property
-      expect(nodeName_({})).toBeUndefined();
+      expect(getNodeName({})).toBeUndefined();
     });
   });
 

package/src/animations/animate-css-driver.js

@@ -137,7 +137,7 @@ export const $$AnimateCssDriverProvider = [
 
             // we iterate directly since safari messes up and doesn't return
             // all the keys for the coords object when iterated
-            forEach(["width", "height", "top", "left"], (key) => {
+            ["width", "height", "top", "left"].forEach((key) => {
               let value = coords[key];
               switch (key) {
                 case "top":

package/src/animations/animate-queue.js

@@ -54,7 +54,7 @@ export const $$AnimateQueueProvider = [
       const keys = classString.split(ONE_SPACE);
       const map = Object.create(null);
 
-      forEach(keys, (key) => {
+      keys.forEach((key) => {
         map[key] = true;
       });
       return map;
@@ -256,7 +256,7 @@ export const $$AnimateQueueProvider = [
           const matches = [];
           const entries = callbackRegistry[event];
           if (entries) {
-            forEach(entries, (entry) => {
+            entries.forEach((entry) => {
               if (contains.call(entry.node, targetNode)) {
                 matches.push(entry.callback);
               } else if (
@@ -694,7 +694,7 @@ export const $$AnimateQueueProvider = [
             runInNextPostDigestOrNow(() => {
               const callbacks = findCallbacks(parentNode, node, event);
               if (callbacks.length) {
-                forEach(callbacks, (callback) => {
+                callbacks.forEach((callback) => {
                   callback(element, phase, data);
                 });
                 cleanupEventListeners(phase, node);

package/src/core/compile/compile.js

@@ -26,7 +26,7 @@ import {
   inherit,
   isUndefined,
   arrayRemove,
-  nodeName_,
+  getNodeName,
   bind,
   trim,
   isBoolean,
@@ -1120,7 +1120,7 @@ export function $CompileProvider($provide, $$sanitizeUriProvider) {
         if (!node) {
           return "html";
         }
-        return nodeName_(node) !== "foreignobject" &&
+        return getNodeName(node) !== "foreignobject" &&
           toString.call(node).match(/SVG/)
           ? "svg"
           : "html";
@@ -1822,7 +1822,8 @@ export function $CompileProvider($provide, $$sanitizeUriProvider) {
                 // Add the matching elements into their slot
 
                 forEach(JQLite($compileNode[0].childNodes), (node) => {
-                  const slotName = slotMap[directiveNormalize(nodeName_(node))];
+                  const slotName =
+                    slotMap[directiveNormalize(getNodeName(node))];
                   if (slotName) {
                     filledSlots[slotName] = true;
                     slots[slotName] =
@@ -2911,7 +2912,7 @@ export function $CompileProvider($provide, $$sanitizeUriProvider) {
           );
         }
 
-        const nodeName = nodeName_(node);
+        const nodeName = getNodeName(node);
         const trustedContext = getTrustedPropContext(nodeName, propName);
 
         let sanitizer = (x) => x;
@@ -2969,7 +2970,7 @@ export function $CompileProvider($provide, $$sanitizeUriProvider) {
         name,
         isNgAttr,
       ) {
-        const nodeName = nodeName_(node);
+        const nodeName = getNodeName(node);
         const trustedContext = getTrustedAttrContext(nodeName, name);
         const mustHaveExpression = !isNgAttr;
         const allOrNothing = ALL_OR_NOTHING_ATTRS[name] || isNgAttr;

package/src/core/compile/compile.spec.js

@@ -6,7 +6,7 @@ import {
   isFunction,
   valueFn,
   isElement,
-  nodeName_,
+  getNodeName,
   extend,
 } from "../../shared/utils";
 import { countChildScopes, countWatchers } from "../../core/scope/scope";
@@ -5603,7 +5603,7 @@ describe("$compile", () => {
         expect(() => {
           element = $compile("<div replace-with-tr></div>")($rootScope);
         }).not.toThrow();
-        expect(nodeName_(element)).toMatch(/tr/i);
+        expect(getNodeName(element)).toMatch(/tr/i);
       });
 
       it("should support templates with root <td> tags", () => {
@@ -5611,7 +5611,7 @@ describe("$compile", () => {
         expect(() => {
           element = $compile("<div replace-with-td></div>")($rootScope);
         }).not.toThrow();
-        expect(nodeName_(element)).toMatch(/td/i);
+        expect(getNodeName(element)).toMatch(/td/i);
       });
 
       it("should support templates with root <th> tags", () => {
@@ -5619,7 +5619,7 @@ describe("$compile", () => {
         expect(() => {
           element = $compile("<div replace-with-th></div>")($rootScope);
         }).not.toThrow();
-        expect(nodeName_(element)).toMatch(/th/i);
+        expect(getNodeName(element)).toMatch(/th/i);
       });
 
       it("should support templates with root <thead> tags", () => {
@@ -5627,7 +5627,7 @@ describe("$compile", () => {
         expect(() => {
           element = $compile("<div replace-with-thead></div>")($rootScope);
         }).not.toThrow();
-        expect(nodeName_(element)).toMatch(/thead/i);
+        expect(getNodeName(element)).toMatch(/thead/i);
       });
 
       it("should support templates with root <tbody> tags", () => {
@@ -5635,7 +5635,7 @@ describe("$compile", () => {
         expect(() => {
           element = $compile("<div replace-with-tbody></div>")($rootScope);
         }).not.toThrow();
-        expect(nodeName_(element)).toMatch(/tbody/i);
+        expect(getNodeName(element)).toMatch(/tbody/i);
       });
 
       it("should support templates with root <tfoot> tags", () => {
@@ -5643,7 +5643,7 @@ describe("$compile", () => {
         expect(() => {
           element = $compile("<div replace-with-tfoot></div>")($rootScope);
         }).not.toThrow();
-        expect(nodeName_(element)).toMatch(/tfoot/i);
+        expect(getNodeName(element)).toMatch(/tfoot/i);
       });
 
       it("should support templates with root <option> tags", () => {
@@ -5651,7 +5651,7 @@ describe("$compile", () => {
         expect(() => {
           element = $compile("<div replace-with-option></div>")($rootScope);
         }).not.toThrow();
-        expect(nodeName_(element)).toMatch(/option/i);
+        expect(getNodeName(element)).toMatch(/option/i);
       });
 
       it("should support templates with root <optgroup> tags", () => {
@@ -5659,7 +5659,7 @@ describe("$compile", () => {
         expect(() => {
           element = $compile("<div replace-with-optgroup></div>")($rootScope);
         }).not.toThrow();
-        expect(nodeName_(element)).toMatch(/optgroup/i);
+        expect(getNodeName(element)).toMatch(/optgroup/i);
       });
 
       it("should support SVG templates using directive.templateNamespace=svg", () => {
@@ -5681,7 +5681,7 @@ describe("$compile", () => {
         )($rootScope);
         const child = element.children().eq(0);
         $rootScope.$digest();
-        expect(nodeName_(child)).toMatch(/a/i);
+        expect(getNodeName(child)).toMatch(/a/i);
         expect(isSVGElement(child[0])).toBe(true);
         expect(child[0].href.baseVal).toBe("/foo/bar");
       });
@@ -5710,7 +5710,7 @@ describe("$compile", () => {
         );
         $rootScope.$digest();
         const child = element.children().eq(0);
-        expect(nodeName_(child)).toMatch(/msup/i);
+        expect(getNodeName(child)).toMatch(/msup/i);
         expect(isUnknownElement(child[0])).toBe(false);
         expect(isHTMLElement(child[0])).toBe(false);
       });
@@ -6566,7 +6566,7 @@ describe("$compile", () => {
           element = $compile("<div replace-with-tr></div>")($rootScope);
         }).not.toThrow();
         $rootScope.$digest();
-        expect(nodeName_(element)).toMatch(/tr/i);
+        expect(getNodeName(element)).toMatch(/tr/i);
       });
 
       it("should support templates with root <td> tags", () => {
@@ -6575,7 +6575,7 @@ describe("$compile", () => {
           element = $compile("<div replace-with-td></div>")($rootScope);
         }).not.toThrow();
         $rootScope.$digest();
-        expect(nodeName_(element)).toMatch(/td/i);
+        expect(getNodeName(element)).toMatch(/td/i);
       });
 
       it("should support templates with root <th> tags", () => {
@@ -6584,7 +6584,7 @@ describe("$compile", () => {
           element = $compile("<div replace-with-th></div>")($rootScope);
         }).not.toThrow();
         $rootScope.$digest();
-        expect(nodeName_(element)).toMatch(/th/i);
+        expect(getNodeName(element)).toMatch(/th/i);
       });
 
       it("should support templates with root <thead> tags", () => {
@@ -6593,7 +6593,7 @@ describe("$compile", () => {
           element = $compile("<div replace-with-thead></div>")($rootScope);
         }).not.toThrow();
         $rootScope.$digest();
-        expect(nodeName_(element)).toMatch(/thead/i);
+        expect(getNodeName(element)).toMatch(/thead/i);
       });
 
       it("should support templates with root <tbody> tags", () => {
@@ -6602,7 +6602,7 @@ describe("$compile", () => {
           element = $compile("<div replace-with-tbody></div>")($rootScope);
         }).not.toThrow();
         $rootScope.$digest();
-        expect(nodeName_(element)).toMatch(/tbody/i);
+        expect(getNodeName(element)).toMatch(/tbody/i);
       });
 
       it("should support templates with root <tfoot> tags", () => {
@@ -6611,7 +6611,7 @@ describe("$compile", () => {
           element = $compile("<div replace-with-tfoot></div>")($rootScope);
         }).not.toThrow();
         $rootScope.$digest();
-        expect(nodeName_(element)).toMatch(/tfoot/i);
+        expect(getNodeName(element)).toMatch(/tfoot/i);
       });
 
       it("should support templates with root <option> tags", () => {
@@ -6620,7 +6620,7 @@ describe("$compile", () => {
           element = $compile("<div replace-with-option></div>")($rootScope);
         }).not.toThrow();
         $rootScope.$digest();
-        expect(nodeName_(element)).toMatch(/option/i);
+        expect(getNodeName(element)).toMatch(/option/i);
       });
 
       it("should support templates with root <optgroup> tags", () => {
@@ -6629,7 +6629,7 @@ describe("$compile", () => {
           element = $compile("<div replace-with-optgroup></div>")($rootScope);
         }).not.toThrow();
         $rootScope.$digest();
-        expect(nodeName_(element)).toMatch(/optgroup/i);
+        expect(getNodeName(element)).toMatch(/optgroup/i);
       });
 
       it("should support SVG templates using directive.templateNamespace=svg", () => {
@@ -6656,7 +6656,7 @@ describe("$compile", () => {
             )($rootScope);
             $rootScope.$digest();
             const child = element.children().eq(0);
-            expect(nodeName_(child)).toMatch(/a/i);
+            expect(getNodeName(child)).toMatch(/a/i);
             expect(isSVGElement(child[0])).toBe(true);
             expect(child[0].href.baseVal).toBe("/foo/bar");
           },
@@ -6692,7 +6692,7 @@ describe("$compile", () => {
             );
             $rootScope.$digest();
             const child = element.children().eq(0);
-            expect(nodeName_(child)).toMatch(/msup/i);
+            expect(getNodeName(child)).toMatch(/msup/i);
             expect(isUnknownElement(child[0])).toBe(false);
             expect(isHTMLElement(child[0])).toBe(false);
           },
@@ -14789,10 +14789,10 @@ describe("$compile", () => {
           JQLite("<div>before<div transclude></div>after</div>")[0].childNodes,
         );
         expect(element.length).toEqual(3);
-        expect(nodeName_(element[1])).toBe("div");
+        expect(getNodeName(element[1])).toBe("div");
         $compile(element)($rootScope);
-        expect(nodeName_(element[1])).toBe("#comment");
-        expect(nodeName_(comment)).toBe("#comment");
+        expect(getNodeName(element[1])).toBe("#comment");
+        expect(getNodeName(comment)).toBe("#comment");
       });
 
       it("should terminate compilation only for element transclusion", () => {
@@ -14973,7 +14973,7 @@ describe("$compile", () => {
           .directive("innerAgain", () => ({
             transclude: true,
             link(scope, element, attr, controllers, transclude) {
-              log.push(`innerAgain:${nodeName_(element)}`);
+              log.push(`innerAgain:${getNodeName(element)}`);
               transclude(scope, (clone) => {
                 element.parent().append(clone);
               });
@@ -14983,13 +14983,13 @@ describe("$compile", () => {
             replace: true,
             templateUrl: "inner.html",
             link(scope, element) {
-              log.push(`inner:${nodeName_(element)}`);
+              log.push(`inner:${getNodeName(element)}`);
             },
           }))
           .directive("outer", () => ({
             transclude: true,
             link(scope, element, attrs, controllers, transclude) {
-              log.push(`outer:${nodeName_(element)}`);
+              log.push(`outer:${getNodeName(element)}`);
               transclude(scope, (clone) => {
                 element.parent().append(clone);
               });

package/src/core/filter/filter.js

@@ -1,4 +1,4 @@
-import { forEach, isObject } from "../../shared/utils";
+import { isObject } from "../../shared/utils";
 import { filterFilter } from "../../filters/filter";
 import { jsonFilter } from "../../filters/filters";
 import { limitToFilter } from "../../filters/limit-to";
@@ -11,7 +11,7 @@ export function $FilterProvider($provide) {
   function register(name, factory) {
     if (isObject(name)) {
       const filters = {};
-      forEach(name, (filter, key) => {
+      Object.entries(name).forEach(([key, filter]) => {
         filters[key] = register(key, filter);
       });
       return filters;

package/src/core/location/location.js

@@ -583,7 +583,7 @@ const locationPrototype = {
         } else if (isObject(search)) {
           search = structuredClone(search, {});
           // remove object undefined or null properties
-          forEach(search, (value, key) => {
+          Object.entries(search).forEach(([key, value]) => {
             if (value == null) delete search[key];
           });
 

package/src/core/parser/parse.js

@@ -778,7 +778,7 @@ function findConstantAndWatchExpressions(ast, $filter, parentIsPure) {
   switch (ast.type) {
     case ASTType.Program:
       allConstants = true;
-      forEach(ast.body, (expr) => {
+      /** @type {[any]} */ (ast.body).forEach((expr) => {
         findConstantAndWatchExpressions(expr.expression, $filter, astIsPure);
         allConstants = allConstants && expr.expression.constant;
       });

package/src/directive/attrs/attrs.js

@@ -1,194 +1,13 @@
 import { BOOLEAN_ATTR } from "../../shared/jqlite/jqlite";
-import { forEach, directiveNormalize } from "../../shared/utils";
+import { directiveNormalize } from "../../shared/utils";
 import { ALIASED_ATTR } from "../../shared/constants";
 
-/**
- * @ngdoc directive
- * @name ngHref
- * @restrict A
- * @priority 99
- *
- * @description
- * Using AngularJS markup like `{{hash}}` in an href attribute will
- * make the link go to the wrong URL if the user clicks it before
- * AngularJS has a chance to replace the `{{hash}}` markup with its
- * value. Until AngularJS replaces the markup the link will be broken
- * and will most likely return a 404 error. The `ngHref` directive
- * solves this problem.
- *
- * The wrong way to write it:
- * ```html
- * <a href="http://www.gravatar.com/avatar/{{hash}}">link1</a>
- * ```
- *
- * The correct way to write it:
- * ```html
- * <a ng-href="http://www.gravatar.com/avatar/{{hash}}">link1</a>
- * ```
- *
- * @element A
- * @param {template} ngHref any string which can contain `{{}}` markup.
- *
- */
-
 export const REGEX_STRING_REGEXP = /^\/(.+)\/([a-z]*)$/;
-/**
- * @ngdoc directive
- * @name ngSrc
- * @restrict A
- * @priority 99
- *
- * @description
- * Using AngularJS markup like `{{hash}}` in a `src` attribute doesn't
- * work right: The browser will fetch from the URL with the literal
- * text `{{hash}}` until AngularJS replaces the expression inside
- * `{{hash}}`. The `ngSrc` directive solves this problem.
- *
- * The buggy way to write it:
- * ```html
- * <img src="http://www.gravatar.com/avatar/{{hash}}" alt="Description"/>
- * ```
- *
- * The correct way to write it:
- * ```html
- * <img ng-src="http://www.gravatar.com/avatar/{{hash}}" alt="Description" />
- * ```
- *
- * @element IMG
- * @param {template} ngSrc any string which can contain `{{}}` markup.
- */
-
-/**
- * @ngdoc directive
- * @name ngSrcset
- * @restrict A
- * @priority 99
- *
- * @description
- * Using AngularJS markup like `{{hash}}` in a `srcset` attribute doesn't
- * work right: The browser will fetch from the URL with the literal
- * text `{{hash}}` until AngularJS replaces the expression inside
- * `{{hash}}`. The `ngSrcset` directive solves this problem.
- *
- * The buggy way to write it:
- * ```html
- * <img srcset="http://www.gravatar.com/avatar/{{hash}} 2x" alt="Description"/>
- * ```
- *
- * The correct way to write it:
- * ```html
- * <img ng-srcset="http://www.gravatar.com/avatar/{{hash}} 2x" alt="Description" />
- * ```
- *
- * @element IMG
- * @param {template} ngSrcset any string which can contain `{{}}` markup.
- */
-
-/**
- * @ngdoc directive
- * @name ngDisabled
- * @restrict A
- * @priority 100
- *
- * @description
- *
- * This directive sets the `disabled` attribute on the element (typically a form control,
- * e.g. `input`, `button`, `select` etc.) if the
- * {@link guide/expression expression} inside `ngDisabled` evaluates to truthy.
- *
- * A special directive is necessary because we cannot use interpolation inside the `disabled`
- * attribute. See the {@link guide/interpolation interpolation guide} for more info.
- *
- * @param {string} ngDisabled If the {@link guide/expression expression} is truthy,
- *     then the `disabled` attribute will be set on the element
- */
-
-/**
- * @ngdoc directive
- * @name ngChecked
- * @restrict A
- * @priority 100
- *
- * @description
- * Sets the `checked` attribute on the element, if the expression inside `ngChecked` is truthy.
- *
- * Note that this directive should not be used together with {@link ngModel `ngModel`},
- * as this can lead to unexpected behavior.
- *
- * A special directive is necessary because we cannot use interpolation inside the `checked`
- * attribute. See the {@link guide/interpolation interpolation guide} for more info.
- *
- * @element INPUT
- * @param {string} ngChecked If the {@link guide/expression expression} is truthy,
- *     then the `checked` attribute will be set on the element
- */
-
-/**
- *
- * @description
- *
- * Sets the `readonly` attribute on the element, if the expression inside `ngReadonly` is truthy.
- * Note that `readonly` applies only to `input` elements with specific types. [See the input docs on
- * MDN](https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input#attr-readonly) for more information.
- *
- * A special directive is necessary because we cannot use interpolation inside the `readonly`
- * attribute. See the {@link guide/interpolation interpolation guide} for more info.
- * @element INPUT
- * @param {string} ngReadonly If the {@link guide/expression expression} is truthy,
- *     then special attribute "readonly" will be set on the element
- */
-
-/**
- * @ngdoc directive
- * @name ngSelected
- * @restrict A
- * @priority 100
- *
- * @description
- *
- * Sets the `selected` attribute on the element, if the expression inside `ngSelected` is truthy.
- *
- * A special directive is necessary because we cannot use interpolation inside the `selected`
- * attribute. See the {@link guide/interpolation interpolation guide} for more info.
- *
- * <div class="alert alert-warning">
- *   **Note:** `ngSelected` does not interact with the `select` and `ngModel` directives, it only
- *   sets the `selected` attribute on the element. If you are using `ngModel` on the select, you
- *   should not use `ngSelected` on the options, as `ngModel` will set the select value and
- *   selected options.
- * </div>
- * @element OPTION
- * @param {string} ngSelected If the {@link guide/expression expression} is truthy,
- *     then special attribute "selected" will be set on the element
- */
-
-/**
- * @ngdoc directive
- * @name ngOpen
- * @restrict A
- * @priority 100
- *
- * @description
- *
- * Sets the `open` attribute on the element, if the expression inside `ngOpen` is truthy.
- *
- * A special directive is necessary because we cannot use interpolation inside the `open`
- * attribute. See the {@link guide/interpolation interpolation guide} for more info.
- *
- * ## A note about browser compatibility
- *
- * Internet Explorer and Edge do not support the `details` element, it is
- * recommended to use {@link ng.ngShow} and {@link ng.ngHide} instead.
- *
- * @element DETAILS
- * @param {string} ngOpen If the {@link guide/expression expression} is truthy,
- *     then special attribute "open" will be set on the element
- */
 
 export const ngAttributeAliasDirectives = {};
 
 // boolean attrs are evaluated
-forEach(BOOLEAN_ATTR, (propName, attrName) => {
+Object.entries(BOOLEAN_ATTR).forEach(([attrName, propName]) => {
   // binding to multiple is not supported
   if (propName === "multiple") return;
 
@@ -220,7 +39,7 @@ forEach(BOOLEAN_ATTR, (propName, attrName) => {
 });
 
 // aliased input attrs are evaluated
-forEach(ALIASED_ATTR, (htmlAttr, ngAttr) => {
+Object.entries(ALIASED_ATTR).forEach(([ngAttr]) => {
   ngAttributeAliasDirectives[ngAttr] = function () {
     return {
       priority: 100,
@@ -244,7 +63,7 @@ forEach(ALIASED_ATTR, (htmlAttr, ngAttr) => {
 });
 
 // ng-src, ng-srcset, ng-href are interpolated
-forEach(["src", "srcset", "href"], (attrName) => {
+["src", "srcset", "href"].forEach((attrName) => {
   const normalized = directiveNormalize(`ng-${attrName}`);
   ngAttributeAliasDirectives[normalized] = [
     "$sce",