[wp-trac] [WordPress Trac] #46895: Personal Data Export Report: A way to display the group count

[WordPress Trac]

#46895: Personal Data Export Report: A way to display the group count
-------------------------------------------------+-------------------------
 Reporter:  birgire                              |       Owner:
                                                 |  SergeyBiryukov
     Type:  enhancement                          |      Status:  reviewing
 Priority:  normal                               |   Milestone:  5.3
Component:  Privacy                              |     Version:  4.9.6
 Severity:  normal                               |  Resolution:
 Keywords:  has-screenshots has-patch has-unit-  |     Focuses:
  tests commit                                   |
-------------------------------------------------+-------------------------
Changes (by garrett-eclipse):

 * keywords:  has-screenshots has-patch has-unit-tests => has-screenshots
     has-patch has-unit-tests commit


Comment:

 Thanks @pputzer I've refreshed the patch
 [https://core.trac.wordpress.org/attachment/ticket/46895/46895.4.diff
 46895.4.diff] to use the count of items to toggle display over-relying on
 a boolean. This means plugins need no change as illustrated in example
 export
 [https://core.trac.wordpress.org/attachment/ticket/46895/index%204.html]
 which should multiple Comments, a single Media and multiple mails form WP
 Mail Logging as an example to illustrate that 3rd parties are supported
 without requiring a change.

 Moving this forward to `commit` as I've also addressed unit test issues
 and removed the now unneeded one that did a check on the boolean.

 @birgire any final thoughts, would love you to test/review.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/46895#comment:13>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform