Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #11885 - DataTable: Select/unselect all with selectionPageOnly="false" does not work properly #11911

Merged
merged 3 commits into from May 15, 2024

Conversation

Rapster
Copy link
Member

@Rapster Rapster commented May 11, 2024

Fix #11885

@Rapster
Copy link
Member Author

Rapster commented May 11, 2024

@melloware could it be that simple? Just want to make sure I'm not missing anything

@melloware
Copy link
Member

Ha let me test! It could be that simple!

@melloware melloware added the 🐞 defect Bug...Something isn't working label May 11, 2024
@Rapster Rapster marked this pull request as draft May 11, 2024 22:00
@Rapster
Copy link
Member Author

Rapster commented May 11, 2024

draft for now, I have doubts 😄

@Rapster Rapster marked this pull request as ready for review May 11, 2024 22:33
@christophs78
Copy link
Contributor

This one IMO would deserve a Integrationtest.

@melloware
Copy link
Member

@christophs78 i think we have some checkAll tests but my guess is we are testing this exact scenario. Maybe we can modify or enhance an existing test?

@christophs78
Copy link
Contributor

Yes, i think so too. I'll look in the evening.

@christophs78
Copy link
Contributor

DataTable006Test should be a good base to build upon. Enhance exisiting tests or add some new mutations.

@Rapster Rapster self-assigned this May 13, 2024
@melloware melloware merged commit b0a9ad8 into primefaces:master May 15, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 defect Bug...Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DataTable: Select/unselect all with selectionPageOnly="false" does not work properly
3 participants