Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
corrected some @Buddy List to @Buddylist
moved the Remove from a user profile page to the users buddylist
removed the pending from the user profile page
simplified the text on the buddylist pending if no sent or received pending
to None.
Comment | File | Size | Author |
---|---|---|---|
buddylist_9.patch | 6.45 KB | dsp1 |
Comments
Comment #1
dldege CreditAttribution: dldege commentedFinally got a look at this. There is some good stuff in here but a few things to discuss and fix.
1. A couple of these things have been fixed in other commits so a new patch needs to be created.
2. I'm ok with adding in the "remove" link but I'd like comments from other users about this since it could be done in your own theme is desired. Also, you need to change how its done
You should not call theme_remove_from_buddylist_link directly as that circumvents the theme system - call theme('remove_from_buddylist_link'...... instead
I'd prefer not to pad out content with break tags - we should control padding with CSS and if the content is not in a container to make that possible we need to fix that.
I think your change to theme_remove_from_buddylist_link is wrong - you need to pass the $buddyuser->uid as the last part of the url.
Thanks
Comment #2
dldege CreditAttribution: dldege commented