Completed
Pull Request — develop (#1492)
by Zack
08:22
created
php_codesniffer/src/Standards/Squiz/Tests/PHP/CommentedOutCodeUnitTest.php 1 patch
Switch Indentation   +26 added lines, -26 removed lines patch added patch discarded remove patch
@@ -43,32 +43,32 @@
 block discarded – undo
43 43
     public function getWarningList($testFile='CommentedOutCodeUnitTest.inc')
44 44
     {
45 45
         switch ($testFile) {
46
-        case 'CommentedOutCodeUnitTest.inc':
47
-            return [
48
-                6   => 1,
49
-                8   => 1,
50
-                15  => 1,
51
-                19  => 1,
52
-                35  => 1,
53
-                87  => 1,
54
-                91  => 1,
55
-                97  => 1,
56
-                109 => 1,
57
-                116 => 1,
58
-                128 => 1,
59
-                147 => 1,
60
-                158 => 1,
61
-            ];
62
-            break;
63
-        case 'CommentedOutCodeUnitTest.css':
64
-            return [
65
-                7  => 1,
66
-                16 => 1,
67
-            ];
68
-            break;
69
-        default:
70
-            return [];
71
-            break;
46
+        	case 'CommentedOutCodeUnitTest.inc':
47
+            	return [
48
+                	6   => 1,
49
+                	8   => 1,
50
+                	15  => 1,
51
+                	19  => 1,
52
+                	35  => 1,
53
+                	87  => 1,
54
+                	91  => 1,
55
+                	97  => 1,
56
+                	109 => 1,
57
+                	116 => 1,
58
+                	128 => 1,
59
+                	147 => 1,
60
+                	158 => 1,
61
+            	];
62
+            	break;
63
+        	case 'CommentedOutCodeUnitTest.css':
64
+            	return [
65
+                	7  => 1,
66
+                	16 => 1,
67
+            	];
68
+            	break;
69
+        	default:
70
+            	return [];
71
+            	break;
72 72
         }//end switch
73 73
 
74 74
     }//end getWarningList()
Please login to merge, or discard this patch.
php_codesniffer/src/Standards/Squiz/Tests/Files/FileExtensionUnitTest.php 1 patch
Switch Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -28,10 +28,10 @@
 block discarded – undo
28 28
     public function getErrorList($testFile='')
29 29
     {
30 30
         switch ($testFile) {
31
-        case 'FileExtensionUnitTest.1.inc':
32
-            return [1 => 1];
33
-        default:
34
-            return [];
31
+        	case 'FileExtensionUnitTest.1.inc':
32
+            	return [1 => 1];
33
+        	default:
34
+            	return [];
35 35
         }
36 36
 
37 37
     }//end getErrorList()
Please login to merge, or discard this patch.
src/Standards/Squiz/Tests/Operators/ComparisonOperatorUsageUnitTest.php 1 patch
Switch Indentation   +50 added lines, -50 removed lines patch added patch discarded remove patch
@@ -28,56 +28,56 @@
 block discarded – undo
28 28
     public function getErrorList($testFile='ComparisonOperatorUsageUnitTest.inc')
29 29
     {
30 30
         switch ($testFile) {
31
-        case 'ComparisonOperatorUsageUnitTest.inc':
32
-            return [
33
-                6   => 1,
34
-                7   => 1,
35
-                10  => 1,
36
-                11  => 1,
37
-                18  => 1,
38
-                19  => 1,
39
-                22  => 1,
40
-                23  => 1,
41
-                29  => 2,
42
-                32  => 2,
43
-                38  => 4,
44
-                47  => 2,
45
-                69  => 1,
46
-                72  => 1,
47
-                75  => 1,
48
-                78  => 1,
49
-                80  => 1,
50
-                82  => 1,
51
-                83  => 1,
52
-                89  => 1,
53
-                92  => 1,
54
-                100 => 1,
55
-                106 => 1,
56
-                112 => 1,
57
-                123 => 1,
58
-                127 => 1,
59
-                131 => 1,
60
-                135 => 1,
61
-            ];
62
-            break;
63
-        case 'ComparisonOperatorUsageUnitTest.js':
64
-            return [
65
-                5  => 1,
66
-                6  => 1,
67
-                17 => 1,
68
-                18 => 1,
69
-                28 => 2,
70
-                40 => 1,
71
-                47 => 1,
72
-                52 => 1,
73
-                63 => 1,
74
-                67 => 1,
75
-                71 => 1,
76
-            ];
77
-            break;
78
-        default:
79
-            return [];
80
-            break;
31
+        	case 'ComparisonOperatorUsageUnitTest.inc':
32
+            	return [
33
+                	6   => 1,
34
+                	7   => 1,
35
+                	10  => 1,
36
+                	11  => 1,
37
+                	18  => 1,
38
+                	19  => 1,
39
+                	22  => 1,
40
+                	23  => 1,
41
+                	29  => 2,
42
+                	32  => 2,
43
+                	38  => 4,
44
+                	47  => 2,
45
+                	69  => 1,
46
+                	72  => 1,
47
+                	75  => 1,
48
+                	78  => 1,
49
+                	80  => 1,
50
+                	82  => 1,
51
+                	83  => 1,
52
+                	89  => 1,
53
+                	92  => 1,
54
+                	100 => 1,
55
+                	106 => 1,
56
+                	112 => 1,
57
+                	123 => 1,
58
+                	127 => 1,
59
+                	131 => 1,
60
+                	135 => 1,
61
+            	];
62
+            	break;
63
+        	case 'ComparisonOperatorUsageUnitTest.js':
64
+            	return [
65
+                	5  => 1,
66
+                	6  => 1,
67
+                	17 => 1,
68
+                	18 => 1,
69
+                	28 => 2,
70
+                	40 => 1,
71
+                	47 => 1,
72
+                	52 => 1,
73
+                	63 => 1,
74
+                	67 => 1,
75
+                	71 => 1,
76
+            	];
77
+            	break;
78
+        	default:
79
+            	return [];
80
+            	break;
81 81
         }//end switch
82 82
 
83 83
     }//end getErrorList()
Please login to merge, or discard this patch.
src/Standards/Squiz/Tests/Commenting/InlineCommentUnitTest.php 1 patch
Switch Indentation   +39 added lines, -39 removed lines patch added patch discarded remove patch
@@ -28,46 +28,46 @@
 block discarded – undo
28 28
     public function getErrorList($testFile='InlineCommentUnitTest.inc')
29 29
     {
30 30
         switch ($testFile) {
31
-        case 'InlineCommentUnitTest.inc':
32
-            $errors = [
33
-                17  => 1,
34
-                27  => 1,
35
-                28  => 1,
36
-                32  => 2,
37
-                36  => 1,
38
-                44  => 2,
39
-                58  => 1,
40
-                61  => 1,
41
-                64  => 1,
42
-                67  => 1,
43
-                95  => 1,
44
-                96  => 1,
45
-                97  => 3,
46
-                118 => 1,
47
-                126 => 2,
48
-                130 => 2,
49
-                149 => 1,
50
-            ];
31
+        	case 'InlineCommentUnitTest.inc':
32
+            	$errors = [
33
+                	17  => 1,
34
+                	27  => 1,
35
+                	28  => 1,
36
+                	32  => 2,
37
+                	36  => 1,
38
+                	44  => 2,
39
+                	58  => 1,
40
+                	61  => 1,
41
+                	64  => 1,
42
+                	67  => 1,
43
+                	95  => 1,
44
+                	96  => 1,
45
+                	97  => 3,
46
+                	118 => 1,
47
+                	126 => 2,
48
+                	130 => 2,
49
+                	149 => 1,
50
+            	];
51 51
 
52
-            return $errors;
53
-        case 'InlineCommentUnitTest.js':
54
-            return [
55
-                31  => 1,
56
-                36  => 2,
57
-                48  => 1,
58
-                51  => 1,
59
-                54  => 1,
60
-                57  => 1,
61
-                102 => 1,
62
-                103 => 1,
63
-                104 => 3,
64
-                118 => 1,
65
-                121 => 1,
66
-                125 => 2,
67
-                129 => 2,
68
-            ];
69
-        default:
70
-            return [];
52
+            	return $errors;
53
+        	case 'InlineCommentUnitTest.js':
54
+            	return [
55
+                	31  => 1,
56
+                	36  => 2,
57
+                	48  => 1,
58
+                	51  => 1,
59
+                	54  => 1,
60
+                	57  => 1,
61
+                	102 => 1,
62
+                	103 => 1,
63
+                	104 => 3,
64
+                	118 => 1,
65
+                	121 => 1,
66
+                	125 => 2,
67
+                	129 => 2,
68
+            	];
69
+        	default:
70
+            	return [];
71 71
         }//end switch
72 72
 
73 73
     }//end getErrorList()
Please login to merge, or discard this patch.
Standards/Squiz/Tests/Commenting/LongConditionClosingCommentUnitTest.inc 1 patch
Switch Indentation   +159 added lines, -159 removed lines patch added patch discarded remove patch
@@ -493,41 +493,41 @@  discard block
 block discarded – undo
493 493
 }
494 494
 
495 495
 switch ($something) {
496
-    case '1':
497
-    // Line 1
498
-    // Line 2
499
-    // Line 3
500
-    // Line 4
501
-    // Line 5
502
-    break;
503
-    case '2':
504
-    // Line 1
505
-    // Line 2
506
-    // Line 3
507
-    // Line 4
508
-    // Line 5
509
-    break;
510
-    case '3':
511
-    // Line 1
512
-    // Line 2
513
-    // Line 3
514
-    // Line 4
515
-    // Line 5
516
-    break;
517
-    case '4':
518
-    // Line 1
519
-    // Line 2
520
-    // Line 3
521
-    // Line 4
522
-    // Line 5
523
-    break;
524
-    case '5':
525
-    // Line 1
526
-    // Line 2
527
-    // Line 3
528
-    // Line 4
529
-    // Line 5
530
-    break;
496
+    	case '1':
497
+    	// Line 1
498
+    	// Line 2
499
+    	// Line 3
500
+    	// Line 4
501
+    	// Line 5
502
+    	break;
503
+    	case '2':
504
+    	// Line 1
505
+    	// Line 2
506
+    	// Line 3
507
+    	// Line 4
508
+    	// Line 5
509
+    	break;
510
+    	case '3':
511
+    	// Line 1
512
+    	// Line 2
513
+    	// Line 3
514
+    	// Line 4
515
+    	// Line 5
516
+    	break;
517
+    	case '4':
518
+    	// Line 1
519
+    	// Line 2
520
+    	// Line 3
521
+    	// Line 4
522
+    	// Line 5
523
+    	break;
524
+    	case '5':
525
+    	// Line 1
526
+    	// Line 2
527
+    	// Line 3
528
+    	// Line 4
529
+    	// Line 5
530
+    	break;
531 531
 }
532 532
 
533 533
 // Wrong comment
@@ -629,69 +629,69 @@  discard block
 block discarded – undo
629 629
 }
630 630
 
631 631
 switch ($foo) {
632
-    case 'one' : {
633
-        // some code here.
634
-        // some code here.
635
-        // some code here.
636
-        // some code here.
637
-        // some code here.
638
-        // some code here.
639
-        // some code here.
640
-        // some code here.
641
-        // some code here.
642
-        // some code here.
643
-        // some code here.
644
-        // some code here.
645
-        // some code here.
646
-        // some code here.
647
-        // some code here.
648
-        // some code here.
649
-        // some code here.
650
-        // some code here.
651
-        // some code here.
652
-        // some code here.
653
-        // some code here.
654
-        // some code here.
655
-        // some code here.
656
-        // some code here.
657
-        // some code here.
658
-        // some code here.
659
-        // some code here.
660
-        // some code here.
661
-        // some code here.
662
-        break;
663
-    }
664
-    case 'one' :
665
-        // some code here.
666
-        // some code here.
667
-        // some code here.
668
-        // some code here.
669
-        // some code here.
670
-        // some code here.
671
-        // some code here.
672
-        // some code here.
673
-        // some code here.
674
-        // some code here.
675
-        // some code here.
676
-        // some code here.
677
-        // some code here.
678
-        // some code here.
679
-        // some code here.
680
-        // some code here.
681
-        // some code here.
682
-        // some code here.
683
-        // some code here.
684
-        // some code here.
685
-        // some code here.
686
-        // some code here.
687
-        // some code here.
688
-        // some code here.
689
-        // some code here.
690
-        // some code here.
691
-        // some code here.
692
-        // some code here.
693
-        // some code here.
694
-    break;
632
+    	case 'one' : {
633
+        	// some code here.
634
+        	// some code here.
635
+        	// some code here.
636
+        	// some code here.
637
+        	// some code here.
638
+        	// some code here.
639
+        	// some code here.
640
+        	// some code here.
641
+        	// some code here.
642
+        	// some code here.
643
+        	// some code here.
644
+        	// some code here.
645
+        	// some code here.
646
+        	// some code here.
647
+        	// some code here.
648
+        	// some code here.
649
+        	// some code here.
650
+        	// some code here.
651
+        	// some code here.
652
+        	// some code here.
653
+        	// some code here.
654
+        	// some code here.
655
+        	// some code here.
656
+        	// some code here.
657
+        	// some code here.
658
+        	// some code here.
659
+        	// some code here.
660
+        	// some code here.
661
+        	// some code here.
662
+        	break;
663
+    	}
664
+    	case 'one' :
665
+        	// some code here.
666
+        	// some code here.
667
+        	// some code here.
668
+        	// some code here.
669
+        	// some code here.
670
+        	// some code here.
671
+        	// some code here.
672
+        	// some code here.
673
+        	// some code here.
674
+        	// some code here.
675
+        	// some code here.
676
+        	// some code here.
677
+        	// some code here.
678
+        	// some code here.
679
+        	// some code here.
680
+        	// some code here.
681
+        	// some code here.
682
+        	// some code here.
683
+        	// some code here.
684
+        	// some code here.
685
+        	// some code here.
686
+        	// some code here.
687
+        	// some code here.
688
+        	// some code here.
689
+        	// some code here.
690
+        	// some code here.
691
+        	// some code here.
692
+        	// some code here.
693
+        	// some code here.
694
+    	break;
695 695
 }//end switch
696 696
 
697 697
 // Yes, code like this does exist.
@@ -701,67 +701,67 @@  discard block
 block discarded – undo
701 701
     return $bar;
702 702
 
703 703
 switch ($foo) {
704
-    case 1:
705
-    case 2:
706
-    case 3:
707
-    case 4:
708
-    case 5:
709
-    case 6:
710
-    case 7:
711
-    case 8:
712
-    case 9:
713
-    case 10:
714
-    case 11:
715
-    case 12:
716
-    case 13:
717
-    case 14:
718
-    case 15:
719
-    case 16:
720
-    case 17:
721
-    case 18:
722
-    case 19:
723
-    case 20:
724
-    case 21:
725
-    case 22:
726
-    case 23:
727
-    case 24:
728
-    case 25:
729
-    case 26:
730
-    case 27:
731
-    case 28:
732
-    case 29:
733
-    case 30:
734
-    case 31:
735
-    case 32:
736
-    case 33:
737
-    case 34:
738
-    case 35:
739
-    case 36:
740
-    case 37:
741
-    case 38:
742
-    case 39:
743
-    case 40:
744
-    case 41:
745
-    case 42:
746
-    case 43:
747
-    case 44:
748
-    case 45:
749
-    case 46:
750
-    case 47:
751
-    case 48:
752
-    case 49:
753
-    case 50:
754
-    case 51:
755
-    case 52:
756
-    case 53:
757
-    case 54:
758
-    case 55:
759
-    case 56:
760
-    case 57:
761
-    case 58:
762
-    case 59:
763
-        echo $foo;
764
-        break;
704
+    	case 1:
705
+    	case 2:
706
+    	case 3:
707
+    	case 4:
708
+    	case 5:
709
+    	case 6:
710
+    	case 7:
711
+    	case 8:
712
+    	case 9:
713
+    	case 10:
714
+    	case 11:
715
+    	case 12:
716
+    	case 13:
717
+    	case 14:
718
+    	case 15:
719
+    	case 16:
720
+    	case 17:
721
+    	case 18:
722
+    	case 19:
723
+    	case 20:
724
+    	case 21:
725
+    	case 22:
726
+    	case 23:
727
+    	case 24:
728
+    	case 25:
729
+    	case 26:
730
+    	case 27:
731
+    	case 28:
732
+    	case 29:
733
+    	case 30:
734
+    	case 31:
735
+    	case 32:
736
+    	case 33:
737
+    	case 34:
738
+    	case 35:
739
+    	case 36:
740
+    	case 37:
741
+    	case 38:
742
+    	case 39:
743
+    	case 40:
744
+    	case 41:
745
+    	case 42:
746
+    	case 43:
747
+    	case 44:
748
+    	case 45:
749
+    	case 46:
750
+    	case 47:
751
+    	case 48:
752
+    	case 49:
753
+    	case 50:
754
+    	case 51:
755
+    	case 52:
756
+    	case 53:
757
+    	case 54:
758
+    	case 55:
759
+    	case 56:
760
+    	case 57:
761
+    	case 58:
762
+    	case 59:
763
+        	echo $foo;
764
+        	break;
765 765
 }
766 766
 
767 767
 // phpcs:set Squiz.Commenting.LongConditionClosingComment lineLimit 5
Please login to merge, or discard this patch.
src/Standards/Squiz/Tests/Commenting/PostStatementCommentUnitTest.php 1 patch
Switch Indentation   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -28,23 +28,23 @@
 block discarded – undo
28 28
     public function getErrorList($testFile='PostStatementCommentUnitTest.inc')
29 29
     {
30 30
         switch ($testFile) {
31
-        case 'PostStatementCommentUnitTest.inc':
32
-            return [
33
-                6  => 1,
34
-                10 => 1,
35
-                18 => 1,
36
-                35 => 1,
37
-            ];
31
+        	case 'PostStatementCommentUnitTest.inc':
32
+            	return [
33
+                	6  => 1,
34
+                	10 => 1,
35
+                	18 => 1,
36
+                	35 => 1,
37
+            	];
38 38
 
39
-        case 'PostStatementCommentUnitTest.1.js':
40
-            return [
41
-                1 => 1,
42
-                4 => 1,
43
-                9 => 1,
44
-            ];
39
+        	case 'PostStatementCommentUnitTest.1.js':
40
+            	return [
41
+                	1 => 1,
42
+                	4 => 1,
43
+                	9 => 1,
44
+            	];
45 45
 
46
-        default:
47
-            return [];
46
+        	default:
47
+            	return [];
48 48
         }//end switch
49 49
 
50 50
     }//end getErrorList()
Please login to merge, or discard this patch.
src/Standards/Squiz/Tests/Commenting/FileCommentUnitTest.3.inc 1 patch
Switch Indentation   -8 removed lines patch added patch discarded remove patch
@@ -1,9 +1,1 @@
 block discarded – undo
1 1
 <?php
