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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Status: Active » Needs review
FileSize
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
CreditAttribution: andypost 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
CreditAttribution: andypost 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
CreditAttribution: Dave Reid 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.