[buddypress-trac] [BuddyPress Trac] #8021: BP Legacy - Export Data page - Erase Data section missing

buddypress-trac noreply at wordpress.org
Tue Sep 20 08:10:25 UTC 2022


#8021: BP Legacy - Export Data page - Erase Data section missing
--------------------------------------+---------------------
 Reporter:  Venutius                  |       Owner:  imath
     Type:  defect (bug)              |      Status:  closed
 Priority:  normal                    |   Milestone:  11.0.0
Component:  Templates                 |     Version:  4.0.0
 Severity:  normal                    |  Resolution:  fixed
 Keywords:  good-first-bug has-patch  |
--------------------------------------+---------------------
Changes (by imath):

 * owner:  (none) => imath
 * status:  new => closed
 * resolution:   => fixed


Comment:

 In [changeset:"13327" 13327]:
 {{{
 #!CommitTicketReference repository="" revision="13327"
 Settings: put the Legacy template pack’s in line with Nouveau’s one

 Makes sure, just like the BP Nouveau template pack, the BP Legacy’s
 `members/single/settings/data.php` template contains a section to inform
 how the user can completely remove their data. This commit also brings
 some complementary improvements:
 - Avoid `is_super_admin( $user_id )` usage in favor of a capability check.
 - Remove some commented code which was duplicated from a previous section
 into the legacy settings/data template.
 - Escape URLs and translated strings.
 - Improve `/* Translators: */ inline comment.
 - Do not show the link to delete an account to an administrator viewing
 their account as they are redirected to the member's home page when trying
 to reach the delete-account screen (Admins cannot delete their account
 form front end).

 Props Venutius, tahmidulkarim

 Closes https://github.com/buddypress/buddypress/pull/23
 Fixes #8021
 }}}

-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/8021#comment:8>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac


More information about the buddypress-trac mailing list