@angular-wave/angular.ts 0.0.39 → 0.0.41

package/src/core/animate/animate.spec.js

@@ -22,10 +22,10 @@ xdescribe("$animate", () => {
 
     it("should enter the element to the start of the parent container", () => {
       for (let i = 0; i < 5; i++) {
-        element.append(jqLite(`<div> ${i}</div>`));
+        element.append(JQLite(`<div> ${i}</div>`));
       }
 
-      const child = jqLite("<div>first</div>");
+      const child = JQLite("<div>first</div>");
       $animate.enter(child, element);
 
       expect(element.text()).toEqual("first 0 1 2 3 4");
@@ -70,9 +70,9 @@ xdescribe("$animate", () => {
     });
 
     it("should run each method and return a promise", () => {
-      const element = jqLite("<div></div>");
-      const move = jqLite("<div></div>");
-      const parent = jqLite($document[0].body);
+      const element = JQLite("<div></div>");
+      const move = JQLite("<div></div>");
+      const parent = JQLite($document[0].body);
       parent.append(move);
 
       expect($animate.enter(element, parent)).toBeAPromise();
@@ -95,7 +95,7 @@ xdescribe("$animate", () => {
 
     it("should add and remove classes on SVG elements", () => {
       if (!window.SVGElement) return;
-      const svg = jqLite("<svg><rect></rect></svg>");
+      const svg = JQLite("<svg><rect></rect></svg>");
       const rect = svg.children();
       $animate.enabled(false);
       expect(rect).toBeShown();
@@ -137,9 +137,9 @@ xdescribe("$animate", () => {
     });
 
     it("should apply and retain inline styles on the element that is animated", () => {
-      const element = jqLite("<div></div>");
-      const parent = jqLite("<div></div>");
-      const other = jqLite("<div></div>");
+      const element = JQLite("<div></div>");
+      const parent = JQLite("<div></div>");
+      const other = JQLite("<div></div>");
       parent.append(other);
       $animate.enabled(true);
 
@@ -183,7 +183,7 @@ xdescribe("$animate", () => {
     });
 
     it("should merge the from and to styles that are provided", () => {
-      const element = jqLite("<div></div>");
+      const element = JQLite("<div></div>");
 
       element[0].style.color = "red";
       $animate.addClass(element, "on", {
@@ -198,7 +198,7 @@ xdescribe("$animate", () => {
     });
 
     it("should avoid cancelling out add/remove when the element already contains the class", () => {
-      const element = jqLite('<div class="ng-hide"></div>');
+      const element = JQLite('<div class="ng-hide"></div>');
 
       $animate.addClass(element, "ng-hide");
       $animate.removeClass(element, "ng-hide");
@@ -208,7 +208,7 @@ xdescribe("$animate", () => {
     });
 
     it("should avoid cancelling out remove/add if the element does not contain the class", () => {
-      const element = jqLite("<div></div>");
+      const element = JQLite("<div></div>");
 
       $animate.removeClass(element, "ng-hide");
       $animate.addClass(element, "ng-hide");
@@ -219,7 +219,7 @@ xdescribe("$animate", () => {
 
     ["enter", "move"].forEach((method) => {
       it('should accept an unwrapped "parent" element for the $prop event', () => {
-        const element = jqLite("<div></div>");
+        const element = JQLite("<div></div>");
         const parent = $document[0].createElement("div");
         $rootElement.append(parent);
 
@@ -230,7 +230,7 @@ xdescribe("$animate", () => {
 
     ["enter", "move"].forEach((method) => {
       it('should accept an unwrapped "after" element for the $prop event', () => {
-        const element = jqLite("<div></div>");
+        const element = JQLite("<div></div>");
         const after = $document[0].createElement("div");
         $rootElement.append(after);
 
@@ -257,8 +257,8 @@ xdescribe("$animate", () => {
           });
         });
 
-        const element = jqLite("<div></div>");
-        const parent2 = jqLite("<div></div>");
+        const element = JQLite("<div></div>");
+        const parent2 = JQLite("<div></div>");
         const parent = $rootElement;
         parent.append(parent2);
 
@@ -327,7 +327,7 @@ xdescribe("$animate", () => {
     () => {
       const spy = spyOn(window, "jqLiteAddClass").and.callThrough();
 
-      const element = jqLite("<div></div>");
+      const element = JQLite("<div></div>");
       const parent = $rootElement;
 
       $animate.enter(element, parent, null, { addClass: () => {} });
@@ -345,8 +345,8 @@ xdescribe("$animate", () => {
   });
 
   it("should not break postDigest for subsequent elements if addClass contains non-valid CSS class names", () => {
-    const element1 = jqLite("<div></div>");
-    const element2 = jqLite("<div></div>");
+    const element1 = JQLite("<div></div>");
+    const element2 = JQLite("<div></div>");
 
     $animate.enter(element1, $rootElement, null, { addClass: " " });
     $animate.enter(element2, $rootElement, null, { addClass: "valid-name" });
@@ -361,7 +361,7 @@ xdescribe("$animate", () => {
     () => {
       const spy = spyOn(window, "jqLiteRemoveClass").and.callThrough();
 
-      const element = jqLite("<div></div>");
+      const element = JQLite("<div></div>");
       const parent = $rootElement;
 
       $animate.enter(element, parent, null, { removeClass: () => {} });
@@ -380,7 +380,7 @@ xdescribe("$animate", () => {
   });
 
   it("should not alter the provided options input in any way throughout the animation", () => {
-    const element = jqLite("<div></div>");
+    const element = JQLite("<div></div>");
     const parent = $rootElement;
 
     const initialOptions = {
@@ -418,7 +418,7 @@ xdescribe("$animate", () => {
 
     function setupClassManipulationLogger(log) {
       () => {
-        const _addClass = jqLiteAddClass;
+        const _addClass = JQLiteAddClass;
         addClass = spyOn(window, "jqLiteAddClass").and.callFake(
           (element, classes) => {
             let names = classes;
@@ -429,7 +429,7 @@ xdescribe("$animate", () => {
           },
         );
 
-        const _removeClass = jqLiteRemoveClass;
+        const _removeClass = JQLiteRemoveClass;
         removeClass = spyOn(window, "jqLiteRemoveClass").and.callFake(
           (element, classes) => {
             let names = classes;
@@ -444,7 +444,7 @@ xdescribe("$animate", () => {
 
     it("should defer class manipulation until end of digest", () => {
       setupClassManipulationLogger(log);
-      element = jqLite("<p>test</p>");
+      element = JQLite("<p>test</p>");
 
       $rootScope.$apply(() => {
         $animate.addClass(element, "test-class1");
@@ -472,7 +472,7 @@ xdescribe("$animate", () => {
 
     it("should defer class manipulation until postDigest when outside of digest", () => {
       setupClassManipulationLogger(log);
-      element = jqLite('<p class="test-class4">test</p>');
+      element = JQLite('<p class="test-class4">test</p>');
 
       $animate.addClass(element, "test-class1");
       $animate.removeClass(element, "test-class1");
@@ -494,7 +494,7 @@ xdescribe("$animate", () => {
     });
 
     it("should perform class manipulation in expected order at end of digest", () => {
-      element = jqLite('<p class="test-class3">test</p>');
+      element = JQLite('<p class="test-class3">test</p>');
 
       setupClassManipulationLogger(log);
 
@@ -509,7 +509,7 @@ xdescribe("$animate", () => {
     });
 
     it("should return a promise which is resolved on a different turn", () => {
-      element = jqLite('<p class="test2">test</p>');
+      element = JQLite('<p class="test2">test</p>');
 
       $animate.addClass(element, "test1").then(log.fn("addClass(test1)"));
       $animate.removeClass(element, "test2").then(log.fn("removeClass(test2)"));
@@ -521,7 +521,7 @@ xdescribe("$animate", () => {
       expect(log).toEqual(["addClass(test1)", "removeClass(test2)"]);
 
       log.reset();
-      element = jqLite('<p class="test4">test</p>');
+      element = JQLite('<p class="test4">test</p>');
 
       $rootScope.$apply(() => {
         $animate.addClass(element, "test3").then(log.fn("addClass(test3)"));
@@ -538,7 +538,7 @@ xdescribe("$animate", () => {
     it("should defer class manipulation until end of digest for SVG", () => {
       if (!window.SVGElement) return;
       setupClassManipulationSpies();
-      element = jqLite("<svg><g></g></svg>");
+      element = JQLite("<svg><g></g></svg>");
       const target = element.children().eq(0);
 
       $rootScope.$apply(() => {
@@ -564,7 +564,7 @@ xdescribe("$animate", () => {
     it("should defer class manipulation until postDigest when outside of digest for SVG", () => {
       if (!window.SVGElement) return;
       setupClassManipulationLogger(log);
-      element = jqLite('<svg><g class="test-class4"></g></svg>');
+      element = JQLite('<svg><g class="test-class4"></g></svg>');
       const target = element.children().eq(0);
 
       $animate.addClass(target, "test-class1");
@@ -588,7 +588,7 @@ xdescribe("$animate", () => {
 
     it("should perform class manipulation in expected order at end of digest for SVG", () => {
       if (!window.SVGElement) return;
-      element = jqLite('<svg><g class="test-class3"></g></svg>');
+      element = JQLite('<svg><g class="test-class3"></g></svg>');
       const target = element.children().eq(0);
 
       setupClassManipulationLogger(log);

package/src/core/cache/cache-factory.html

@@ -0,0 +1,18 @@
+<!doctype html>
+<html>
+  <head>
+    <meta charset="utf-8" />
+    <title>AngularTS Test Runner</title>
+
+    <link rel="shortcut icon" type="image/png" href="/images/favicon.ico" />
+    <link rel="stylesheet" href="/jasmine/jasmine-5.1.2/jasmine.css" />
+    <script src="/jasmine/jasmine-5.1.2/jasmine.js"></script>
+    <script src="/jasmine/jasmine-5.1.2/jasmine-html.js"></script>
+    <script src="/jasmine/jasmine-5.1.2/boot0.js"></script>
+    <script src="/jasmine/jasmine-5.1.2/boot1.js"></script>
+    <script type="module" src="/src/core/cache/cache-factory.spec.js"></script>
+  </head>
+  <body>
+    <div id="dummy"></div>
+  </body>
+</html>

package/src/core/cache/{cache-factor.test.js → cache-factory.test.js}

@@ -1,6 +1,6 @@
 import { test, expect } from "@playwright/test";
 
-const TEST_URL = "src/core/cache/cache-factor.html";
+const TEST_URL = "src/core/cache/cache-factory.html";
 
 test("unit tests contain no errors", async ({ page }) => {
   await page.goto(TEST_URL);

package/src/core/compile/compile.js

@@ -1,6 +1,8 @@
 import {
-  jqLite,
+  JQLite,
+  cleanElementData,
   getBooleanAttrName,
+  getOrSetCacheData,
   isTextNode,
   startingTag,
 } from "../../shared/jqlite/jqlite";
@@ -229,6 +231,12 @@ export function $CompileProvider($provide, $$sanitizeUriProvider) {
         $provide.factory(name + Suffix, [
           "$injector",
           "$exceptionHandler",
+          /**
+           *
+           * @param {*} $injector
+           * @param {import('../exception-handler').angular.ErrorHandler} $exceptionHandler
+           * @returns
+           */
           function ($injector, $exceptionHandler) {
             const directives = [];
             forEach(hasDirectives[name], (directiveFactory, index) => {
@@ -1001,11 +1009,11 @@ export function $CompileProvider($provide, $$sanitizeUriProvider) {
         ignoreDirective,
         previousCompileContext,
       ) {
-        if (!($compileNodes instanceof jqLite)) {
+        if (!($compileNodes instanceof JQLite)) {
           // jquery always rewraps, whereas we need to preserve the original selector so that we can
           // modify it.
 
-          $compileNodes = jqLite($compileNodes);
+          $compileNodes = JQLite($compileNodes);
         }
 
         /**
@@ -1066,14 +1074,14 @@ export function $CompileProvider($provide, $$sanitizeUriProvider) {
             // might change, so we need to recreate the namespace adapted compileNodes
             // for call to the link function.
             // Note: This will already clone the nodes...
-            $linkNode = jqLite(
+            $linkNode = JQLite(
               wrapTemplate(
                 namespace,
-                jqLite("<div></div>").append($compileNodes).html(),
+                JQLite("<div></div>").append($compileNodes).html(),
               ),
             );
           } else if (cloneConnectFn) {
-            $linkNode = jqLite(
+            $linkNode = JQLite(
               Array.from($compileNodes).map((element) =>
                 element.cloneNode(true),
               ),
@@ -1128,8 +1136,8 @@ export function $CompileProvider($provide, $$sanitizeUriProvider) {
        * @param {function(ng.IScope, cloneAttachFn=)} transcludeFn A linking function, where the
        *        scope argument is auto-generated to the new child of the transcluded parent scope.
        * @param {Element=} $rootElement If the nodeList is the root of the compilation tree then
-       *        the rootElement must be set the jqLite collection of the compile root. This is
-       *        needed so that the jqLite collection items can be replaced with widgets.
+       *        the rootElement must be set the JQLite collection of the compile root. This is
+       *        needed so that the JQLite collection items can be replaced with widgets.
        * @param {number=} maxPriority Max directive priority.
        * @returns {Function} A composite linking function of all of the matched directives or null.
        */
@@ -1510,7 +1518,7 @@ export function $CompileProvider($provide, $$sanitizeUriProvider) {
           nodes.push(node);
         }
 
-        return jqLite(nodes);
+        return JQLite(nodes);
       }
 
       /**
@@ -1595,7 +1603,7 @@ export function $CompileProvider($provide, $$sanitizeUriProvider) {
        *                                                  scope argument is auto-generated to the new
        *                                                  child of the transcluded parent scope.
        * @param {JQLite} jqCollection If we are working on the root of the compile tree then this
-       *                              argument has the root jqLite array so that we can replace nodes
+       *                              argument has the root JQLite array so that we can replace nodes
        *                              on it.
        * @param {Object=} originalReplaceDirective An optional directive that will be ignored when
        *                                           compiling the transclusion.
@@ -1627,7 +1635,7 @@ export function $CompileProvider($provide, $$sanitizeUriProvider) {
         let hasTranscludeDirective = false;
         let hasTemplate = false;
         let { hasElementTranscludeDirective } = previousCompileContext;
-        let $compileNode = (templateAttrs.$$element = jqLite(compileNode));
+        let $compileNode = (templateAttrs.$$element = JQLite(compileNode));
         let directive;
         let directiveName;
         let $template;
@@ -1755,7 +1763,7 @@ export function $CompileProvider($provide, $$sanitizeUriProvider) {
               hasElementTranscludeDirective = true;
               terminalPriority = directive.priority;
               $template = $compileNode;
-              $compileNode = templateAttrs.$$element = jqLite(
+              $compileNode = templateAttrs.$$element = JQLite(
                 document.createComment(""),
               );
               compileNode = $compileNode[0];
@@ -1813,7 +1821,7 @@ export function $CompileProvider($provide, $$sanitizeUriProvider) {
 
                 // Add the matching elements into their slot
 
-                forEach(jqLite($compileNode[0].childNodes), (node) => {
+                forEach(JQLite($compileNode[0].childNodes), (node) => {
                   const slotName = slotMap[directiveNormalize(nodeName_(node))];
                   if (slotName) {
                     filledSlots[slotName] = true;
@@ -1840,7 +1848,7 @@ export function $CompileProvider($provide, $$sanitizeUriProvider) {
                 for (const slotName in slots) {
                   if (slots[slotName]) {
                     // Only define a transclusion function if the slot was filled
-                    const slotCompileNodes = jqLite(slots[slotName].childNodes);
+                    const slotCompileNodes = JQLite(slots[slotName].childNodes);
                     slots[slotName] = compilationGenerator(
                       mightHaveMultipleTransclusionError,
                       slotCompileNodes,
@@ -1849,7 +1857,7 @@ export function $CompileProvider($provide, $$sanitizeUriProvider) {
                   }
                 }
 
-                $template = jqLite($template.childNodes);
+                $template = JQLite($template.childNodes);
               }
 
               $compileNode.empty(); // clear contents
@@ -2076,7 +2084,7 @@ export function $CompileProvider($provide, $$sanitizeUriProvider) {
             attrs = templateAttrs;
             $element = templateAttrs.$$element;
           } else {
-            $element = jqLite(linkNode);
+            $element = JQLite(linkNode);
             attrs = new Attributes($element, templateAttrs);
           }
 
@@ -2714,7 +2722,7 @@ export function $CompileProvider($provide, $$sanitizeUriProvider) {
                 }
                 replaceWith(
                   linkRootElement,
-                  jqLite(beforeTemplateLinkNode),
+                  JQLite(beforeTemplateLinkNode),
                   linkNode,
                 );
 
@@ -3044,12 +3052,12 @@ export function $CompileProvider($provide, $$sanitizeUriProvider) {
       }
 
       /**
-       * This is a special jqLite.replaceWith, which can replace items which
-       * have no parents, provided that the containing jqLite collection is provided.
+       * This is a special JQLite.replaceWith, which can replace items which
+       * have no parents, provided that the containing JQLite collection is provided.
        *
        * @param {JQLite} $rootElement The root of the compile tree. Used so that we can replace nodes
        *                               in the root of the tree.
-       * @param {JQLite} elementsToRemove The jqLite element which we are going to replace. We keep
+       * @param {JQLite} elementsToRemove The JQLite element which we are going to replace. We keep
        *                                  the shell, but replace its DOM node reference.
        * @param {Node} newNode The new DOM node.
        */
@@ -3105,17 +3113,17 @@ 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");
+          JQLite(firstElementToRemove).off("$destroy");
         }
 
         // 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`
+        // Update the JQLite collection to only contain the `newNode`
         for (i = 1; i < removeCount; i++) {
           delete elementsToRemove[i];
         }
@@ -3402,7 +3410,7 @@ function tokenDifference(str1, str2) {
 }
 
 function removeComments(jqNodes) {
-  jqNodes = jqLite(jqNodes);
+  jqNodes = JQLite(jqNodes);
   let i = jqNodes.length;
 
   if (i <= 1) {

package/src/core/compile/compile.md

@@ -5,7 +5,7 @@
 -
 - - "node" - DOM Node
 - - "element" - DOM Element or Node
-- - "$node" or "$element" - jqLite-wrapped node or element
+- - "$node" or "$element" - JQLite-wrapped node or element
 -
 -
 - Compiler related stuff:
@@ -728,7 +728,7 @@
 - </div>
 -
 - If you want to manually control the insertion and removal of the transcluded content in your directive
-- then you must use this transclude function. When you call a transclude function it returns a jqLite/JQuery
+- then you must use this transclude function. When you call a transclude function it returns a JQLite/JQuery
 - object that contains the compiled DOM, which is linked to the correct transclusion scope.
 -
 - When you call a transclusion function you can pass in a **clone attach function**. This function accepts

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/core/cookie-reader.spec.js

@@ -1,5 +1,5 @@
-import { publishExternalAPI } from "../../public";
-import { createInjector } from "../../injector";
+import { publishExternalAPI } from "../public";
+import { createInjector } from "../injector";
 
 describe("$$cookieReader", () => {
   let $$cookieReader;

package/src/core/core.html

@@ -0,0 +1,22 @@
+<!doctype html>
+<html>
+  <head>
+    <meta charset="utf-8" />
+    <title>AngularTS Test Runner</title>
+
+    <link rel="shortcut icon" type="image/png" href="/images/favicon.ico" />
+    <link rel="stylesheet" href="/jasmine/jasmine-5.1.2/jasmine.css" />
+    <script src="/jasmine/jasmine-5.1.2/jasmine.js"></script>
+    <script src="/jasmine/jasmine-5.1.2/jasmine-html.js"></script>
+    <script src="/jasmine/jasmine-5.1.2/boot0.js"></script>
+    <script src="/jasmine/jasmine-5.1.2/boot1.js"></script>
+    <script type="module" src="/src/core/cookie-reader.spec.js"></script>
+    <script type="module" src="/src/core/document.spec.js"></script>
+    <script type="module" src="/src/core/on.spec.js"></script>
+    <script type="module" src="/src/core/prop.spec.js"></script>
+    <script type="module" src="/src/core/root-element.spec.js"></script>
+  </head>
+  <body>
+    <div id="dummy"></div>
+  </body>
+</html>

package/src/core/core.test.js

@@ -0,0 +1,12 @@
+import { test, expect } from "@playwright/test";
+
+const TEST_URL = "src/core/core.html";
+
+test("unit tests contain no errors", async ({ page }) => {
+  await page.goto(TEST_URL);
+  await page.content();
+  await page.waitForTimeout(100);
+  await expect(page.locator(".jasmine-overall-result")).toHaveText(
+    /0 failures/,
+  );
+});

package/src/core/document.spec.js

@@ -1,6 +1,6 @@
-import { publishExternalAPI } from "../../public";
-import { createInjector } from "../../injector";
-import { jqLite } from "../../shared/jqlite/jqlite";
+import { publishExternalAPI } from "../public";
+import { createInjector } from "../injector";
+import { JQLite } from "../shared/jqlite/jqlite";
 
 describe("$document", () => {
   let $document, $httpBackend, $http, $$isDocumentHidden;
@@ -15,7 +15,7 @@ describe("$document", () => {
   });
 
   it("should inject $document", () => {
-    expect($document).toEqual(jqLite(window.document));
+    expect($document).toEqual(JQLite(window.document));
   });
 });
 

package/src/core/exception-handler.js

@@ -1,9 +1,5 @@
-/* eslint-disable no-use-before-define */
 /**
- * @ngdoc service
  * @name $exceptionHandler
- * @requires ng.$log
- *
  *
  * @description
  * Any uncaught exception in AngularJS expressions is delegated to this service.
@@ -29,35 +25,46 @@
  * ```
  *
  * <hr />
- * Note, that code executed in event-listeners (even those registered using jqLite's `on`/`bind`
- * methods) does not delegate exceptions to the {@link ng.$exceptionHandler $exceptionHandler}
+ * Note, that code executed in event-listeners (even those registered using JQLite's `on`/`bind`
+ * methods) does not delegate exceptions to the {@link angular.ErrorHandler }
  * (unless executed during a digest).
  *
  * If you wish, you can manually delegate exceptions, e.g.
  * `try { ... } catch(e) { $exceptionHandler(e); }`
  *
- * @param {Error} exception Exception associated with the error.
- * @param {string=} cause Optional information about the context in which
- *       the error was thrown.
- *
+ */
+
+/**
+ * @typedef {import('../index').angular.ServiceProvider} angular.ExceptionHandlerProvider
+ */
+
+/** @type {import('../services/log').angular.LogService} */
+let log;
+
+/**
+ * @callback angular.ErrorHandler
+ * @param {Error} exception - Exception associated with the error.
+ * @param {string} [cause] - Optional information about the context in which the error was thrown.
+ * @returns {void}
+ */
+export const errorHandler = (exception, cause) => {
+  log.error(exception, cause);
+};
+
+/**
+ * @constructor
+ * @this {angular.ExceptionHandlerProvider}
  */
 export function $ExceptionHandlerProvider() {
   this.$get = [
     "$log",
+    /**
+     * @param {import('../services/log').angular.LogService} $log
+     * @returns {angular.ErrorHandler}
+     */
     function ($log) {
-      return function () {
-        $log.error.apply($log, arguments);
-      };
+      log = $log;
+      return errorHandler;
     },
   ];
 }
-
-// let $log;
-
-// export function ExceptionHandler(log) {
-//   $log = log || window.console;
-
-//   return function(exception, cause) {
-//     $log.error.apply($log, arguments);
-//   };
-// }