[buddypress-trac] [BuddyPress] #3695: Basic profile privacy
buddypress-trac at lists.automattic.com
buddypress-trac at lists.automattic.com
Wed Nov 2 15:09:19 UTC 2011
#3695: Basic profile privacy
-----------------------------------------------+---------------------------
Reporter: boonebgorges | Owner: cnorris23
Type: enhancement | Status: accepted
Priority: normal | Milestone: Future
Component: Core | Release
Severity: normal | Version:
Keywords: needs-patch dev-feedback needs-ui | Resolution:
-----------------------------------------------+---------------------------
Comment (by boonebgorges):
We had some discussions about this in last week's dev chat. Some
takeaways:
- My suggestion (1) above is really a necessity. The patch currently
requires another database hit for every profile field, which will not
scale well.
- Paul's main concern (which I share) is that we are compatible with any
future move that WP might make to using WP caps/current_user_can(). My
understanding of WP caps is incomplete (to say the least), but from what I
do get, you can filter map_meta_cap with whatever data you want, which
means that your suggested method should work fine. But in any case, before
considering for inclusion in BP core, I think it's crucial that we either
do a proof-of-concept to show that we are able to do the necessary mapping
down the road, *or* that we actually rewrite cnorris23's patch to use
current_user_can(). In this way, this privacy stuff could be a testing
ground for the future migration of BP caps (which are currently all over
the place) to a more organized WP caps schema
- It probably makes sense to plugin-first with this, since time is getting
short on this dev cycle. It should be fairly easy to wrap into a plugin
shell. If you need any filters in xprofile to make it possible for such a
plugin to work with 1.6, please don't hesitate to ask.
If I get a chance myself, I can start repatching in accordance with these
notes, though I'm pretty booked for the very near future.
--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/3695#comment:7>
BuddyPress <http://buddypress.org/>
BuddyPress
More information about the buddypress-trac
mailing list