What's the easiest way to resize $user->picture?

I like the idea... Maybe you could start a feature request, to where the theme can decide what it wants. You might take a look at image.module, because it uses a thumbnail, medium size, and the original size for images, and other sizes are configurable.

Another option I'd like to see is to allow the user's picture to be displayed in certain node types, as in the forum only. This can be done with the theme, but it's a hack, and not one I like to use. That's a whole new topic though...

--
Bradlis7.com | Churchofchristnet

I made a patch out of this and added it to http://drupal.org/node/60987. It needs more work, though...

What do you think would be the best way to get more images on the user profile?
Thanks

Hi,

I first tried to install the patch but MAC OSX's terminal claimed it was bad code that ended halfway. Might also have been me being a klutz. Command line is not one of my better features.

However, I used the code posted above instead and now I've got two problems;

1) The user image I'm testing with don't resize when I use $icon or $mini in the user_profile.tpl.php. Am I supposed to determine image sizes anywhere else than in the snippet above or have any other modules installed? I have GD and the image_module installed if that matters.

2) I get the image path written out everywhere the image appears. At a look at the source code it says;

<div class="content">http://quicksilver.local/fixxxa/filer/medlemsbilder/picture-1.gif<div class="picture"><a href="/fixxxa/?q=medlemmar/annika" title="Titta på användarprofilen."><img src="//quicksilver.local/fixxxa/filer/medlemsbilder/picture-1.gif" alt="Annikas bild" title="Annikas bild" /></a></div>

The code in the user_profile.tpl.php is

<?php
  if($user->picture) {
    print theme('user_picture', $user, '$mini');
  }
?>

I haven't the faintest idea from where that path comes from. I've redone the process twice to be certain there are no errors, but alas still no success.

I hope somebody can shred some light on this.

Oh, and I'm using Drupal 4.7.2 with a modded Friends Electric theme.

:-)

Ayza

Hello all,

I think this mod is a very good idea. I don't understand why drupal doesn't do it by itself yet, or why theres is no module for this.

Either way, I don't like at all to modify drupal's release files because that stops me to upgrade when a new release is available. So I was thinking, what about creating the resized images via cron? It will be a hook which would check if the small images doesn't exist or if the user picture has been updated (date is newer than thumbs), and the create or recreate the small ones.

That way we will have a lag between a picture modification and cron update, but will be way cleaner.

Any ideas of how to implement this? I just don't have time right now, and I would appreciate any code! :)

Based on the code shown above (very good job, ajwwong), I have created what will do the same, but without touching a single line of Drupla's code. This means we don't need to modify user.module file.

The way I have done it, instead of creating thumbnails when user uploads his picture, pictures are resized on demand, if they haven't been resized previously or if user's image has changed.

Step 1: Add this to the end of your template.php (identical to previous 2nd step):

<?php
  function phptemplate_user_picture($account, $size = '') {
    return _phptemplate_callback('user_picture', array('account' => $account, 'size' => $size));
  }
?>

Step 2: Add the file user_picture.tpl.php in your active template folder and put the following into it:

<?php
  if (variable_get('user_pictures', 0)) 
  {
    if ($account->picture && file_exists($account->picture)) 
    {
      if ($size == 'node') // 80x80
      {
        $info = image_get_info($account->picture);
        $newpicture = dirname($account->picture) . '/picture-' . $account->uid . ".80." . $info['extension'];
        if (!file_exists($newpicture) || (filectime($newpicture) < filectime($account->picture))) {
          image_scale($account->picture, $newpicture, 80, 80);
        } 
        $picture = file_create_url($newpicture);
      }
      else if ($size == 'comment')  // 48x48
      {
        $info = image_get_info($account->picture);
        $newpicture = dirname($account->picture) . '/picture-' . $account->uid . ".48." . $info['extension'];
        if (!file_exists($newpicture) || (filectime($newpicture) < filectime($account->picture))) {
          image_scale($account->picture, $newpicture, 48, 48);
        } 
        $picture = file_create_url($newpicture);
      }
      else if ($size == 'mini')  // 20x20
      {
        $info = image_get_info($account->picture);
        $newpicture = dirname($account->picture) . '/picture-' . $account->uid . ".20." . $info['extension'];
        if (!file_exists($newpicture) || (filectime($newpicture) < filectime($account->picture))) {
          image_scale($account->picture, $newpicture, 20, 20);
        } 
        $picture = file_create_url($newpicture);
      }
      else
      {
        $picture = file_create_url($account->picture);
      }
    }
    else if (variable_get('user_picture_default', '')) {
      $picture = variable_get('user_picture_default', '');
    }

    if (isset($picture)) {
      $alt = t('%user\'s picture', array('%user' => $account->name ? $account->name : variable_get('anonymous', 'Anonymous')));
      $picture = theme('image', $picture, $alt, $alt, '', false);
      if (!empty($account->uid) && user_access('access user profiles')) {
        $picture = l($picture, "user/$account->uid", array('title' => t('View user profile.')), NULL, NULL, FALSE, TRUE);
      }

      print_r("<div class=\"picture\">$picture</div>");
    }
  }
 
