Refactor

primefaces · Nov 17, 2023 · ee2b462 · ee2b462
1 parent b4028d9
commit ee2b462
Show file tree

Hide file tree

Showing 6 changed files with 2 additions and 14 deletions.
diff --git a/src/app/components/listbox/listbox.interface.ts b/src/app/components/listbox/listbox.interface.ts
@@ -44,10 +44,6 @@ export interface ListboxSelectAllChangeEvent {
      * Boolean value indicates whether all data is selected.
      */
     checked: boolean;
-    /**
-     * Method to invoke on model value change.
-     */
-    updateModel?: (value?: any, event?: Event) => void;
 }
 /**
  * Custom filter event.

diff --git a/src/app/components/listbox/listbox.ts b/src/app/components/listbox/listbox.ts
@@ -865,8 +865,7 @@ export class Listbox implements AfterContentInit, OnInit, ControlValueAccessor,
         if(this.selectAll !== null) {
             this.onSelectAllChange.emit({
                 originalEvent: event,
-                checked: !this.allSelected(),
-                updateModel: this.updateModel.bind(this)
+                checked: !this.allSelected()
             })
         } else {
             const value = this.allSelected()

diff --git a/src/app/components/multiselect/multiselect.interface.ts b/src/app/components/multiselect/multiselect.interface.ts
@@ -42,10 +42,6 @@ export interface MultiSelectSelectAllChangeEvent {
      * Boolean value indicates whether all data is selected.
      */
     checked: boolean;
-    /**
-     * Method to invoke on model value change.
-     */
-    updateModel?: (value?: any, event?: Event) => void;
 }
 /**
  * Custom filter event.

diff --git a/src/app/components/multiselect/multiselect.ts b/src/app/components/multiselect/multiselect.ts
@@ -1770,8 +1770,7 @@ export class MultiSelect implements OnInit, AfterViewInit, AfterContentInit, Aft
         if(this.selectAll !== null) {
             this.onSelectAllChange.emit({
                 originalEvent: event,
-                checked: !this.allSelected(),
-                updateModel: this.updateModel.bind(this)
+                checked: !this.allSelected()
             })
         } else {
             const value = this.allSelected()

diff --git a/src/app/showcase/doc/listbox/virtualscrolldoc.ts b/src/app/showcase/doc/listbox/virtualscrolldoc.ts
@@ -47,7 +47,6 @@ export class VirtualScrollDoc {
     onSelectAllChange(event) {
         this.selectedItems = event.checked ? [...this.items] : [];
         this.selectAll = event.checked;
-        event.updateModel(this.selectedItems, event.originalEvent);
     }
 
     onChange(event) {

diff --git a/src/app/showcase/doc/multiselect/virtualscrolldoc.ts b/src/app/showcase/doc/multiselect/virtualscrolldoc.ts
@@ -43,7 +43,6 @@ export class VirtualScrollDoc {
     onSelectAllChange(event) {
         this.selectedItems = event.checked ? [...this.items] : [];
         this.selectAll = event.checked;
-        event.updateModel(this.selectedItems, event.originalEvent);
     }
 
     onChange(event) {