Completed
Pull Request — master (#126)
by
unknown
51:39
created
packages/Article/src/Service/EventService.php 2 patches
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -63,6 +63,9 @@
 block discarded – undo
63 63
         return $this->getPagination($select, $page, $limit);
64 64
     }
65 65
 
66
+    /**
67
+     * @param integer $limit
68
+     */
66 69
     public function fetchLatest($limit)
67 70
     {
68 71
         return $this->articleEventsMapper->getLatest($limit);
Please login to merge, or discard this patch.
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types=1);
3
+declare(strict_types = 1);
4 4
 
5 5
 namespace Article\Service;
6 6
 
@@ -102,15 +102,15 @@  discard block
 block discarded – undo
102 102
 
103 103
         $article = $articleFilter->getValues();
104 104
         $article += [
105
-            'admin_user_uuid' => $this->adminUsersMapper->getUuid($article['admin_user_id']),
105
+            'admin_user_uuid' => $this->adminUsersMapper->getUuid($article[ 'admin_user_id' ]),
106 106
             'type'            => ArticleType::EVENT,
107 107
             'article_id'      => $id,
108 108
             'article_uuid'    => $uuId,
109 109
         ];
110
-        unset($article['admin_user_id']);
110
+        unset($article[ 'admin_user_id' ]);
111 111
 
112
-        $article['category_uuid'] = $this->categoryMapper->get($article['category_id'])->category_uuid;
113
-        unset($article['category_id']);
112
+        $article[ 'category_uuid' ] = $this->categoryMapper->get($article[ 'category_id' ])->category_uuid;
113
+        unset($article[ 'category_id' ]);
114 114
 
115 115
         $event = $eventFilter->getValues() + [
116 116
                 'featured_img' => $this->upload->uploadImage($data, 'featured_img'),
@@ -132,31 +132,31 @@  discard block
 block discarded – undo
132 132
             throw new FilterException($articleFilter->getMessages() + $eventFilter->getMessages());
133 133
         }
134 134
 
135
-        $article = $articleFilter->getValues() + ['article_uuid' => $oldArticle->article_uuid];
135
+        $article = $articleFilter->getValues() + [ 'article_uuid' => $oldArticle->article_uuid ];
136 136
         $event = $eventFilter->getValues() + [
137 137
                 'featured_img' => $this->upload->uploadImage($data, 'featured_img'),
138 138
                 'main_img'     => $this->upload->uploadImage($data, 'main_img'),
139 139
             ];
140 140
 
141
-        $article['admin_user_uuid'] = $this->adminUsersMapper->getUuid($article['admin_user_id']);
142
-        $article['category_uuid'] = $this->categoryMapper->get($article['category_id'])->category_uuid;
143
-        unset($article['category_id'], $article['admin_user_id']);
141
+        $article[ 'admin_user_uuid' ] = $this->adminUsersMapper->getUuid($article[ 'admin_user_id' ]);
142
+        $article[ 'category_uuid' ] = $this->categoryMapper->get($article[ 'category_id' ])->category_uuid;
143
+        unset($article[ 'category_id' ], $article[ 'admin_user_id' ]);
144 144
 
145 145
         // We don't want to force user to re-upload image on edit
146
-        if (!$event['featured_img']) {
147
-            unset($event['featured_img']);
146
+        if (!$event[ 'featured_img' ]) {
147
+            unset($event[ 'featured_img' ]);
148 148
         } else {
149 149
             $this->upload->deleteFile($oldArticle->featured_img);
150 150
         }
151 151
 
152
-        if (!$event['main_img']) {
153
-            unset($event['main_img']);
152
+        if (!$event[ 'main_img' ]) {
153
+            unset($event[ 'main_img' ]);
154 154
         } else {
155 155
             $this->upload->deleteFile($oldArticle->main_img);
156 156
         }
157 157
 
158
-        $this->articleMapper->update($article, ['article_uuid' => $article['article_uuid']]);
159
-        $this->articleEventsMapper->update($event, ['article_uuid' => $article['article_uuid']]);
158
+        $this->articleMapper->update($article, [ 'article_uuid' => $article[ 'article_uuid' ] ]);
159
+        $this->articleEventsMapper->update($event, [ 'article_uuid' => $article[ 'article_uuid' ] ]);
160 160
     }
161 161
 
162 162
     public function deleteArticle($id)
@@ -169,7 +169,7 @@  discard block
 block discarded – undo
169 169
 
170 170
         $this->upload->deleteFile($event->main_img);
171 171
         $this->upload->deleteFile($event->featured_img);
172
-        $this->articleEventsMapper->delete(['article_uuid' => $event->article_uuid]);
172
+        $this->articleEventsMapper->delete([ 'article_uuid' => $event->article_uuid ]);
173 173
         $this->delete($event->article_uuid);
174 174
     }
