Merge pull request #7048 from yigitfindikli/multiselectvsblfnc

fixed #7047 Multiselect Headercheckbox not working correctly
primefaces · Dec 31, 2018 · 53d6b9a · 53d6b9a
2 parents ecca8c6 + 4a6edc8
commit 53d6b9a
Showing 1 changed file with 15 additions and 1 deletion.
diff --git a/src/app/components/multiselect/multiselect.ts b/src/app/components/multiselect/multiselect.ts
@@ -428,7 +428,7 @@ export class MultiSelect implements OnInit,AfterViewInit,AfterContentInit,AfterV
 
     isAllChecked() {
         if (this.filterValue && this.filterValue.trim().length) {
-            return this.value && this.visibleOptions&&this.visibleOptions.length && (this.value.length == this.visibleOptions.length);
+            return this.value && this.visibleOptions&&this.visibleOptions.length && this.isVisibleOptionsAllChecked();
         }
         else {
             let optionCount = this.getEnabledOptionCount();
@@ -437,6 +437,20 @@ export class MultiSelect implements OnInit,AfterViewInit,AfterContentInit,AfterV
         }
     }
 
+    isVisibleOptionsAllChecked() {
+        if (!this.visibleOptions) {
+            return false;
+        }
+        else {
+            for (let option of this.visibleOptions) {
+                if (!this.value.find(value => value == option.value)) {
+                    return false;
+                }
+            }
+            return true;
+        }
+    }
+
     getEnabledOptionCount(): number {
         if (this.options) {
             let count = 0;