WP_Privacy_Request_Table should not show views that have a count of 0.

[pbiron]

Other core list tables that have a get_views() method do not output a view if the count for that view is 0, e.g., WP_Posts_List_Table doesn't output "Pending (0)" if there are no posts with $post_status === 'pending').

However, WP_Privacy_Requests_Table does output "Pending (0)" if there are no pending requests.

For consistency's sake, I think WP_Privacy_Requests_Table should skip views with count of 0.

[garrett-eclipse]

Comments list table displaying all status counts even with 0 items.

[garrett-eclipse]

Thanks @pbiron for the ticket and patch, applying and testing the patch works as outlined.

I am a bit hesitant of moving forward here as I see Requests being more in line with Comments than posts and the Comments list table does display the additional status' for Pending, Approved, Spam and Trash.

In my eyes displaying all status' is helpful to the user experience as it outlines the expected process visually by showing what status buckets the requests will need to pass through.

I'm marking for 2nd-opinion and also flagging to get some design feedback on the UX here. I'm all for consistency so if it's decided to suppress empty status labels then we should also address the Comments list table or potentially vice versa if displaying them is useful for to the user for the interface then resurfacing them on other list tables may be the avenue forward.

[davidbaumwald]

@pbiron Is this still in the cards for 5.5 with Beta 1 landing next week?

[pbiron]

@garrett-eclipse what do you think?

Personally, I still think views with 0 counts should not be shown. I'm pretty sure that of all the list tables in core, the privacy and comments list tables are the only ones that show views with 0 count. That is: posts, plugins, users, ms sites, ms themes, ms users (the later 3 in multisite) all "hide" views with 0 counts. The terms and links list tables don't support views (which they should, but that's another matter).

But, this is not really that big of a deal, so if the design feedback that has been requested doesn't come in time for this to land in 5.5, it can be punted.

[mapk]

The Design Team discussed this [in Slack today]​https://wordpress.slack.com/archives/C02S78ZAL/p1593708788256000. We're in agreement that the 0 count views should be hidden.

[pbiron]

Related: #50538

[SergeyBiryukov]

In 48284:

Privacy: Hide views that have a count of 0 on privacy request tables, for consistency with other list tables.

Props pbiron, garrett-eclipse, mapk, melchoyce, estelaris.
Fixes #47495.