Opened 9 years ago
Closed 9 years ago
#5780 closed enhancement (no action required)
bp_edit_profile_button default arguments
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Milestone: | Priority: | low | |
| Severity: | minor | Version: | |
| Component: | Extended Profile | Keywords: | |
| Cc: |
Description
Hi,
I'm just working with this function to display the xprofile edit button and I can see that is used nowhere in BuddyPress. Since that I see that the block_self value is true where I suppose it should be false, right? Could be also useful to have the opposite, that would be shown just for the logged in user profile. What you think about this?
Thanks :)
Change History (9)
#3
@
9 years ago
bp_edit_profile_button() is old; one of the original xProfile functions. It was last used in BP 1.1.3 in the bp-sn-parent theme. Looking back at the current function and that theme, I think that it looks like the button generated by this function was intended to be used as a shortcut for site admins. Therefore I am a bit concerned about changing the block_self value just in case someone has painstakingly ported an original theme through all the releases since then (admittedly an edge case).
Personally I am split between not doing anything, or as you've just blown the dust off this function, marking it as deprecated. I don't think we would recommend anyone to use it in a modern theme.
#5
@
9 years ago
- Milestone changed from Awaiting Review to 2.2
- Priority changed from normal to low
- Severity changed from normal to minor
#9
@
9 years ago
- Milestone 2.2 deleted
- Resolution set to invalid
- Status changed from new to closed
Closing as invalid; it works as per my notes in https://buddypress.trac.wordpress.org/ticket/5780#comment:3
Also bp_button doesn't seem add the id defined in the id argument to the button href.