175 175
 }
Please login to merge, or discard this patch.
packages/Article/src/Service/PostService.php 2 patches
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -203,6 +203,9 @@
 block discarded – undo
203 203
         return $this->articleMapper->getCategories($articleId);
204 204
     }
205 205
 
206
+    /**
207
+     * @param integer $limit
208
+     */
206 209
     public function getLatestWeb($limit)
207 210
     {
208 211
         return $this->articlePostsMapper->getLatest($limit);
Please login to merge, or discard this patch.
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types=1);
3
+declare(strict_types = 1);
4 4
 
5 5
 namespace Article\Service;
6 6
 
@@ -125,19 +125,19 @@  discard block
 block discarded – undo
125 125
 
126 126
         $article = $articleFilter->getValues();
127 127
         $article += [
128
-            'admin_user_uuid' => $this->adminUsersMapper->getUuid($article['admin_user_id']),
128
+            'admin_user_uuid' => $this->adminUsersMapper->getUuid($article[ 'admin_user_id' ]),
129 129
             'type'            => ArticleType::POST,
130 130
             'article_id'      => $id,
131 131
             'article_uuid'    => $uuId,
132 132
         ];
133 133
 
134
-        $article['category_uuid'] = $this->categoryMapper->get($article['category_id'])->category_uuid;
135
-        unset($article['category_id'], $article['admin_user_id']);
134
+        $article[ 'category_uuid' ] = $this->categoryMapper->get($article[ 'category_id' ])->category_uuid;
135
+        unset($article[ 'category_id' ], $article[ 'admin_user_id' ]);
136 136
 
137 137
         $post = $postFilter->getValues() + [
138 138
                 'featured_img' => $this->upload->uploadImage($data, 'featured_img'),
139 139
                 'main_img'     => $this->upload->uploadImage($data, 'main_img'),
140
-                'article_uuid' => $article['article_uuid'],
140
+                'article_uuid' => $article[ 'article_uuid' ],
141 141
             ];
142 142
 
143 143
         $this->articleMapper->insert($article);
@@ -154,31 +154,31 @@  discard block
 block discarded – undo
154 154
             throw new FilterException($articleFilter->getMessages() + $postFilter->getMessages());
155 155
         }
156 156
 
157
-        $article = $articleFilter->getValues() + ['article_uuid' => $oldArticle->article_uuid];
157
+        $article = $articleFilter->getValues() + [ 'article_uuid' => $oldArticle->article_uuid ];
158 158
         $post = $postFilter->getValues() + [
159 159
                 'featured_img' => $this->upload->uploadImage($data, 'featured_img'),
160 160
                 'main_img'     => $this->upload->uploadImage($data, 'main_img'),
161 161
             ];
162 162
 
163
-        $article['admin_user_uuid'] = $this->adminUsersMapper->getUuid($article['admin_user_id']);
164
-        $article['category_uuid'] = $this->categoryMapper->get($article['category_id'])->category_uuid;
165
-        unset($article['category_id'], $article['admin_user_id']);
163
+        $article[ 'admin_user_uuid' ] = $this->adminUsersMapper->getUuid($article[ 'admin_user_id' ]);
164
+        $article[ 'category_uuid' ] = $this->categoryMapper->get($article[ 'category_id' ])->category_uuid;
165
+        unset($article[ 'category_id' ], $article[ 'admin_user_id' ]);
166 166
 
167 167
         // We don't want to force user to re-upload image on edit
168
-        if (!$post['featured_img']) {
169
-            unset($post['featured_img']);
168
+        if (!$post[ 'featured_img' ]) {
169
+            unset($post[ 'featured_img' ]);
170 170
         } else {
171 171
             $this->upload->deleteFile($oldArticle->featured_img);
172 172
         }
173 173
 
174
-        if (!$post['main_img']) {
175
-            unset($post['main_img']);
174
+        if (!$post[ 'main_img' ]) {
175
+            unset($post[ 'main_img' ]);
176 176
         } else {
177 177
             $this->upload->deleteFile($oldArticle->main_img);
178 178
         }
179 179
 
180
-        $this->articleMapper->update($article, ['article_uuid' => $article['article_uuid']]);
181
-        $this->articlePostsMapper->update($post, ['article_uuid' => $article['article_uuid']]);
180
+        $this->articleMapper->update($article, [ 'article_uuid' => $article[ 'article_uuid' ] ]);
181
+        $this->articlePostsMapper->update($post, [ 'article_uuid' => $article[ 'article_uuid' ] ]);
182 182
     }
183 183
 
184 184
     public function deleteArticle($id)
@@ -191,7 +191,7 @@  discard block
 block discarded – undo
191 191
 
192 192
         $this->upload->deleteFile($post->main_img);
193 193
         $this->upload->deleteFile($post->featured_img);
194
-        $this->articlePostsMapper->delete(['article_uuid' => $post->article_uuid]);
194
+        $this->articlePostsMapper->delete([ 'article_uuid' => $post->article_uuid ]);
195 195
         $this->delete($post->article_uuid);
196 196
     }
197 197
 
Please login to merge, or discard this patch.
packages/Article/src/Service/VideoService.php 2 patches
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -91,6 +91,9 @@
 block discarded – undo
91 91
         return $this->getPagination($select, $page, $limit);
92 92
     }
93 93
 
94
+    /**
95
+     * @param integer $limit
96
+     */
94 97
     public function fetchLatest($limit)
95 98
     {
96 99
         return $this->articleVideosMapper->getLatest($limit);
Please login to merge, or discard this patch.
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types=1);
3
+declare(strict_types = 1);
4 4
 
5 5
 namespace Article\Service;
6 6
 
@@ -128,14 +128,14 @@  discard block
 block discarded – undo
128 128
 
129 129
         $article = $articleFilter->getValues();
130 130
         $article += [
131
-            'admin_user_uuid' => $this->adminUsersMapper->getUuid($article['admin_user_id']),
131
+            'admin_user_uuid' => $this->adminUsersMapper->getUuid($article[ 'admin_user_id' ]),
132 132
             'type'            => ArticleType::VIDEO,
133 133
             'article_id'      => $id,
134 134
             'article_uuid'    => $uuId,
135 135
         ];
136 136
 
137
-        $article['category_uuid'] = $this->categoryMapper->get($article['category_id'])->category_uuid;
138
-        unset($article['category_id'], $article['admin_user_id']);
137
+        $article[ 'category_uuid' ] = $this->categoryMapper->get($article[ 'category_id' ])->category_uuid;
138
+        unset($article[ 'category_id' ], $article[ 'admin_user_id' ]);
139 139
 
140 140
         $videos = $videosFilter->getValues() + [
141 141
                 'featured_img' => $this->upload->uploadImage($data, 'featured_img'),
@@ -157,11 +157,11 @@  discard block
 block discarded – undo
157 157
             throw new FilterException($articleFilter->getMessages() + $videosFilter->getMessages());
158 158
         }
159 159
 
160
-        $article = $articleFilter->getValues() + ['article_uuid' => $oldArticle->article_uuid];
161
-        $article['category_uuid'] = $this->categoryMapper->get($article['category_id'])->category_uuid;
162
-        $article['admin_user_uuid'] = $this->adminUsersMapper->getUuid($article['admin_user_id']);
163
-        unset($article['category_id']);
164
-        unset($article['admin_user_id']);
160
+        $article = $articleFilter->getValues() + [ 'article_uuid' => $oldArticle->article_uuid ];
161
+        $article[ 'category_uuid' ] = $this->categoryMapper->get($article[ 'category_id' ])->category_uuid;
162
+        $article[ 'admin_user_uuid' ] = $this->adminUsersMapper->getUuid($article[ 'admin_user_id' ]);
163
+        unset($article[ 'category_id' ]);
164
+        unset($article[ 'admin_user_id' ]);
165 165
 
166 166
         $videos = $videosFilter->getValues() + [
167 167
                 'featured_img' => $this->upload->uploadImage($data, 'featured_img'),
@@ -169,20 +169,20 @@  discard block
 block discarded – undo
169 169
             ];
170 170
 
171 171
         // We don't want to force user to re-upload image on edit
