Passed
Push — master ( afc50f...67ec34 )
by Mihail
04:58
created
Apps/Controller/Api/Content.php 1 patch
Spacing   +21 added lines, -21 removed lines patch added patch discarded remove patch
@@ -33,12 +33,12 @@  discard block
 block discarded – undo
33 33
         parent::before();
34 34
         $configs = AppRecord::getConfigs('app', 'Content');
35 35
         // prevent null-type config data
36
-        if ((int)$configs['gallerySize'] > 0) {
37
-            $this->maxSize = (int)$configs['gallerySize'] * 1024;
36
+        if ((int) $configs['gallerySize'] > 0) {
37
+            $this->maxSize = (int) $configs['gallerySize'] * 1024;
38 38
         }
39 39
 
40
-        if ((int)$configs['galleryResize'] > 0) {
41
-            $this->maxResize = (int)$configs['galleryResize'];
40
+        if ((int) $configs['galleryResize'] > 0) {
41
+            $this->maxResize = (int) $configs['galleryResize'];
42 42
         }
43 43
     }
44 44
     
@@ -116,8 +116,8 @@  discard block
 block discarded – undo
116 116
         }
117 117
 
118 118
         // check if directory exist
119
-        if (!Directory::exist('/upload/gallery/' . $id)) {
120
-            Directory::create('/upload/gallery/' . $id);
119
+        if (!Directory::exist('/upload/gallery/'.$id)) {
120
+            Directory::create('/upload/gallery/'.$id);
121 121
         }
122 122
 
123 123
         // get file object
@@ -129,7 +129,7 @@  discard block
 block discarded – undo
129 129
 
130 130
         // check file size
131 131
         if ($file->getSize() < 1 || $file->getSize() > $this->maxSize) {
132
-            throw new ForbiddenException(__('File size is too big. Max size: %size%kb', ['size' => (int)($this->maxSize/1024)]));
132
+            throw new ForbiddenException(__('File size is too big. Max size: %size%kb', ['size' => (int) ($this->maxSize / 1024)]));
133 133
         }
134 134
 
135 135
         // check file extension
@@ -138,37 +138,37 @@  discard block
 block discarded – undo
138 138
         }
139 139
 
140 140
         // create origin directory
141
-        $originPath = '/upload/gallery/' . $id . '/orig/';
141
+        $originPath = '/upload/gallery/'.$id.'/orig/';
142 142
         if (!Directory::exist($originPath)) {
143 143
             Directory::create($originPath);
144 144
         }
145 145
 
146 146
         // lets make a new file name
147
-        $fileName = App::$Security->simpleHash($file->getClientOriginalName() . $file->getSize());
148
-        $fileNewName = $fileName . '.' . $file->guessExtension();
147
+        $fileName = App::$Security->simpleHash($file->getClientOriginalName().$file->getSize());
148
+        $fileNewName = $fileName.'.'.$file->guessExtension();
149 149
         // save file from tmp to gallery origin directory
150 150
         $file->move(Normalize::diskFullPath($originPath), $fileNewName);
151 151
 
152 152
         // lets resize preview image for it
153
-        $thumbPath = '/upload/gallery/' . $id . '/thumb/';
153
+        $thumbPath = '/upload/gallery/'.$id.'/thumb/';
154 154
         if (!Directory::exist($thumbPath)) {
155 155
             Directory::create($thumbPath);
156 156
         }
157 157
 
158 158
         $thumb = new Image();
159
-        $thumb->setCacheDir(root . '/Private/Cache/images');
159
+        $thumb->setCacheDir(root.'/Private/Cache/images');
160 160
 
161 161
         // open original file, resize it and save
162
-        $thumbSaveName = Normalize::diskFullPath($thumbPath) . '/' . $fileName . '.jpg';
163
-        $thumb->open(Normalize::diskFullPath($originPath) . DIRECTORY_SEPARATOR . $fileNewName)
162
+        $thumbSaveName = Normalize::diskFullPath($thumbPath).'/'.$fileName.'.jpg';
163
+        $thumb->open(Normalize::diskFullPath($originPath).DIRECTORY_SEPARATOR.$fileNewName)
164 164
             ->cropResize($this->maxResize)
165 165
             ->save($thumbSaveName, 'jpg', 90);
166 166
         $thumb = null;
167 167
 
168 168
         // dont ask me why there is 2nd lvl array (can contains multiply items to frontend response)