?>

Of course the code can be modified for diferent image sizes and even better! It would be very easy to modify it to allow passing the final size in pixels (instead of 'node', comment' and 'mini') and get images at runtime of any size.

I hope you like it! For a demonstration, check out this pages and compare the different picture sizes:

• http://spaniards.es/node/1
• http://spaniards.es/user/1

Enjoy!

Try something like this

  if (!form_get_errors()) {
    if ($file = file_save_upload('picture_upload', variable_get('user_picture_path', 'pictures') .'/picture-'. $user->uid .'.'. $info['extension'], 1)) {
      $form_values['picture'] = $file->filepath;
	  image_scale($file->filepath,  'files/'.variable_get('user_picture_path', 'pictures').'/pictureSmall-'. $user->uid. '.' . $info['extension'], 72, 59);
    }
    else {
      form_set_error('picture_upload', t("Failed to upload the picture image; the %directory directory doesn't exist.", array('%directory' => variable_get('user_picture_path', 'pictures'))));
    }
  }

Otherwise when user will update his old picture, the thumbnail or other versions will not be updated.

WELL WELL WELL

I went to try uploading a giant picture, couldn't, set IMCE permission to "limitless upload", tried again, and voila, there's a an auto-resize tick box next to 'upload'. WOW COOL ... IMCE already does what I wanted to do.

Problem solved.

IMCE, very cool.

Quint

yeah, I'm in 5.1

Quint
calculator for building stairs -- www.Shalla.Net

subscribing

When i hover over the picture i get the reading. i tried dyutiman's code:

  if (!form_get_errors()) {
    if ($file = file_save_upload('picture_upload', variable_get('user_picture_path', 'profilepics') .'/picture-'. $user->uid .'.'. $info['extension'], 1)) {
      $form_values['picture'] = $file->filepath;
      image_scale($file->filepath,  'files/'.variable_get('user_picture_path', 'profilepics').'/pictureSmall-'. $user->uid. '.' . $info['extension'], 72, 59);
    }
    else {
      form_set_error('picture_upload', t("Failed to upload the picture image; the %directory directory doesn't exist.", array('%directory' => variable_get('user_picture_path', 'profilepics'))));
    }
  }

with the replacement of 'pictures' as my pics are stored in /files/profilepics...
i think i've changed it wrongly? (doesn't work)

?

Mainly because that's only changing the display size. What's desired is a physical resize of the image, not just a soft resize.

Steel Rat
My Drupal Sites:
RPGMapShare.com
Infinite Ordnance
Malvern Rouge Valley Youth Center

Hey guys,
code works great. Apart from when the size of the image uploaded is smaller than the resized size. Then it fails to work.
How should I adapt the code?
Currently I am using:

  if (variable_get('user_pictures', 0))
  {
    if ($account->picture && file_exists($account->picture))
    {
      if ($size == 'node') // 200x200
      {
        $info = image_get_info($account->picture);
        $newpicture = dirname($account->picture) . '/picture-' . $account->uid . ".200." . $info['extension'];
        if (!file_exists($newpicture) || (filectime($newpicture) < filectime($account->picture))) {
          image_scale($account->picture, $newpicture, 200, 200);
        }
        $picture = file_create_url($newpicture);
      }
      else if ($size == 'pnode')  // 150x150
      {
        $info = image_get_info($account->picture);
        $newpicture = dirname($account->picture) . '/picture-' . $account->uid . ".150." . $info['extension'];
        if (!file_exists($newpicture) || (filectime($newpicture) < filectime($account->picture))) {
          image_scale($account->picture, $newpicture, 150, 150);
        }
        $picture = file_create_url($newpicture);
      }
      else if ($size == 'taxonomy')  // 80x80
      {
        $info = image_get_info($account->picture);
        $newpicture = dirname($account->picture) . '/picture-' . $account->uid . ".95." . $info['extension'];
        if (!file_exists($newpicture) || (filectime($newpicture) < filectime($account->picture))) {
          image_scale($account->picture, $newpicture, 95, 95);
        }
        $picture = file_create_url($newpicture);
      }
      else
      {
        $picture = file_create_url($account->picture);
      }
    }
    else if (variable_get('user_picture_default', '')) {
      $picture = variable_get('user_picture_default', '');
    }

    if (isset($picture)) {
      $alt = t($account->name.'\'s picture', array('%user' => $account->name ? $account->name : variable_get('anonymous', 'Anonymous')));
      $picture = theme('image', $picture, $alt, $alt, '', false);
      if (!empty($account->uid) && user_access('access user profiles')) {
        $picture = l($picture, "user/$account->uid", array('title' => t('View user profile.')), NULL, NULL, FALSE, TRUE);
      }

      print_r("<div class=\"picture\">$picture</div>");
    }
  }