172
-        if (!$videos['featured_img']) {
173
-            unset($videos['featured_img']);
172
+        if (!$videos[ 'featured_img' ]) {
173
+            unset($videos[ 'featured_img' ]);
174 174
         } else {
175 175
             $this->upload->deleteFile($oldArticle->featured_img);
176 176
         }
177 177
 
178
-        if (!$videos['main_img']) {
179
-            unset($videos['main_img']);
178
+        if (!$videos[ 'main_img' ]) {
179
+            unset($videos[ 'main_img' ]);
180 180
         } else {
181 181
             $this->upload->deleteFile($oldArticle->main_img);
182 182
         }
183 183
 
184
-        $this->articleMapper->update($article, ['article_uuid' => $article['article_uuid']]);
185
-        $this->articleVideosMapper->update($videos, ['article_uuid' => $article['article_uuid']]);
184
+        $this->articleMapper->update($article, [ 'article_uuid' => $article[ 'article_uuid' ] ]);
185
+        $this->articleVideosMapper->update($videos, [ 'article_uuid' => $article[ 'article_uuid' ] ]);
186 186
     }
187 187
 
188 188
     public function deleteArticle($id)
@@ -195,7 +195,7 @@  discard block
 block discarded – undo
195 195
 
196 196
         $this->upload->deleteFile($video->main_img);
197 197
         $this->upload->deleteFile($video->featured_img);
198
-        $this->articleVideosMapper->delete(['article_uuid' => $video->article_uuid]);
198
+        $this->articleVideosMapper->delete([ 'article_uuid' => $video->article_uuid ]);
199 199
         $this->delete($video->article_uuid);
200 200
     }
201 201
 }
Please login to merge, or discard this patch.
packages/Category/src/Mapper/CategoryMapper.php 2 patches
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -44,6 +44,9 @@
 block discarded – undo
44 44
         return $select;
45 45
     }
46 46
 
47
+    /**
48
+     * @param integer $limit
49
+     */
47 50
     public function getCategoryPostsSelect($categoryId = null, $limit = null)
48 51
     {
49 52
         $select = $this->getSql()->select()
Please login to merge, or discard this patch.
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types=1);
3
+declare(strict_types = 1);
4 4
 
5 5
 namespace Category\Mapper;
6 6
 
@@ -32,12 +32,12 @@  discard block
 block discarded – undo
32 32
 
33 33
     public function get($id)
34 34
     {
35
-        return $this->select(['category_id' => $id])->current();
35
+        return $this->select([ 'category_id' => $id ])->current();
36 36
     }
37 37
 
38 38
     public function getPaginationSelect()
39 39
     {
40
-        $select = $this->getSql()->select()->order(['name' => 'asc']);
40
+        $select = $this->getSql()->select()->order([ 'name' => 'asc' ]);
41 41
 
42 42
         return $select;
43 43
     }
@@ -45,17 +45,17 @@  discard block
 block discarded – undo
45 45
     public function getCategoryPostsSelect($categoryId = null, $limit = null)
46 46
     {
47 47
         $select = $this->getSql()->select()
48
-            ->columns(['category_name' => 'name', 'category_slug' => 'slug'])
48
+            ->columns([ 'category_name' => 'name', 'category_slug' => 'slug' ])
49 49
             ->join('articles', 'articles.category_uuid = category.category_uuid',
50
-                ['article_id', 'slug', 'admin_user_uuid', 'published_at']
51
-            )->join('article_posts', 'article_posts.article_uuid = articles.article_uuid', ['*'], 'right')
50
+                [ 'article_id', 'slug', 'admin_user_uuid', 'published_at' ]
51
+            )->join('article_posts', 'article_posts.article_uuid = articles.article_uuid', [ '*' ], 'right')
52 52
             ->join('admin_users', 'admin_users.admin_user_uuid = articles.admin_user_uuid',
53
-                ['admin_user_id', 'first_name', 'last_name', 'face_img']
54
-            )->where(['articles.status' => 1])
55
-            ->order(['published_at' => 'desc']);
53
+                [ 'admin_user_id', 'first_name', 'last_name', 'face_img' ]
54
+            )->where([ 'articles.status' => 1 ])
55
+            ->order([ 'published_at' => 'desc' ]);
56 56
 
57 57
         if ($categoryId) {
58
-            $select->where(['category_id' => $categoryId]);
58
+            $select->where([ 'category_id' => $categoryId ]);
59 59
         }
60 60
 
