#6442 closed defect (bug) (fixed)
Undefined function used in bp_group_avatar_edit_form()
Reported by: | tw2113 | Owned by: | djpaul |
---|---|---|---|
Milestone: | 2.4 | Priority: | normal |
Severity: | normal | Version: | 1.0 |
Component: | Groups | Keywords: | needs-patch good-first-bug early |
Cc: |
Description
function bp_group_avatar_edit_form() { groups_avatar_upload(); }
No found location of a defined groups_avatar_upload function
Change History (4)
#2
@
10 years ago
- Keywords needs-patch good-first-bug early added
- Milestone changed from Awaiting Review to 2.4
To add to above comments, groups_avatar_upload
was removed in the 1.2 release. No-one seems to have reported it missing in the intervening years, and there was a ton of stuff changed between 1.1 and 1.2.
Let's deprecate this function and stick in our backwards compatibility files, as it doesn't cost anything, and who knows - might even fix a fatal error if someone tries to upgrade a 1.1 site. :)
#3
@
9 years ago
- Owner set to djpaul
- Resolution set to fixed
- Status changed from new to closed
In 10255:
Just did a quick search,
bp_group_avatar_edit_form()
was used in 1.0 branch inbp-themes/bpmember/groups/admin/group-avatar.php
andbp-themes/bpskeletonmember/groups/admin/group-avatar.php
Since 1.1, this function is no more used. I doubt anyone is actually using it anymore because it would generate a fatal error as
groups_avatar_upload()
doesn't exist. I think we should just removebp_group_avatar_edit_form()
.