From 215df3cd35b31329629bb0dd191b84bf6d4e4d6f Mon Sep 17 00:00:00 2001 From: Moes Date: Thu, 8 Oct 2015 15:04:28 +1000 Subject: [PATCH] Update Knockout.d.ts update `objectForEach` to include key value add `addOrRemoveItem` consolidate `arrayPushAll` into one line using the `|` operator enabling `setTextContent` IT's PART OF THE MINIFIED API SURFACE https://github.com/knockout/knockout/blob/master/src/utils.js#L599 --- knockout/knockout.d.ts | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/knockout/knockout.d.ts b/knockout/knockout.d.ts index 0fbe8cbad3..e3e0cd49e8 100644 --- a/knockout/knockout.d.ts +++ b/knockout/knockout.d.ts @@ -275,9 +275,7 @@ interface KnockoutUtils { arrayFilter(array: T[], predicate: (item: T) => boolean): T[]; - arrayPushAll(array: T[], valuesToPush: T[]): T[]; - - arrayPushAll(array: KnockoutObservableArray, valuesToPush: T[]): T[]; + arrayPushAll(array: T[] | KnockoutObservableArray, valuesToPush: T[]): T[]; extend(target: Object, source: Object): Object; @@ -313,8 +311,8 @@ interface KnockoutUtils { toggleDomNodeCssClass(node: any, className: string, shouldHaveClass: boolean): void; - //setTextContent(element: any, textContent: string): void; // NOT PART OF THE MINIFIED API SURFACE (ONLY IN knockout-{version}.debug.js) https://github.com/SteveSanderson/knockout/issues/670 - + setTextContent(element: any, textContent: string | KnockoutObservable): void; // IT's PART OF THE MINIFIED API SURFACE https://github.com/knockout/knockout/blob/master/src/utils.js#L599 + setElementName(element: any, name: string): void; forceRefresh(node: any): void; @@ -339,9 +337,9 @@ interface KnockoutUtils { isIe7: boolean; - objectForEach(obj: any, action: Function): void; - - setPrototypeOf(obj: any, proto: any): void; + objectForEach(obj: any, action: (key: any, value: any) => void): void; + + addOrRemoveItem(array: T[] | KnockoutObservable, value: T, included: T): void; } interface KnockoutArrayChange {