Skip to content

Commit bfe69cd

Browse files
committed
Apply fixes from StyleCI
1 parent fd1cf17 commit bfe69cd

File tree

7 files changed

+19
-3
lines changed

7 files changed

+19
-3
lines changed

october/plugins/rainlab/blog/controllers/Categories.php

+1-1
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ public function index_onDelete()
3333
{
3434
if (($checkedIds = post('checked')) && is_array($checkedIds) && count($checkedIds)) {
3535
foreach ($checkedIds as $categoryId) {
36-
if ((!$category = Category::find($categoryId))) {
36+
if (!$category = Category::find($categoryId)) {
3737
continue;
3838
}
3939

october/plugins/rainlab/blog/formwidgets/BlogMarkdown.php

+2-2
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@ protected function checkUploadPostback()
9393
$response = Response::make()->setContent($result);
9494
$response->send();
9595

96-
exit();
96+
exit;
9797
} catch (Exception $ex) {
9898
$message = $uploadedFileName
9999
? Lang::get('cms::lang.asset.error_uploading_file', ['name' => $uploadedFileName, 'error' => $ex->getMessage()])
@@ -107,7 +107,7 @@ protected function checkUploadPostback()
107107
$response = Response::make()->setContent($result);
108108
$response->send();
109109

110-
exit();
110+
exit;
111111
}
112112
}
113113
}

october/tests/unit/cms/classes/CmsObjectTest.php

+6
Original file line numberDiff line numberDiff line change
@@ -150,6 +150,7 @@ public function testFillNotFillable()
150150

151151
/**
152152
* @expectedException \October\Rain\Exception\ValidationException
153+
*
153154
* @expectedExceptionMessage Invalid file name
154155
*/
155156
public function testFillInvalidFileNameSymbol()
@@ -166,6 +167,7 @@ public function testFillInvalidFileNameSymbol()
166167

167168
/**
168169
* @expectedException \October\Rain\Exception\ValidationException
170+
*
169171
* @expectedExceptionMessage Invalid file name
170172
*/
171173
public function testFillInvalidFileNamePath()
@@ -182,6 +184,7 @@ public function testFillInvalidFileNamePath()
182184

183185
/**
184186
* @expectedException \October\Rain\Exception\ValidationException
187+
*
185188
* @expectedExceptionMessage Invalid file name
186189
*/
187190
public function testFillInvalidFileSlash()
@@ -198,6 +201,7 @@ public function testFillInvalidFileSlash()
198201

199202
/**
200203
* @expectedException \October\Rain\Exception\ValidationException
204+
*
201205
* @expectedExceptionMessage The File Name field is required
202206
*/
203207
public function testFillEmptyFileName()
@@ -266,7 +270,9 @@ public function testRename()
266270

267271
/**
268272
* @depends testRename
273+
*
269274
* @expectedException \October\Rain\Exception\ApplicationException
275+
*
270276
* @expectedExceptionMessage already exists
271277
*/
272278
public function testRenameToExistingFile()

october/tests/unit/cms/classes/ControllerTest.php

+7
Original file line numberDiff line numberDiff line change
@@ -89,6 +89,7 @@ public function testRoot()
8989

9090
/**
9191
* @expectedException Cms\Classes\CmsException
92+
*
9293
* @expectedExceptionMessage is not found
9394
*/
9495
public function testLayoutNotFound()
@@ -148,6 +149,7 @@ public function testLayoutInSubdirectory()
148149

149150
/**
150151
* @expectedException Twig_Error_Runtime
152+
*
151153
* @expectedExceptionMessage is not found
152154
*/
153155
public function testPartialNotFound()
@@ -195,6 +197,7 @@ protected function configAjaxRequestMock($handler, $partials = false)
195197

196198
/**
197199
* @expectedException Cms\Classes\CmsException
200+
*
198201
* @expectedExceptionMessage AJAX handler 'onNoHandler' was not found.
199202
*/
200203
public function testAjaxHandlerNotFound()
@@ -208,6 +211,7 @@ public function testAjaxHandlerNotFound()
208211

209212
/**
210213
* @expectedException Cms\Classes\CmsException
214+
*
211215
* @expectedExceptionMessage Invalid AJAX handler name: delete.
212216
*/
213217
public function testAjaxInvalidHandlerName()
@@ -221,6 +225,7 @@ public function testAjaxInvalidHandlerName()
221225

222226
/**
223227
* @expectedException Cms\Classes\CmsException
228+
*
224229
* @expectedExceptionMessage Invalid partial name: p:artial.
225230
*/
226231
public function testAjaxInvalidPartial()
@@ -234,6 +239,7 @@ public function testAjaxInvalidPartial()
234239

235240
/**
236241
* @expectedException Cms\Classes\CmsException
242+
*
237243
* @expectedExceptionMessage The partial 'partial' is not found.
238244
*/
239245
public function testAjaxPartialNotFound()
@@ -372,6 +378,7 @@ public function testComponentAjax()
372378

373379
/**
374380
* @expectedException October\Rain\Exception\SystemException
381+
*
375382
* @expectedExceptionMessage is not registered for the component
376383
*/
377384
public function testComponentClassNotFound()

october/tests/unit/cms/classes/ThemeTest.php

+1
Original file line numberDiff line numberDiff line change
@@ -65,6 +65,7 @@ public function testGetActiveTheme()
6565

6666
/**
6767
* @expectedException \October\Rain\Exception\SystemException
68+
*
6869
* @expectedExceptionMessage The active theme is not set.
6970
*/
7071
public function testNoActiveTheme()

october/tests/unit/plugins/database/ModelTest.php

+1
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@ public function testCreateFirstPost()
2525

2626
/**
2727
* @expectedException \Illuminate\Database\Eloquent\MassAssignmentException
28+
*
2829
* @expectedExceptionMessage title
2930
*/
3031
public function testGuardedAttribute()

october/tests/unit/plugins/database/SimpleTreeModelTest.php

+1
Original file line numberDiff line numberDiff line change
@@ -166,6 +166,7 @@ public function testListsNested()
166166

167167
/**
168168
* @expectedException \Exception
169+
*
169170
* @expectedExceptionMessage Column mismatch in listsNested method
170171
*/
171172
public function testListsNestedUnknownColumn()

0 commit comments

Comments
 (0)