Changeset 1156 for trunk/common

Timestamp:

12/06/11 01:14:30 (6 months ago)

Author:

bogdan2412

Message:

Fix errors that appeared during deployment of avatar optimization

• Use hardlinks for full size avatars when possible, so we conserve

disk space.

• Skip users with invalid usernames
• Create 'tiny', 'small', etc folders if missing
• Added verbose parameter that allows one to view progress
• Coding style, don't use " when not needed.

There are still some tasks left, as described in
REVIEW URL ​http://reviewboard.infoarena.ro/r/180/

File:

• trunk/common/avatar.php (modified) (1 diff)

trunk/common/avatar.php

@@ -21 +21 @@
             'L50x50' => 'normal/' , '75x75'=> 'forum/', '150x150' => 'big/');
 
-    // Copying the original image
-    copy($filepath, IA_AVATAR_FOLDER.'full/'.$new_filename);
+    // Hardlink / Copy the original image
+    $new_filepath = IA_AVATAR_FOLDER . 'full/' . $new_filename;
+    if (is_file($new_filepath) || is_link($new_filepath)) {
+        unlink($new_filepath);
+    }
+    if (!link($filepath, $new_filepath)) {
+        if (!copy($filepath, $new_filepath)) {
+            log_error('Unable to copy user avatar into avatar folder');
+        }
+    }
 
     list($image_width, $image_height, $image_type, $image_attribute) =