[buddypress-trac] [BuddyPress] #4482: Better member type support in bp_group_has_members()
buddypress-trac
noreply at wordpress.org
Sun May 19 23:34:40 UTC 2013
#4482: Better member type support in bp_group_has_members()
--------------------------+---------------------------
Reporter: boonebgorges | Owner: boonebgorges
Type: enhancement | Status: assigned
Priority: high | Milestone: 1.8
Component: Groups | Version:
Severity: major | Resolution:
Keywords: needs-patch |
--------------------------+---------------------------
Comment (by trishasalas):
My comments won't have too much to do with your method, looks as elegant
as possible to me.
I knew the join was not a good way to handle the situation but what you
are seeing with my code is:
1) My lack of familiarity with BuddyPress (if I grab the users from group
members how do I know who they are from just their id? is that in
BP_User_query? I thought it might be but wasn't sure.
2) My lack of php coding experience. In retrospect I probably should have
said as much but I wanted to try.
The idealist in my loves the use of OOP techniques, I would like to see
WordPress head that direction but understand and really trust the devs
working on that code base.
Once the dust settles on the push for 1.8 I would value some direction as
to how to get my coding skills where they need to be to be helpful in
situations like this. I am doing as much as I can find as far as online
classes, etc. The problem with php is that there is so much out there and
not all of it is good.
All that being said. I like the idea of uniform queries throughout
BuddyPress and I would love to do some documentation in the codex on that.
I know it's in the code but I can definitely see the value of getting it
out there.
--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/4482#comment:14>
BuddyPress <http://buddypress.org/>
BuddyPress
More information about the buddypress-trac
mailing list