I had to use the overridden user picture handler on a D7 project and noticed it was working, so I went through and updated everything for the latest D7 and Views changes.

CommentFileSizeAuthor
#1 993024-imagecache-profiles-update.patch5.2 KBdave reid

Comments

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Active » Needs review
StatusFileSize
new 993024-imagecache-profiles-update.patch5.2 KB

Summary of changes:
1. Views no longer needs your imagecache_profiles_views_handlers() and just wants all your handler files declared in the module's .info files[] array.
2. Adjusted the views handler to work with the new core image API options.

Tested this and it works perfectly with Views 7.x-3.x and Drupal 7.0-rc1!

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Needs review » Fixed

Commited, Thanx a lot Dave! I'm going to release RC1 latter this day

andypost’s picture

andypost
he/him
Primary language Russian
commented

The only thing that scares me - 'getsize' parameter that semms unused in core neither theme_image() nor theme_image_style()

Seems like a core bug!!! If you know an issue please post it here.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

Yeah it's not used in core anymore but its intentional, which is why that part was removed from imagecache_profiles_preprocess_user_picture().

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.