[buddypress-trac] [BuddyPress Trac] #6931: Cover Image location is incorrect for blogs other than the primary blog

[buddypress-trac]

#6931: Cover Image location is incorrect for blogs other than the primary blog
-------------------------------------+-----------------------
 Reporter:  rbaccaro                 |       Owner:
     Type:  defect (bug)             |      Status:  reopened
 Priority:  normal                   |   Milestone:  2.6
Component:  Component - Attachments  |     Version:  2.4.0
 Severity:  normal                   |  Resolution:
 Keywords:  has-patch 2nd-opinion    |
-------------------------------------+-----------------------
Changes (by imath):

 * status:  closed => reopened
 * resolution:  worksforme =>
 * milestone:   => 2.6


Comment:

 Hi @rbaccaro

 With the gist, i think you don't need the code you shared in #comment:2 it
 should work for every object using the BP Attachment API (avatars/ cover
 images...)

 I actually think we need to improve this part but take the time to check
 changes introduced would not have side effects on plugins extending the
 Attachment API etc..

 I think for multiblog we can do something like what i'm suggesting in the
 patch attached to this ticket. So i'll leave the ticket open so that
 others can comment/ give their feedback.

 Thanks again for your feedback :)

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/6931#comment:4>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac