add timestamp to avatar filenames to prevent overlap

[quix0rs-gnu-social.git] / actions / avatar.php

diff --git a/actions/avatar.php b/actions/avatar.php
index 43f02a88d593cfe42e87a17db1058d4d24b4f108..023a8414a18252338f87eeebf581401d41d3ffdd 100644 (file)
--- a/actions/avatar.php
+++ b/actions/avatar.php
@@ -108,7 +108,7 @@ class AvatarAction extends SettingsAction {
 
                $user = common_current_user();
 
-               $filename = common_avatar_filename($user, image_type_to_extension($info[2]));
+               $filename = common_avatar_filename($user, image_type_to_extension($info[2]), NULL, common_timestamp());
                $filepath = common_avatar_path($filename);
 
                if (!move_uploaded_file($_FILES['avatarfile']['tmp_name'], $filepath)) {
@@ -128,17 +128,6 @@ class AvatarAction extends SettingsAction {
                $avatar->url = common_avatar_url($filename);
                $avatar->created = DB_DataObject_Cast::dateTime(); # current time
 
-               $val = $avatar->validate();
-               
-               if ($val !== TRUE) {
-                       $err = '';
-                       foreach ($val as $k=>$v) {
-                               $err .= _t('Something wrong with ') . $k;
-                               $this->show_form($err);
-                               return;
-                       }
-               }
-               
                foreach (array(AVATAR_PROFILE_SIZE, AVATAR_STREAM_SIZE, AVATAR_MINI_SIZE) as $size) {
                        $scaled[] = $this->scale_avatar($user, $avatar, $size);
                }
@@ -177,9 +166,9 @@ class AvatarAction extends SettingsAction {
                imagecopyresampled($image_s, $image_a, 0, 0, 0, 0,
                                                   $size, $size, $square, $square);
 
-               $ext = ($avatar->mediattype == 'image/jpeg') ? ".jpg" : ".png";
+               $ext = ($avatar->mediattype == 'image/jpeg') ? ".jpeg" : ".png";
 
-               $filename = common_avatar_filename($user, $ext, $size);
+               $filename = common_avatar_filename($user, $ext, $size, common_timestamp());
 
                if ($avatar->mediatype == 'image/jpeg') {
                        imagejpeg($image_s, common_avatar_path($filename));