Browse code

Clean up files

doishub authored on08/12/2020 15:43:32
Showing2 changed files
... ...
@@ -6,22 +6,33 @@
6 6
  */
7 7
 namespace Oveleon\ContaoMemberExtensionBundle;
8 8
 
9
+use Contao\Config;
10
+use Contao\Dbafs;
11
+use Contao\File;
12
+use Contao\FilesModel;
13
+use Contao\Frontend;
14
+use Contao\FrontendUser;
15
+use Contao\MemberModel;
16
+use Contao\StringUtil;
17
+use Contao\Validator;
18
+
9 19
 /**
10 20
  * Class Member
11 21
  *
12 22
  * @author Fabian Ekert <fabian@oveleon.de>
23
+ * @author Daniele Sciannimanica <https://github.com/doishub>
13 24
  */
14
-class Member extends \Frontend
25
+class Member extends Frontend
15 26
 {
16 27
     /**
17 28
      * Update avatar of member
18 29
      *
19
-     * @param \FrontendUser $objUser
20
-     * @param array         $arrData
30
+     * @param FrontendUser $objUser
31
+     * @param array        $arrData
21 32
      */
22 33
     public function updateAvatar($objUser, $arrData)
23 34
     {
24
-        $objMember = \MemberModel::findByPk($objUser->id);
35
+        $objMember = MemberModel::findByPk($objUser->id);
25 36
 
26 37
         if ($objMember === null)
27 38
         {
... ...
@@ -34,7 +45,7 @@ class Member extends \Frontend
34 45
         // Sanitize the filename
35 46
         try
36 47
         {
37
-            $file['name'] = \StringUtil::sanitizeFileName($file['name']);
48
+            $file['name'] = StringUtil::sanitizeFileName($file['name']);
38 49
         }
39 50
         catch (\InvalidArgumentException $e)
40 51
         {
... ...
@@ -44,7 +55,7 @@ class Member extends \Frontend
44 55
         }
45 56
 
46 57
         // Invalid file name
47
-        if (!\Validator::isValidFileName($file['name']))
58
+        if (!Validator::isValidFileName($file['name']))
48 59
         {
49 60
             // ToDo: Fehler: Dateiname beinhaltet unzulässige Zeichen
50 61
 
... ...
@@ -63,8 +74,8 @@ class Member extends \Frontend
63 74
             return;
64 75
         }
65 76
 
66
-        $objFile = new \File($file['name']);
67
-        $uploadTypes = \StringUtil::trimsplit(',', \Config::get('validImageTypes'));
77
+        $objFile = new File($file['name']);
78
+        $uploadTypes = StringUtil::trimsplit(',', \Config::get('validImageTypes'));
68 79
 
69 80
         // File type is not allowed
70 81
         if (!\in_array($objFile->extension, $uploadTypes))
... ...
@@ -77,7 +88,7 @@ class Member extends \Frontend
77 88
 
78 89
         if ($arrImageSize = @getimagesize($file['tmp_name']))
79 90
         {
80
-            $intImageWidth = \Config::get('imageWidth');
91
+            $intImageWidth = Config::get('imageWidth');
81 92
 
82 93
             // Image exceeds maximum image width
83 94
             if ($intImageWidth > 0 && $arrImageSize[0] > $intImageWidth)
... ...
@@ -88,7 +99,7 @@ class Member extends \Frontend
88 99
                 return;
89 100
             }
90 101
 
91
-            $intImageHeight = \Config::get('imageHeight');
102
+            $intImageHeight = Config::get('imageHeight');
92 103
 
93 104
             // Image exceeds maximum image height
94 105
             if ($intImageHeight > 0 && $arrImageSize[1] > $intImageHeight)
... ...
@@ -109,7 +120,7 @@ class Member extends \Frontend
109 120
 
110 121
             $intUploadFolder = $objMember->homeDir;
111 122
 
112
-            $objUploadFolder = \FilesModel::findByUuid($intUploadFolder);
123
+            $objUploadFolder = FilesModel::findByUuid($intUploadFolder);
113 124
 
114 125
             // The upload folder could not be found
115 126
             if ($objUploadFolder === null)
... ...
@@ -126,25 +137,25 @@ class Member extends \Frontend
126 137
 
127 138
                 // Move the file to its destination
128 139
                 $this->Files->move_uploaded_file($file['tmp_name'], $strUploadFolder . '/' . $file['name']);
129
-                $this->Files->chmod($strUploadFolder . '/' . $file['name'], \Config::get('defaultFileChmod'));
140
+                $this->Files->chmod($strUploadFolder . '/' . $file['name'], Config::get('defaultFileChmod'));
130 141
 
131 142
                 $strUuid = null;
132 143
                 $strFile = $strUploadFolder . '/' . $file['name'];
133 144
 
134 145
                 // Generate the DB entries
135
-                if (\Dbafs::shouldBeSynchronized($strFile))
146
+                if (Dbafs::shouldBeSynchronized($strFile))
136 147
                 {
137
-                    $objModel = \FilesModel::findByPath($strFile);
148
+                    $objModel = FilesModel::findByPath($strFile);
138 149
 
139 150
                     if ($objModel === null)
140 151
                     {
141
-                        $objModel = \Dbafs::addResource($strFile);
152
+                        $objModel = Dbafs::addResource($strFile);
142 153
                     }
143 154
 
144
-                    $strUuid = \StringUtil::binToUuid($objModel->uuid);
155
+                    $strUuid = StringUtil::binToUuid($objModel->uuid);
145 156
 
146 157
                     // Update the hash of the target folder
147
-                    \Dbafs::updateFolderHashes($strUploadFolder);
158
+                    Dbafs::updateFolderHashes($strUploadFolder);
148 159
 
149 160
                     // Update member avatar
150 161
                     $objMember->avatar = $objModel->uuid;
... ...
@@ -195,6 +206,6 @@ class Member extends \Frontend
195 206
             $upload_max_filesize = round($upload_max_filesize * 1024 * 1024 * 1024);
196 207
         }
197 208
 
198
-        return min($upload_max_filesize, \Config::get('maxFileSize'));
209
+        return min($upload_max_filesize, Config::get('maxFileSize'));
199 210
     }
200
-}
201 211
\ No newline at end of file
212
+}
... ...
@@ -74,7 +74,7 @@ abstract class ModuleMemberExtension extends Module
74 74
      *
75 75
      * @param $objMember
76 76
      * @param $objTemplate
77
-     * @param $varImageSize
77
+     * @param $varImgSize
78 78
      */
79 79
     protected function addAvatarToTemplate($objMember, $objTemplate, $varImgSize)
80 80
     {