61 61
         if ($limit) {
@@ -81,14 +81,14 @@  discard block
 block discarded – undo
81 81
         $inHomepage = null,
82 82
         $inCategoryList = null
83 83
     ) {
84
-        $select = $this->getSql()->select()->where(['type' => ArticleType::POST]);
84
+        $select = $this->getSql()->select()->where([ 'type' => ArticleType::POST ]);
85 85
 
86 86
         if ($inHomepage !== null) {
87
-            $select->where(['is_in_homepage' => $inHomepage]);
87
+            $select->where([ 'is_in_homepage' => $inHomepage ]);
88 88
         }
89 89
 
90 90
         if ($inCategoryList !== null) {
91
-            $select->where(['is_in_category_list' => $inCategoryList]);
91
+            $select->where([ 'is_in_category_list' => $inCategoryList ]);
92 92
         }
93 93
 
94 94
         if ($limit) {
Please login to merge, or discard this patch.
packages/Article/src/Mapper/ArticleMapper.php 2 patches
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -1,5 +1,5 @@  discard block
 block discarded – undo
1 1
 <?php
2
-declare(strict_types=1);
2
+declare(strict_types = 1);
3 3
 
4 4
 namespace Article\Mapper;
5 5
 
@@ -45,9 +45,9 @@  discard block
 block discarded – undo
45 45
     public function getCategories($articleId)
46 46
     {
47 47
         $select = $this->getSql()->select()
48
-            ->columns([])
49
-            ->join('category', 'category.category_uuid = articles.category_uuid', ['name', 'slug', 'category_id'])
50
-            ->where(['articles.article_id' => $articleId]);
48
+            ->columns([ ])
49
+            ->join('category', 'category.category_uuid = articles.category_uuid', [ 'name', 'slug', 'category_id' ])
50
+            ->where([ 'articles.article_id' => $articleId ]);
51 51
 
52 52
         return $this->selectWith($select);
53 53
     }
Please login to merge, or discard this patch.
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -7,7 +7,6 @@
 block discarded – undo
7 7
 use Zend\Db\Adapter\Adapter;
8 8
 use Zend\Db\Adapter\AdapterAwareInterface;
9 9
 use Zend\Db\Sql\Delete;
10
-use Zend\Db\Sql\Insert;
11 10
 use Zend\Db\TableGateway\AbstractTableGateway;
12 11
 
13 12
 /**
Please login to merge, or discard this patch.
packages/Article/src/Factory/Service/VideoServiceFactory.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -1,5 +1,5 @@  discard block
 block discarded – undo
1 1
 <?php
2
-declare(strict_types=1);
2
+declare(strict_types = 1);
3 3
 
4 4
 namespace Article\Factory\Service;
5 5
 
@@ -21,8 +21,8 @@  discard block
 block discarded – undo
21 21
      */
22 22
     public function __invoke(ContainerInterface $container): VideoService
23 23
     {
24
-        $config = $container->get('config')['upload'];
25
-        $upload = new Upload($config['public_path'], $config['non_public_path']);
24
+        $config = $container->get('config')[ 'upload' ];
25
+        $upload = new Upload($config[ 'public_path' ], $config[ 'non_public_path' ]);
26 26
 
27 27
         return new VideoService(
28 28
             $container->get(ArticleMapper::class),
Please login to merge, or discard this patch.
packages/Article/src/Factory/Service/PostServiceFactory.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -1,5 +1,5 @@  discard block
 block discarded – undo
1 1
 <?php
2
-declare(strict_types=1);
2
+declare(strict_types = 1);
3 3
 
4 4
 namespace Article\Factory\Service;
5 5
 
@@ -20,8 +20,8 @@  discard block
 block discarded – undo
20 20
      */
21 21
     public function __invoke(ContainerInterface $container): PostService
22 22
     {
23
-        $config = $container->get('config')['upload'];
24
-        $upload = new Upload($config['public_path'], $config['non_public_path']);
23
+        $config = $container->get('config')[ 'upload' ];
24
+        $upload = new Upload($config[ 'public_path' ], $config[ 'non_public_path' ]);
25 25
 
26 26
         return new PostService(
27 27
             $container->get(ArticleMapper::class),
Please login to merge, or discard this patch.
packages/Article/src/Factory/Controller/DiscussionFactory.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -1,5 +1,5 @@
 block discarded – undo
1 1
 <?php
2
-declare(strict_types=1);
2
+declare(strict_types = 1);
3 3
 
4 4
 namespace Article\Factory\Controller;
5 5
 
Please login to merge, or discard this patch.
packages/Newsletter/src/Service/NewsletterServiceFactory.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types=1);
3
+declare(strict_types = 1);
4 4
 
5 5
 namespace Newsletter\Service;
6 6
 
Please login to merge, or discard this patch.