@angular-wave/angular.ts 0.0.40 → 0.0.41

package/package.json

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

package/src/animations/animate-queue.js

@@ -1,4 +1,4 @@
-import { JQLite } from "../shared/jqlite/jqlite";
+import { getOrSetCacheData, JQLite } from "../shared/jqlite/jqlite";
 import {
   isUndefined,
   forEach,
@@ -757,7 +757,7 @@ export const $$AnimateQueueProvider = [
           let elementDisabled = disabledElementsLookup.get(node);
           let animateChildren;
 
-          let parentHost = JQLite.data(node, NG_ANIMATE_PIN_DATA);
+          let parentHost = getOrSetCacheData(node, NG_ANIMATE_PIN_DATA);
           if (parentHost) {
             parentNode = getDomNode(parentHost);
           }
@@ -794,7 +794,10 @@ export const $$AnimateQueueProvider = [
             }
 
             if (isUndefined(animateChildren) || animateChildren === true) {
-              const value = JQLite.data(parentNode, NG_ANIMATE_CHILDREN_DATA);
+              const value = getOrSetCacheData(
+                parentNode,
+                NG_ANIMATE_CHILDREN_DATA,
+              );
               if (isDefined(value)) {
                 animateChildren = value;
               }
@@ -817,7 +820,7 @@ export const $$AnimateQueueProvider = [
 
             if (!rootNodeDetected) {
               // If `rootNode` is not detected, check if `parentNode` is pinned to another element
-              parentHost = JQLite.data(parentNode, NG_ANIMATE_PIN_DATA);
+              parentHost = getOrSetCacheData(parentNode, NG_ANIMATE_PIN_DATA);
               if (parentHost) {
                 // The pin target element becomes the next parent element
                 parentNode = getDomNode(parentHost);

package/src/core/compile/compile.js

@@ -1,6 +1,8 @@
 import {
   JQLite,
+  cleanElementData,
   getBooleanAttrName,
+  getOrSetCacheData,
   isTextNode,
   startingTag,
 } from "../../shared/jqlite/jqlite";
@@ -3111,7 +3113,7 @@ export function $CompileProvider($provide, $$sanitizeUriProvider) {
           // Copy over user data (that includes AngularJS's $scope etc.). Don't copy private
           // data here because there's no public interface in jQuery to do that and copying over
           // event listeners (which is the main use of private data) wouldn't work anyway.
-          JQLite.data(newNode, JQLite.data(firstElementToRemove));
+          getOrSetCacheData(newNode, getOrSetCacheData(firstElementToRemove));
 
           // Remove $destroy event listeners from `firstElementToRemove`
           JQLite(firstElementToRemove).off("$destroy");
@@ -3119,7 +3121,7 @@ export function $CompileProvider($provide, $$sanitizeUriProvider) {
 
         // Cleanup any data/listeners on the elements and children.
         // This includes invoking the $destroy event on any elements with listeners.
-        JQLite.cleanData(fragment.querySelectorAll("*"));
+        cleanElementData(fragment.querySelectorAll("*"));
 
         // Update the JQLite collection to only contain the `newNode`
         for (i = 1; i < removeCount; i++) {

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

@@ -1,6 +1,6 @@
 import { publishExternalAPI } from "../../public";
 import { createInjector } from "../../injector";
-import { dealoc, JQLite } from "../../shared/jqlite/jqlite";
+import { dealoc, JQLite, getOrSetCacheData } from "../../shared/jqlite/jqlite";
 import {
   forEach,
   isFunction,
@@ -6416,47 +6416,6 @@ describe("$compile", () => {
           expect(element.text()).toEqual("3==3");
         });
 
-        describe("when directive is in a repeater", () => {
-          let is;
-          beforeEach(() => {
-            is = [1, 2];
-          });
-
-          function runTest() {
-            $templateCache.put(
-              "/mock/hello",
-              "<span>i=<span ng-transclude></span>;</span>",
-            );
-            element = JQLite(
-              `<div><b hello ng-repeat="i in [${is}]">{{i}}</b></div>`,
-            );
-            $compile(element)($rootScope);
-            $rootScope.$digest();
-            expect(element.text()).toEqual(`i=${is.join(";i=")};`);
-          }
-
-          it("should work with another library patching JQLite/jQuery.cleanData after Angular", () => {
-            let cleanedCount = 0;
-            const currentCleanData = JQLite.cleanData;
-            JQLite.cleanData = function (elems) {
-              cleanedCount += elems.length;
-              // Don't return the output and explicitly pass only the first parameter
-              // so that we're sure we're not relying on either of them. jQuery UI patch
-              // behaves in this way.
-              currentCleanData(elems);
-            };
-
-            runTest();
-
-            // The initial ng-repeat div is dumped after parsing hence we expect cleanData
-            // count to be one larger than size of the iterated array.
-            expect(cleanedCount).toBe(is.length + 1);
-
-            // Restore the previous cleanData.
-            JQLite.cleanData = currentCleanData;
-          });
-        });
-
         describe("replace and not exactly one root element", () => {
           beforeEach(() => {
             publishExternalAPI().decorator("$exceptionHandler", () => {
@@ -17331,7 +17290,7 @@ describe("$compile", () => {
 
       const preCompiledChildren = getAll(toCompile);
       forEach(preCompiledChildren, (element, i) => {
-        JQLite.data(element, "foo", `template#${i}`);
+        getOrSetCacheData(element, "foo", `template#${i}`);
       });
 
       const linkedElements = $compile(toCompile)($rootScope);

package/src/directive/include/include.js

@@ -1,5 +1,5 @@
 import { isDefined } from "../../shared/utils";
-import { JQLiteBuildFragment } from "../../shared/jqlite/jqlite";
+import { buildFragment } from "../../shared/jqlite/jqlite";
 
 export const ngIncludeDirective = [
   "$templateRequest",
@@ -11,12 +11,12 @@ export const ngIncludeDirective = [
     terminal: true,
     transclude: "element",
     controller: () => {},
-    compile(element, attr) {
+    compile(_element, attr) {
       const srcExp = attr.ngInclude || attr.src;
       const onloadExp = attr.onload || "";
       const autoScrollExp = attr.autoscroll;
 
-      return (scope, $element, $attr, ctrl, $transclude) => {
+      return (scope, $element, _$attr, ctrl, $transclude) => {
         let changeCounter = 0;
         let currentScope;
         let previousElement;
@@ -110,15 +110,13 @@ export const ngIncludeFillContentDirective = [
     restrict: "ECA",
     priority: -400,
     require: "ngInclude",
-    link(scope, $element, $attr, ctrl) {
+    link(scope, $element, _$attr, ctrl) {
       if (toString.call($element[0]).match(/SVG/)) {
         // WebKit: https://bugs.webkit.org/show_bug.cgi?id=135698 --- SVG elements do not
         // support innerHTML, so detect this here and try to generate the contents
         // specially.
         $element.empty();
-        $compile(
-          JQLiteBuildFragment(ctrl.template, window.document).childNodes,
-        )(
+        $compile(buildFragment(ctrl.template).childNodes)(
           scope,
           (clone) => {
             $element.append(clone);

package/src/directive/options/options.js

@@ -1,4 +1,4 @@
-import { JQLite, JQLiteRemove, startingTag } from "../../shared/jqlite/jqlite";
+import { JQLite, removeElement, startingTag } from "../../shared/jqlite/jqlite";
 import {
   equals,
   forEach,
@@ -594,9 +594,9 @@ export const ngOptionsDirective = [
           for (let i = options.items.length - 1; i >= 0; i--) {
             const option = options.items[i];
             if (isDefined(option.group)) {
-              JQLiteRemove(option.element.parentNode);
+              removeElement(option.element.parentNode);
             } else {
-              JQLiteRemove(option.element);
+              removeElement(option.element);
             }
           }
         }

package/src/services/http/http.js

@@ -219,13 +219,17 @@ function parseHeaders(headers) {
   }
 
   if (isString(headers)) {
-    forEach(headers.split("\n"), (line) => {
-      i = line.indexOf(":");
-      fillInParsed(
-        lowercase(trim(line.substr(0, i))),
-        trim(line.substr(i + 1)),
-      );
-    });
+    forEach(
+      headers.split("\n"),
+      /** @param {string} line */
+      (line) => {
+        i = line.indexOf(":");
+        fillInParsed(
+          line.substring(0, i).trim().toLowerCase(),
+          trim(line.substring(i + 1)),
+        );
+      },
+    );
   } else if (isObject(headers)) {
     forEach(headers, (headerVal, headerKey) => {
       fillInParsed(lowercase(headerKey), trim(headerVal));