fix(MultiSelect): Check for NaN values in isSelectionAllDisabled method

[dfernandez-elastic]

Fixes #16039

Added a check for NaN values.

Note:

I couldn't add new tests because my unit tests were not working locally.

When running npm test, I've got a lot of errors with this message: export 'async' (imported as 'async') was not found in '@angular/core/testing'

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment

Name	Status	Preview	Comments	Updated (UTC)
primeng	⬜️ Ignored (Inspect)	Visit Preview		Jul 18, 2024 0:55am

[rosenthalj]

Another way to fix this issue would be to add the following to multiselect.'s ngOnInit method:

        if(isNaN(this.selectionLimit)) {
            this.selectionLimit = undefined
        }

Note: ObjectUtils.isNotEmpty(this.selectionLimit) is call in multiple places throughout multiselect.ts:

[dfernandez-elastic]

Hi @rosenthalj I wanted to fix only the Toggle All behavior. I can implement it in the ngOnInit method too. Let's wait for a maintainer response.

Thanks for your suggestion!

[cetincakiroglu]

Could you please try the mentioned solution?