[buddypress-trac] [BuddyPress Trac] #5197: Access Member Profiles from the Admin Backend
buddypress-trac
noreply at wordpress.org
Tue Jan 28 17:49:07 UTC 2014
#5197: Access Member Profiles from the Admin Backend
------------------------------------------+------------------
Reporter: svenl77 | Owner:
Type: enhancement | Status: new
Priority: high | Milestone: 2.0
Component: Members | Version:
Severity: normal | Resolution:
Keywords: has-patch 2nd-opinion commit |
------------------------------------------+------------------
Comment (by johnjamesjacoby):
Some general feedback, either for a new patch, or after commit:
* Missing escaping in several places.
* Would like to shy away from `dashicons` specific classes.
* New stats functions for each component should have `_get_` equivalents
with filters on the results.
* New stats functions should probably be more robust, so they can be
reused elsewhere.
* `profile_nav` method should be more extensible, allowing for third party
tabs.
* Allowed actions in `user_admin_load` and friends should be filterable.
* Should move the `bp_members_admin` and `xprofile_user_admin` actions
into a method so there are no global scope `is_admin()` checks.
* Thinking we should namespace all new functions with `bp_` even if it
does not match the existing conventions in some files.
--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/5197#comment:25>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac
More information about the buddypress-trac
mailing list