In function members_page, theme_user_picture($account) should be written as theme('user_picture', $account). The difference is that the second way allows theme overrides. I'm sorry I didn't make a patch (I got confused about which versions to use in cvs).

CommentFileSizeAuthor
#2 theme.patch713 bytesjody lynn

Comments

junyor’s picture

junyor commented

You should make a patch against the DRUPAL-5 branch.

jody lynn’s picture

jody lynn
she/her
Primary language English
commented
Status: Active » Needs review
StatusFileSize
new theme.patch713 bytes

Patch attached

junyor’s picture

junyor commented
Status: Needs review » Reviewed & tested by the community

Patch applies and works as it should.

psi-jack’s picture

psi-jack commented

Ahh, good. This is correct. I spent the last few days tracking this down, because I use imagecache_profiles (based on lullabot's approach), then it suddenly hit me. theme_user_profile is the last stack of the checking phase for theming, and shouldn't be called directly, as per Drupal API docs say to use theme().

Patch works, and fixes the problem and corrects the API regarding this specific issue.