169 169
         $output = [
170
-            'thumbnailUrl' => '/upload/gallery/' . $id . '/thumb/' . $fileName . '.jpg',
171
-            'url' => '/upload/gallery/' . $id . '/orig/' . $fileNewName,
170
+            'thumbnailUrl' => '/upload/gallery/'.$id.'/thumb/'.$fileName.'.jpg',
171
+            'url' => '/upload/gallery/'.$id.'/orig/'.$fileNewName,
172 172
             'name' => $fileNewName
173 173
         ];
174 174
 
@@ -197,7 +197,7 @@  discard block
 block discarded – undo
197 197
             throw new NativeException('Permission denied');
198 198
         }
199 199
 
200
-        $thumbDir = Normalize::diskFullPath('/upload/gallery/' . $id . '/orig/');
200
+        $thumbDir = Normalize::diskFullPath('/upload/gallery/'.$id.'/orig/');
201 201
         if (!Directory::exist($thumbDir)) {
202 202
             throw new NotFoundException('Nothing found');
203 203
         }
@@ -212,8 +212,8 @@  discard block
 block discarded – undo
212 212
             $fileExt = Str::lastIn($file, '.');
213 213
             $fileName = Str::sub($file, 0, -Str::length($fileExt));
214 214
             $output[] = [
215
-                'thumbnailUrl' => '/upload/gallery/' . $id . '/thumb/' . $fileName . '.jpg',
216
-                'url' => '/upload/gallery/' . $id . '/orig/' . $file,
215
+                'thumbnailUrl' => '/upload/gallery/'.$id.'/thumb/'.$fileName.'.jpg',
216
+                'url' => '/upload/gallery/'.$id.'/orig/'.$file,
217 217
                 'name' => $file
218 218
             ];
219 219
         }
@@ -245,8 +245,8 @@  discard block
 block discarded – undo
245 245
         }
246 246
 
247 247
         // generate path
248
-        $thumb = '/upload/gallery/' . $id . '/thumb/' . $fileName . '.jpg';
249
-        $full = '/upload/gallery/' . $id . '/orig/' . $file;
248
+        $thumb = '/upload/gallery/'.$id.'/thumb/'.$fileName.'.jpg';
249
+        $full = '/upload/gallery/'.$id.'/orig/'.$file;
250 250
 
251 251
         // check if file exists and remove
