Revert "primefaces#14223 fix for regression value can be '' or null"

This reverts commit fa44251.
pete-mcwilliams · Nov 28, 2023 · 79640af · 79640af
1 parent fa44251
commit 79640af
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/app/components/dropdown/dropdown.ts b/src/app/components/dropdown/dropdown.ts
@@ -858,7 +858,7 @@ export class Dropdown implements OnInit, AfterViewInit, AfterContentInit, AfterV
     }
 
     get isVisibleClearIcon(): boolean | undefined {
-        return this.modelValue() != null && this.hasSelectedOption() && this.showClear && !this.disabled;
+        return this.modelValue() != null && ObjectUtils.isNotEmpty(this.modelValue()) && this.modelValue() !== '' && this.showClear && !this.disabled;
     }
 
     get containerClass() {
@@ -1459,7 +1459,7 @@ export class Dropdown implements OnInit, AfterViewInit, AfterContentInit, AfterV
     }
 
     hasSelectedOption() {
-        return this.modelValue() !== undefined;
+        return ObjectUtils.isNotEmpty(this.modelValue());
     }
 
     isValidSelectedOption(option) {