2
-/**
3
- * File comment.
4
- *
5
- * @package    Package
6
- * @subpackage Subpackage
7
- * @author     Squiz Pty Ltd <[email protected]>
8
- * @copyright  2010-2014 Squiz Pty Ltd (ABN 77 084 670 600)
9
- *
Please login to merge, or discard this patch.
src/Standards/Squiz/Tests/Commenting/FileCommentUnitTest.php 1 patch
Switch Indentation   +20 added lines, -20 removed lines patch added patch discarded remove patch
@@ -28,29 +28,29 @@
 block discarded – undo
28 28
     public function getErrorList($testFile='FileCommentUnitTest.inc')
29 29
     {
30 30
         switch ($testFile) {
31
-        case 'FileCommentUnitTest.1.inc':
32
-        case 'FileCommentUnitTest.1.js':
33
-            return [
34
-                1  => 1,
35
-                22 => 2,
36
-                23 => 1,
37
-                24 => 2,
38
-                25 => 2,
39
-                26 => 1,
40
-                27 => 2,
41
-                28 => 2,
42
-                32 => 2,
43
-            ];
31
+        	case 'FileCommentUnitTest.1.inc':
32
+        	case 'FileCommentUnitTest.1.js':
33
+            	return [
34
+                	1  => 1,
35
+                	22 => 2,
36
+                	23 => 1,
37
+                	24 => 2,
38
+                	25 => 2,
39
+                	26 => 1,
40
+                	27 => 2,
41
+                	28 => 2,
42
+                	32 => 2,
43
+            	];
44 44
 
45
-        case 'FileCommentUnitTest.4.inc':
46
-        case 'FileCommentUnitTest.6.inc':
47
-            return [1 => 1];
45
+        	case 'FileCommentUnitTest.4.inc':
46
+        	case 'FileCommentUnitTest.6.inc':
47
+            	return [1 => 1];
48 48
 
49
-        case 'FileCommentUnitTest.5.inc':
50
-            return [2 => 1];
49
+        	case 'FileCommentUnitTest.5.inc':
50
+            	return [2 => 1];
51 51
 
52
-        default:
53
-            return [];
52
+        	default:
53
+            	return [];
54 54
         }//end switch
55 55
 
56 56
     }//end getErrorList()
Please login to merge, or discard this patch.
Standards/Squiz/Tests/Commenting/LongConditionClosingCommentUnitTest.php 1 patch
Switch Indentation   +49 added lines, -49 removed lines patch added patch discarded remove patch
@@ -28,55 +28,55 @@
 block discarded – undo
28 28
     public function getErrorList($testFile='LongConditionClosingCommentUnitTest.inc')
29 29
     {
30 30
         switch ($testFile) {
31
-        case 'LongConditionClosingCommentUnitTest.inc':
32
-            return [
33
-                49  => 1,
34
-                99  => 1,
35
-                146 => 1,
36
-                192 => 1,
37
-                215 => 1,
38
-                238 => 1,
39
-                261 => 1,
40
-                286 => 1,
41
-                309 => 1,
42
-                332 => 1,
43
-                355 => 1,
44
-                378 => 1,
45
-                493 => 1,
46
-                531 => 1,
47
-                536 => 1,
48
-                540 => 1,
49
-                562 => 1,
50
-                601 => 1,
51
-                629 => 1,
52
-                663 => 1,
53
-                765 => 1,
54
-                798 => 1,
55
-                811 => 1,
56
-                897 => 1,
57
-                931 => 1,
58
-                962 => 1,
59
-            ];
60
-            break;
61
-        case 'LongConditionClosingCommentUnitTest.js':
62
-            return [
63
-                47  => 1,
64
-                97  => 1,
65
-                144 => 1,
66
-                190 => 1,
67
-                213 => 1,
68
-                238 => 1,
69
-                261 => 1,
70
-                284 => 1,
71
-                307 => 1,
72
-                401 => 1,
73
-                439 => 1,
74
-                444 => 1,
75
-            ];
76
-            break;
77
-        default:
78
-            return [];
79
-            break;
31
+        	case 'LongConditionClosingCommentUnitTest.inc':
32
+            	return [
33
+                	49  => 1,
34
+                	99  => 1,
35
+                	146 => 1,
36
+                	192 => 1,
37
+                	215 => 1,
38
+                	238 => 1,
39
+                	261 => 1,
40
+                	286 => 1,
41
+                	309 => 1,
42
+                	332 => 1,
43
+                	355 => 1,
44
+                	378 => 1,
45
+                	493 => 1,
46
+                	531 => 1,
47
+                	536 => 1,
48
+                	540 => 1,
49
+                	562 => 1,
50
+                	601 => 1,
51
+                	629 => 1,
52
+                	663 => 1,
53
+                	765 => 1,
54
+                	798 => 1,
55
+                	811 => 1,
56
+                	897 => 1,
57
+                	931 => 1,
58
+                	962 => 1,
59
+            	];
60
+            	break;
61
+        	case 'LongConditionClosingCommentUnitTest.js':
62
+            	return [
63
+                	47  => 1,
64
+                	97  => 1,
65
+                	144 => 1,
66
+                	190 => 1,
67
+                	213 => 1,
68
+                	238 => 1,
69
+                	261 => 1,
70
+                	284 => 1,
71
+                	307 => 1,
72
+                	401 => 1,
73
+                	439 => 1,
74
+                	444 => 1,
75
+            	];
76
+            	break;
77
+        	default:
78
+            	return [];
79
+            	break;
80 80
         }//end switch
81 81
 
82 82
     }//end getErrorList()
Please login to merge, or discard this patch.