Changeset 15019 for events/phpframework/piece_framework/trunk/web/frontend/webapp/actions/Account/PictureAction.php

Timestamp:

07/02/08 00:58:49 (6 months ago)

Author:

kumatch

Message:

Adjusted the reviewed and refactored behaviors to the presentation layer.

Files:

• events/phpframework/piece_framework/trunk/web/frontend/webapp/actions/Account/PictureAction.php (modified) (8 diffs)

events/phpframework/piece_framework/trunk/web/frontend/webapp/actions/Account/PictureAction.php

@@ -13 +13 @@
 require_once 'Piece/Unity/Service/FlowAction.php';
 require_once 'Piece/Unity/Service/Runemaster.php';
+require_once 'Phwittr/Self.php';
 require_once 'Phwittr/Account.php';
 
@@ -40 +41 @@
      */
 
-    var $_userId;
+    var $_self;
 
     /**#@-*/
@@ -50 +51 @@
     function entryForm()
     {
-        if (is_null($this->_userId)) {
-            $user = $this->_context->getSession()->getAttribute('user');
-            $this->_userId = $user->id;
+        if (is_null($this->_self)) {
+            $loginUser = $this->_context->getSession()->getAttribute('loginUser');
+            $this->_self = new Phwittr_Self($loginUser->id);
         }
     }
@@ -60 +61 @@
         $master = new Piece_Unity_Service_Runemaster();
         $master->addForm('AccountPicture', '/account/picture', true, true);
+
+        $viewElement = $this->_context->getViewElement();
+        $viewElement->setElementByRef('self', $this->_self);
     }
 
@@ -71 +75 @@
 
         $saveFilePath = PHWITTR_PROFILE_IMAGES_DIRECTORY .
-            "/{$this->_userId}" . "/{$formValue->uploadedData['name']}";
+            "/{$this->_self->id}" . "/{$formValue->uploadedData['name']}";
         $saveDirectory = dirname($saveFilePath);
 
@@ -78 +82 @@
         move_uploaded_file($formValue->uploadedData['tmp_name'], $saveFilePath);
 
-        $account = new Phwittr_Account();
-        $account->updatePicture($this->_userId, $saveFilePath);
+        Phwittr_Account::updatePicture($this->_self->id, $saveFilePath);
+        unset($this->_self);
+        $this->_self = null;
 
         return 'DisplayForm';
@@ -93 +98 @@
     {
         if (!file_exists($directory)) {
-            mkdir(dirname($directory));
+            mkdir($directory);
         } else {
             $dir = new DirectoryIterator($directory);
@@ -100 +105 @@
                     continue;
                 }
-                if ($dir->isDir()) {
-                    $this->_removeInnerDirectory("{$directory}/{$target}");
-                    rmdir("{$directory}/{$target}");
-                } else {
-                    unlink("{$directory}/{$target}");
-                }
+                unlink("{$directory}/{$target}");
             }
         }