252 252
         if (File::exist($thumb) || File::exist($full)) {
Please login to merge, or discard this patch.
Apps/View/Admin/default/content/index.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -35,9 +35,9 @@  discard block
 block discarded – undo
35 35
                     <span class="sr-only">Toggle Dropdown</span>
36 36
                 </button>
37 37
                 <ul class="dropdown-menu" role="menu">
38
-                    <li><?= Url::link(['content/index', null, null, ['type' => 'all']], '<i class="fa fa-list"></i> ' . __('All')) ?></li>
39
-                    <li><?= Url::link(['content/index', null, null, ['type' => 'moderate']], '<i class="fa fa-exclamation"></i> ' . __('Moderate')) ?></li>
40
-                    <li><?= Url::link(['content/index', null, null, ['type' => 'trash']], '<i class="fa fa-trash"></i> ' . __('Trash')) ?></li>
38
+                    <li><?= Url::link(['content/index', null, null, ['type' => 'all']], '<i class="fa fa-list"></i> '.__('All')) ?></li>
39
+                    <li><?= Url::link(['content/index', null, null, ['type' => 'moderate']], '<i class="fa fa-exclamation"></i> '.__('Moderate')) ?></li>
40
+                    <li><?= Url::link(['content/index', null, null, ['type' => 'trash']], '<i class="fa fa-trash"></i> '.__('Trash')) ?></li>
41 41
                     <li>
42 42
                         <a class="trigger right-caret"><i class="fa fa-table"></i> <?= __('Categories') ?></a>
43 43
                         <ul class="dropdown-menu sub-menu">
@@ -51,9 +51,9 @@  discard block
 block discarded – undo
51 51
         </div>
52 52
         <?php
53 53
         if ($type === 'trash') {
54
-            echo Url::link(['content/clear'], '<i class="fa fa-minus"></i> ' . __('Remove all'), ['class' => 'btn btn-danger pull-right']);
54
+            echo Url::link(['content/clear'], '<i class="fa fa-minus"></i> '.__('Remove all'), ['class' => 'btn btn-danger pull-right']);
55 55
         } else {
56
-            echo Url::link(['content/update', 0], '<i class="fa fa-plus"></i> ' . __('Add content'), ['class' => 'btn btn-primary pull-right']);
56
+            echo Url::link(['content/update', 0], '<i class="fa fa-plus"></i> '.__('Add content'), ['class' => 'btn btn-primary pull-right']);
57 57
         }
58 58
         ?>
59 59
     </div>
@@ -62,23 +62,23 @@  discard block
 block discarded – undo
62 62
 <?php
63 63
 
64 64
 if ($records->count() < 1) {
65
-    echo '<p class="alert alert-warning">' . __('Content is not found') . '</p>';
65
+    echo '<p class="alert alert-warning">'.__('Content is not found').'</p>';
66 66
     return;
67 67
 }
68 68
 
69 69
 $items = [];
70 70
 $moderate = false;
71 71
 foreach ($records as $content) {
72
-    $frontLink = \App::$Alias->scriptUrl . '/content/read';
72
+    $frontLink = \App::$Alias->scriptUrl.'/content/read';
73 73
     $frontPath = null;
74 74
     if (!Str::likeEmpty($content->getCategory()->path)) {
75
-        $frontLink .= '/' . $content->getCategory()->path;
76
-        $frontPath .= '/' . $content->getCategory()->path;
75
+        $frontLink .= '/'.$content->getCategory()->path;
76
+        $frontPath .= '/'.$content->getCategory()->path;
77 77
     }
78
-    $frontLink .= '/' . $content->path;
79
-    $frontPath .= '/' . $content->path;
78
+    $frontLink .= '/'.$content->path;
79
+    $frontPath .= '/'.$content->path;
80 80
     $frontPath = Str::sub($frontPath, 0, 30);
81
-    $actionIcons = '<a href="' . $frontLink . '" target="_blank"><i class="fa fa-eye fa-lg"></i></a> ';
81
+    $actionIcons = '<a href="'.$frontLink.'" target="_blank"><i class="fa fa-eye fa-lg"></i></a> ';
82 82
     $actionIcons .= Url::link(['content/update', $content->id], '<i class="fa fa-pencil fa-lg"></i> ');
83 83
     if ($type === 'trash') {
84 84
         $actionIcons .= Url::link(['content/restore', $content->id], '<i class="fa fa-refresh fa-lg"></i>');
@@ -86,16 +86,16 @@  discard block
 block discarded – undo
86 86
         $actionIcons .= Url::link(['content/delete', $content->id], '<i class="fa fa-trash-o fa-lg"></i>');
87 87
     }
88 88
 
89
-    if (!(bool)$content->display) {
89
+    if (!(bool) $content->display) {
90 90
         $moderate = true;
91 91
     }
92 92
 
93 93
     $items[] = [
94
-        'property' => ['class' => 'checkbox-row' . (!(bool)$content->display ? ' alert-warning' : null)],
94
+        'property' => ['class' => 'checkbox-row'.(!(bool) $content->display ? ' alert-warning' : null)],
95 95
         1 => ['text' => $content->id, 'html' => true, '!secure' => true],
96
-        2 => ['text' => (!(bool)$content->display ? '<i class="fa fa-exclamation text-warning"></i> ' : null) . Url::link(['content/update', $content->id], Serialize::getDecodeLocale($content->title)), 'html' => true],
96
+        2 => ['text' => (!(bool) $content->display ? '<i class="fa fa-exclamation text-warning"></i> ' : null).Url::link(['content/update', $content->id], Serialize::getDecodeLocale($content->title)), 'html' => true],
97 97
         3 => ['text' => Serialize::getDecodeLocale($content->getCategory()->title)],
98
-        4 =>['text' => '<a href="' . $frontLink . '" target="_blank">' . $frontPath . '</a>', 'html' => true],
98
+        4 =>['text' => '<a href="'.$frontLink.'" target="_blank">'.$frontPath.'</a>', 'html' => true],
99 99
         5 => ['text' => Date::convertToDatetime($content->updated_at, Date::FORMAT_TO_SECONDS)],
100 100
         6 => ['text' => $actionIcons, 'html' => true, 'property' => ['class' => 'text-center']]
101 101
     ];
Please login to merge, or discard this patch.