-
-
Save shivanshuag/9c87118e8791f6bac86a to your computer and use it in GitHub Desktop.
Securesite PHP_codesniffer output
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
~/webapps/drupal7 ⮀ ⮀ ⮀ drupalcs7 sites/all/modules/securesite ⮂ | |
FILE: .../shivanshu/webapps/drupal7/sites/all/modules/securesite/securesite.info | |
-------------------------------------------------------------------------------- | |
FOUND 2 ERROR(S) AFFECTING 2 LINE(S) | |
-------------------------------------------------------------------------------- | |
7 | ERROR | It's only necessary to declare files[] if they declare a class or | |
| | interface. | |
15 | ERROR | Files must end in a single new line character | |
-------------------------------------------------------------------------------- | |
UPGRADE TO PHP_CODESNIFFER 2.0 TO FIX ERRORS AUTOMATICALLY | |
-------------------------------------------------------------------------------- | |
FILE: ...anshu/webapps/drupal7/sites/all/modules/securesite/securesite.admin.inc | |
-------------------------------------------------------------------------------- | |
FOUND 10 ERROR(S) AND 4 WARNING(S) AFFECTING 11 LINE(S) | |
-------------------------------------------------------------------------------- | |
60 | ERROR | If the line declaring an array spans longer than 80 | |
| | characters, each element should be broken into its own line | |
129 | ERROR | Function comment short description must end with a full stop | |
139 | ERROR | If the line declaring an array spans longer than 80 | |
| | characters, each element should be broken into its own line | |
139 | WARNING | A comma should follow the last multiline array item. Found: ) | |
165 | ERROR | If the line declaring an array spans longer than 80 | |
| | characters, each element should be broken into its own line | |
167 | ERROR | If the line declaring an array spans longer than 80 | |
| | characters, each element should be broken into its own line | |
194 | ERROR | If the line declaring an array spans longer than 80 | |
| | characters, each element should be broken into its own line | |
218 | WARNING | A comma should follow the last multiline array item. Found: ) | |
239 | ERROR | Missing function doc comment | |
272 | WARNING | Line exceeds 80 characters; contains 111 characters | |
272 | ERROR | Comments may not appear after statements. | |
273 | WARNING | Line exceeds 80 characters; contains 109 characters | |
273 | ERROR | Comments may not appear after statements. | |
274 | ERROR | Comments may not appear after statements. | |
-------------------------------------------------------------------------------- | |
UPGRADE TO PHP_CODESNIFFER 2.0 TO FIX ERRORS AUTOMATICALLY | |
-------------------------------------------------------------------------------- | |
FILE: ...pps/drupal7/sites/all/modules/securesite/digest_md5/digest_md5.conf.php | |
-------------------------------------------------------------------------------- | |
FOUND 3 ERROR(S) AFFECTING 3 LINE(S) | |
-------------------------------------------------------------------------------- | |
41 | ERROR | Perl-style comments are not allowed; use "// Comment" instead | |
47 | ERROR | Perl-style comments are not allowed; use "// Comment" instead | |
137 | ERROR | Case breaking statements must be followed by a single blank line | |
-------------------------------------------------------------------------------- | |
UPGRADE TO PHP_CODESNIFFER 2.0 TO FIX ERRORS AUTOMATICALLY | |
-------------------------------------------------------------------------------- | |
FILE: .../webapps/drupal7/sites/all/modules/securesite/digest_md5/digest_md5.php | |
-------------------------------------------------------------------------------- | |
FOUND 85 ERROR(S) AFFECTING 42 LINE(S) | |
-------------------------------------------------------------------------------- | |
30 | ERROR | Concat operator must be surrounded by spaces | |
30 | ERROR | String concat is not required here; use a single string instead | |
30 | ERROR | Concat operator must be surrounded by spaces | |
31 | ERROR | Concat operator must be surrounded by spaces | |
32 | ERROR | Concat operator must be surrounded by spaces | |
32 | ERROR | Concat operator must be surrounded by spaces | |
33 | ERROR | Concat operator must be surrounded by spaces | |
33 | ERROR | Concat operator must be surrounded by spaces | |
34 | ERROR | Concat operator must be surrounded by spaces | |
34 | ERROR | Concat operator must be surrounded by spaces | |
35 | ERROR | Concat operator must be surrounded by spaces | |
35 | ERROR | Concat operator must be surrounded by spaces | |
36 | ERROR | Concat operator must be surrounded by spaces | |
36 | ERROR | Concat operator must be surrounded by spaces | |
37 | ERROR | Concat operator must be surrounded by spaces | |
37 | ERROR | Concat operator must be surrounded by spaces | |
38 | ERROR | Concat operator must be surrounded by spaces | |
38 | ERROR | Concat operator must be surrounded by spaces | |
39 | ERROR | Concat operator must be surrounded by spaces | |
39 | ERROR | Concat operator must be surrounded by spaces | |
40 | ERROR | Concat operator must be surrounded by spaces | |
40 | ERROR | Concat operator must be surrounded by spaces | |
41 | ERROR | Concat operator must be surrounded by spaces | |
41 | ERROR | Concat operator must be surrounded by spaces | |
42 | ERROR | Concat operator must be surrounded by spaces | |
42 | ERROR | Concat operator must be surrounded by spaces | |
107 | ERROR | If the line declaring an array spans longer than 80 characters, | |
| | each element should be broken into its own line | |
109 | ERROR | If the line declaring an array spans longer than 80 characters, | |
| | each element should be broken into its own line | |
109 | ERROR | Concat operator must be surrounded by spaces | |
109 | ERROR | Concat operator must be surrounded by spaces | |
109 | ERROR | Concat operator must be surrounded by spaces | |
109 | ERROR | Concat operator must be surrounded by spaces | |
109 | ERROR | Concat operator must be surrounded by spaces | |
109 | ERROR | Concat operator must be surrounded by spaces | |
113 | ERROR | Concat operator must be surrounded by spaces | |
113 | ERROR | Concat operator must be surrounded by spaces | |
115 | ERROR | If the line declaring an array spans longer than 80 characters, | |
| | each element should be broken into its own line | |
115 | ERROR | Concat operator must be surrounded by spaces | |
121 | ERROR | There must be an empty line before the parameter block | |
121 | ERROR | Last parameter comment requires a blank newline after it | |
121 | ERROR | Missing parameter type at position 1 | |
125 | ERROR | Data type of return value is missing | |
130 | ERROR | If the line declaring an array spans longer than 80 characters, | |
| | each element should be broken into its own line | |
141 | ERROR | Concat operator must be surrounded by spaces | |
141 | ERROR | Concat operator must be surrounded by spaces | |
141 | ERROR | Concat operator must be surrounded by spaces | |
141 | ERROR | Concat operator must be surrounded by spaces | |
149 | ERROR | Concat operator must be surrounded by spaces | |
149 | ERROR | Concat operator must be surrounded by spaces | |
159 | ERROR | There must be an empty line before the parameter block | |
159 | ERROR | Last parameter comment requires a blank newline after it | |
159 | ERROR | Missing parameter type at position 1 | |
165 | ERROR | Data type of return value is missing | |
169 | ERROR | global variables should start with a single underscore followed | |
| | by the module and another underscore | |
169 | ERROR | global variables should start with a single underscore followed | |
| | by the module and another underscore | |
201 | ERROR | Case breaking statements must be followed by a single blank line | |
291 | ERROR | If the line declaring an array spans longer than 80 characters, | |
| | each element should be broken into its own line | |
291 | ERROR | Concat operator must be surrounded by spaces | |
291 | ERROR | Concat operator must be surrounded by spaces | |
291 | ERROR | Concat operator must be surrounded by spaces | |
291 | ERROR | Concat operator must be surrounded by spaces | |
291 | ERROR | Concat operator must be surrounded by spaces | |
291 | ERROR | Concat operator must be surrounded by spaces | |
291 | ERROR | Concat operator must be surrounded by spaces | |
291 | ERROR | Concat operator must be surrounded by spaces | |
296 | ERROR | If the line declaring an array spans longer than 80 characters, | |
| | each element should be broken into its own line | |
298 | ERROR | If the line declaring an array spans longer than 80 characters, | |
| | each element should be broken into its own line | |
303 | ERROR | Case breaking statements must be followed by a single blank line | |
307 | ERROR | If the line declaring an array spans longer than 80 characters, | |
| | each element should be broken into its own line | |
315 | ERROR | If the line declaring an array spans longer than 80 characters, | |
| | each element should be broken into its own line | |
318 | ERROR | Concat operator must be surrounded by spaces | |
318 | ERROR | Concat operator must be surrounded by spaces | |
321 | ERROR | Concat operator must be surrounded by spaces | |
324 | ERROR | Concat operator must be surrounded by spaces | |
324 | ERROR | Concat operator must be surrounded by spaces | |
325 | ERROR | Concat operator must be surrounded by spaces | |
325 | ERROR | Concat operator must be surrounded by spaces | |
327 | ERROR | Case breaking statements must be followed by a single blank line | |
329 | ERROR | Concat operator must be surrounded by spaces | |
329 | ERROR | Concat operator must be surrounded by spaces | |
330 | ERROR | Concat operator must be surrounded by spaces | |
332 | ERROR | Case breaking statements must be followed by a single blank line | |
340 | ERROR | Concat operator must be surrounded by spaces | |
342 | ERROR | Concat operator must be surrounded by spaces | |
342 | ERROR | Concat operator must be surrounded by spaces | |
-------------------------------------------------------------------------------- | |
UPGRADE TO PHP_CODESNIFFER 2.0 TO FIX ERRORS AUTOMATICALLY | |
-------------------------------------------------------------------------------- | |
FILE: ...ps/drupal7/sites/all/modules/securesite/digest_md5/stored_passwords.php | |
-------------------------------------------------------------------------------- | |
FOUND 25 ERROR(S) AND 1 WARNING(S) AFFECTING 14 LINE(S) | |
-------------------------------------------------------------------------------- | |
73 | ERROR | There must be an empty line before the parameter block | |
73 | ERROR | Last parameter comment requires a blank newline after it | |
73 | ERROR | Missing parameter type at position 1 | |
78 | WARNING | Line exceeds 80 characters; contains 93 characters | |
79 | ERROR | Data type of return value is missing | |
94 | ERROR | Case breaking statements must be followed by a single blank | |
| | line | |
109 | ERROR | Case breaking statements must be followed by a single blank | |
| | line | |
127 | ERROR | Concat operator must be surrounded by spaces | |
127 | ERROR | String concat is not required here; use a single string | |
| | instead | |
127 | ERROR | Concat operator must be surrounded by spaces | |
128 | ERROR | Concat operator must be surrounded by spaces | |
129 | ERROR | Concat operator must be surrounded by spaces | |
129 | ERROR | Concat operator must be surrounded by spaces | |
130 | ERROR | Concat operator must be surrounded by spaces | |
130 | ERROR | Concat operator must be surrounded by spaces | |
131 | ERROR | Concat operator must be surrounded by spaces | |
131 | ERROR | Concat operator must be surrounded by spaces | |
132 | ERROR | Concat operator must be surrounded by spaces | |
132 | ERROR | Concat operator must be surrounded by spaces | |
133 | ERROR | Concat operator must be surrounded by spaces | |
133 | ERROR | Concat operator must be surrounded by spaces | |
134 | ERROR | Concat operator must be surrounded by spaces | |
134 | ERROR | Concat operator must be surrounded by spaces | |
135 | ERROR | Concat operator must be surrounded by spaces | |
135 | ERROR | String concat is not required here; use a single string | |
| | instead | |
135 | ERROR | Concat operator must be surrounded by spaces | |
-------------------------------------------------------------------------------- | |
UPGRADE TO PHP_CODESNIFFER 2.0 TO FIX ERRORS AUTOMATICALLY | |
-------------------------------------------------------------------------------- | |
FILE: ...ivanshu/webapps/drupal7/sites/all/modules/securesite/securesite.install | |
-------------------------------------------------------------------------------- | |
FOUND 2 ERROR(S) AFFECTING 2 LINE(S) | |
-------------------------------------------------------------------------------- | |
50 | ERROR | Case breaking statements must be followed by a single blank line | |
55 | ERROR | Case breaking statements must be followed by a single blank line | |
-------------------------------------------------------------------------------- | |
UPGRADE TO PHP_CODESNIFFER 2.0 TO FIX ERRORS AUTOMATICALLY | |
-------------------------------------------------------------------------------- | |
FILE: .../shivanshu/webapps/drupal7/sites/all/modules/securesite/securesite.test | |
-------------------------------------------------------------------------------- | |
FOUND 176 ERROR(S) AND 29 WARNING(S) AFFECTING 203 LINE(S) | |
-------------------------------------------------------------------------------- | |
39 | WARNING | Line exceeds 80 characters; contains 102 characters | |
40 | WARNING | Line exceeds 80 characters; contains 103 characters | |
41 | WARNING | Line exceeds 80 characters; contains 83 characters | |
51 | WARNING | Line exceeds 80 characters; contains 81 characters | |
52 | WARNING | Line exceeds 80 characters; contains 84 characters | |
53 | WARNING | Line exceeds 80 characters; contains 84 characters | |
83 | WARNING | Line exceeds 80 characters; contains 103 characters | |
84 | WARNING | Line exceeds 80 characters; contains 87 characters | |
91 | WARNING | Line exceeds 80 characters; contains 83 characters | |
92 | WARNING | Line exceeds 80 characters; contains 82 characters | |
97 | WARNING | Line exceeds 80 characters; contains 94 characters | |
98 | WARNING | Line exceeds 80 characters; contains 93 characters | |
99 | WARNING | Line exceeds 80 characters; contains 95 characters | |
100 | WARNING | Line exceeds 80 characters; contains 94 characters | |
101 | WARNING | Line exceeds 80 characters; contains 81 characters | |
110 | WARNING | Line exceeds 80 characters; contains 83 characters | |
111 | WARNING | Line exceeds 80 characters; contains 82 characters | |
113 | WARNING | Line exceeds 80 characters; contains 84 characters | |
114 | WARNING | Line exceeds 80 characters; contains 85 characters | |
115 | WARNING | Line exceeds 80 characters; contains 84 characters | |
122 | WARNING | Line exceeds 80 characters; contains 85 characters | |
144 | ERROR | Visibility must be declared on method "getInfo" | |
155 | ERROR | Visibility must be declared on method "setUp" | |
168 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteScriptsStoredPasswordsRealm" | |
176 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteScriptsStoredPasswordsAdd" | |
184 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteScriptsStoredPasswordsUpdateExisting" | |
193 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteScriptsStoredPasswordsUpdateNonexistent" | |
201 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteScriptsStoredPasswordsDeleteExisting" | |
210 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteScriptsStoredPasswordsDeleteNonexistent" | |
218 | ERROR | Visibility must be declared on method "tearDown" | |
231 | ERROR | Visibility must be declared on method "getInfo" | |
242 | ERROR | Visibility must be declared on method "setUp" | |
269 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteScriptDigestMD5Challenge" | |
276 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteScriptDigestMD5Stored" | |
291 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteScriptDigestMD5Expired" | |
305 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteScriptDigestMD5Wrong" | |
319 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteScriptDigestMD5Unstored" | |
334 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteScriptDigestMD5AuthStored" | |
352 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteScriptDigestMD5AuthReplay" | |
370 | ERROR | Visibility must be declared on method "tearDown" | |
383 | ERROR | Visibility must be declared on method "getInfo" | |
394 | ERROR | Visibility must be declared on method "setUp" | |
408 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteFunctionUserSaveAdd" | |
416 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteFunctionUserSaveUpdatePassUnchanged" | |
425 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteFunctionUserSaveUpdatePassChanged" | |
435 | ERROR | Visibility must be declared on method "tearDown" | |
448 | ERROR | Visibility must be declared on method "getInfo" | |
459 | ERROR | Visibility must be declared on method "setUp" | |
473 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteUserLoadUID" | |
475 | WARNING | Line exceeds 80 characters; contains 392 characters | |
480 | WARNING | Line exceeds 80 characters; contains 93 characters | |
481 | WARNING | Line exceeds 80 characters; contains 131 characters | |
483 | ERROR | Inline comments must end in full-stops, exclamation marks, or | |
| | question marks | |
493 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteUserLoadPass" | |
494 | WARNING | Line exceeds 80 characters; contains 88 characters | |
495 | WARNING | Line exceeds 80 characters; contains 108 characters | |
504 | ERROR | Visibility must be declared on method "tearDown" | |
517 | ERROR | Visibility must be declared on method "getInfo" | |
528 | ERROR | Visibility must be declared on method "setUp" | |
542 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteFunctionUserDelete" | |
556 | ERROR | Visibility must be declared on method "getInfo" | |
567 | ERROR | Visibility must be declared on method "setUp" | |
575 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteFunctionForcedDisabled" | |
582 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteFunctionForcedAlways" | |
590 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteFunctionForcedOffline" | |
600 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteFunctionForced403" | |
610 | ERROR | Visibility must be declared on method "tearDown" | |
626 | ERROR | Visibility must be declared on method "getInfo" | |
637 | ERROR | Visibility must be declared on method "setUp" | |
647 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteFunctionDigestValidateNull" | |
655 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteFunctionDigestValidateEmpty" | |
663 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteFunctionDigestValidateData" | |
676 | ERROR | If the line declaring an array spans longer than 80 | |
| | characters, each element should be broken into its own line | |
683 | ERROR | Visibility must be declared on method "tearDown" | |
696 | ERROR | Visibility must be declared on method "getInfo" | |
707 | ERROR | Visibility must be declared on method "setUp" | |
717 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteFunctionFakeRealmNormal" | |
724 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteFunctionFakeRealmMSIE" | |
732 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteFunctionFakeRealmOpera" | |
740 | ERROR | Visibility must be declared on method "tearDown" | |
753 | ERROR | Visibility must be declared on method "getInfo" | |
764 | ERROR | Visibility must be declared on method "setUp" | |
772 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteDialogPageNeither" | |
784 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteDialogPageLogin" | |
797 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteDialogPageReset" | |
808 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteDialogPageBoth" | |
820 | ERROR | Visibility must be declared on method "tearDown" | |
835 | ERROR | Visibility must be declared on method "getInfo" | |
846 | ERROR | Visibility must be declared on method "setUp" | |
851 | ERROR | If the line declaring an array spans longer than 80 | |
| | characters, each element should be broken into its own line | |
857 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteNameConflictUserRegister" | |
866 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteNameConflictUserCreate" | |
868 | ERROR | If the line declaring an array spans longer than 80 | |
| | characters, each element should be broken into its own line | |
876 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteNameConflictUserEdit" | |
886 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteNameConflictGuest" | |
901 | ERROR | Visibility must be declared on method "getInfo" | |
912 | ERROR | Visibility must be declared on method "setUp" | |
919 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteForceDisabled" | |
932 | ERROR | Visibility must be declared on method "getInfo" | |
943 | ERROR | Visibility must be declared on method "setUp" | |
953 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteForceAlwaysNobody" | |
961 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteForceAlwaysUser" | |
972 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteForceAlwaysGuest" | |
985 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteForceAlwaysResetValid" | |
986 | WARNING | Line exceeds 80 characters; contains 90 characters | |
986 | ERROR | Comments may not appear after statements. | |
988 | WARNING | Line exceeds 80 characters; contains 89 characters | |
988 | ERROR | Comments may not appear after statements. | |
997 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteForceAlwaysResetInvalid" | |
1006 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteForceAlwaysResetSubmit" | |
1015 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteForceAlwaysCronAll" | |
1016 | ERROR | If the line declaring an array spans longer than 80 | |
| | characters, each element should be broken into its own line | |
1025 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteForceAlwaysCronForm" | |
1035 | ERROR | Visibility must be declared on method "tearDown" | |
1049 | ERROR | Visibility must be declared on method "getInfo" | |
1060 | ERROR | Visibility must be declared on method "setUp" | |
1068 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteForceOfflineNormal" | |
1076 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteForceOfflineMaintenance" | |
1091 | ERROR | Visibility must be declared on method "getInfo" | |
1102 | ERROR | Visibility must be declared on method "setUp" | |
1112 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteForce403Normal" | |
1120 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteForce403Restricted" | |
1128 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteForce403User" | |
1142 | ERROR | Visibility must be declared on method "getInfo" | |
1153 | ERROR | Visibility must be declared on method "setUp" | |
1159 | WARNING | Line exceeds 80 characters; contains 85 characters | |
1161 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteConfig403Save" | |
1169 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteConfig403ResetCurrent" | |
1178 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteConfig403Page" | |
1188 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteConfig403ResetPrevious" | |
1197 | ERROR | Visibility must be declared on method "tearDown" | |
1212 | ERROR | Visibility must be declared on method "getInfo" | |
1223 | ERROR | Visibility must be declared on method "setUp" | |
1231 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteTypeBasicNone" | |
1249 | ERROR | Visibility must be declared on method "getInfo" | |
1260 | ERROR | Visibility must be declared on method "setUp" | |
1266 | ERROR | If the line declaring an array spans longer than 80 | |
| | characters, each element should be broken into its own line | |
1272 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteTypeBasicUserWrong" | |
1281 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteTypeBasicUserNoAccess" | |
1292 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteTypeBasicUserAccess" | |
1306 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteTypeBasicUserChange" | |
1317 | ERROR | Visibility must be declared on method "tearDown" | |
1330 | ERROR | Visibility must be declared on method "getInfo" | |
1341 | ERROR | Visibility must be declared on method "setUp" | |
1345 | ERROR | If the line declaring an array spans longer than 80 | |
| | characters, each element should be broken into its own line | |
1352 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteTypeBasicGuestUnsetEmptyNoAccess" | |
1363 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteTypeBasicGuestUnsetEmptyAccess" | |
1371 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteTypeBasicGuestUnsetRandomNoAccess" | |
1381 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteTypeBasicGuestUnsetRandomAccess" | |
1390 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteTypeBasicGuestUnsetChange" | |
1401 | ERROR | Visibility must be declared on method "tearDown" | |
1414 | ERROR | Visibility must be declared on method "getInfo" | |
1425 | ERROR | Visibility must be declared on method "setUp" | |
1433 | ERROR | If the line declaring an array spans longer than 80 | |
| | characters, each element should be broken into its own line | |
1439 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteTypeBasicGuestSetEmpty" | |
1450 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteTypeBasicGuestSetWrong" | |
1459 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteTypeBasicGuestSetCorrect" | |
1468 | ERROR | Visibility must be declared on method "tearDown" | |
1481 | ERROR | Visibility must be declared on method "getInfo" | |
1492 | ERROR | Visibility must be declared on method "setUp" | |
1496 | ERROR | If the line declaring an array spans longer than 80 | |
| | characters, each element should be broken into its own line | |
1502 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteTypeFormNone" | |
1515 | ERROR | Visibility must be declared on method "getInfo" | |
1526 | ERROR | Visibility must be declared on method "setUp" | |
1533 | ERROR | If the line declaring an array spans longer than 80 | |
| | characters, each element should be broken into its own line | |
1539 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteTypeFormUserWrong" | |
1548 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteTypeFormUserNoAccess" | |
1557 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteTypeFormUserAccess" | |
1573 | ERROR | Visibility must be declared on method "getInfo" | |
1584 | ERROR | Visibility must be declared on method "setUp" | |
1588 | ERROR | If the line declaring an array spans longer than 80 | |
| | characters, each element should be broken into its own line | |
1594 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteTypeFormGuestUnsetEmptyNoAccess" | |
1603 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteTypeFormGuestUnsetRandomNoAccess" | |
1612 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteTypeFormGuestUnsetRandomAccess" | |
1627 | ERROR | Visibility must be declared on method "getInfo" | |
1638 | ERROR | Visibility must be declared on method "setUp" | |
1643 | ERROR | If the line declaring an array spans longer than 80 | |
| | characters, each element should be broken into its own line | |
1649 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteTypeDigestNone" | |
1669 | ERROR | Visibility must be declared on method "getInfo" | |
1680 | ERROR | Visibility must be declared on method "setUp" | |
1686 | ERROR | If the line declaring an array spans longer than 80 | |
| | characters, each element should be broken into its own line | |
1694 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteTypeDigestUserUnstoredBasic" | |
1707 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteTypeDigestUserUnstoredForm" | |
1717 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteTypeDigestUserUnstoredStore" | |
1731 | ERROR | Visibility must be declared on method "tearDown" | |
1744 | ERROR | Visibility must be declared on method "getInfo" | |
1755 | ERROR | Visibility must be declared on method "setUp" | |
1760 | ERROR | If the line declaring an array spans longer than 80 | |
| | characters, each element should be broken into its own line | |
1768 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteTypeDigestUserStoredWrong" | |
1777 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteTypeDigestUserStoredCorrect" | |
1791 | ERROR | Visibility must be declared on method "tearDown" | |
1804 | ERROR | Visibility must be declared on method "getInfo" | |
1815 | ERROR | Visibility must be declared on method "setUp" | |
1821 | ERROR | If the line declaring an array spans longer than 80 | |
| | characters, each element should be broken into its own line | |
1828 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteTypeDigestGuestUnsetEmpty" | |
1838 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteTypeDigestGuestUnsetRandom" | |
1853 | ERROR | Visibility must be declared on method "getInfo" | |
1864 | ERROR | Visibility must be declared on method "setUp" | |
1870 | ERROR | If the line declaring an array spans longer than 80 | |
| | characters, each element should be broken into its own line | |
1877 | ERROR | If the line declaring an array spans longer than 80 | |
| | characters, each element should be broken into its own line | |
1884 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteDigestGuestSetEmpty" | |
1893 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteDigestGuestSetRandom" | |
1902 | ERROR | Visibility must be declared on method | |
| | "testSecureSiteDigestGuestSetCorrect" | |
1914 | ERROR | Visibility must be declared on method "tearDown" | |
1927 | ERROR | Missing parameter type at position 1 | |
-------------------------------------------------------------------------------- | |
UPGRADE TO PHP_CODESNIFFER 2.0 TO FIX ERRORS AUTOMATICALLY | |
-------------------------------------------------------------------------------- | |
FILE: ...hivanshu/webapps/drupal7/sites/all/modules/securesite/securesite.module | |
-------------------------------------------------------------------------------- | |
FOUND 5 ERROR(S) AND 4 WARNING(S) AFFECTING 9 LINE(S) | |
-------------------------------------------------------------------------------- | |
43 | WARNING | Format should be "* Implements hook_foo().", "* Implements | |
| | hook_foo_BAR_ID_bar() for xyz_bar().", or "* Implements | |
| | hook_foo_BAR_ID_bar() for xyz_bar.tpl.php.". | |
110 | WARNING | Line exceeds 80 characters; contains 83 characters | |
126 | ERROR | Function comment short description must be on a single line, | |
| | further text should be a separate paragraph | |
143 | ERROR | Case breaking statements must be followed by a single blank | |
| | line | |
159 | ERROR | Case breaking statements must be followed by a single blank | |
| | line | |
165 | ERROR | Case breaking statements must be followed by a single blank | |
| | line | |
218 | ERROR | A cast statement must be followed by a single space | |
275 | WARNING | Line exceeds 80 characters; contains 83 characters | |
276 | WARNING | Line exceeds 80 characters; contains 83 characters | |
-------------------------------------------------------------------------------- | |
UPGRADE TO PHP_CODESNIFFER 2.0 TO FIX ERRORS AUTOMATICALLY | |
-------------------------------------------------------------------------------- | |
FILE: ...webapps/drupal7/sites/all/modules/securesite/theme/securesite.theme.inc | |
-------------------------------------------------------------------------------- | |
FOUND 18 ERROR(S) AFFECTING 18 LINE(S) | |
-------------------------------------------------------------------------------- | |
9 | ERROR | Function comment short description must end with a full stop | |
11 | ERROR | Missing parameter type at position 1 | |
20 | ERROR | Function comment short description must end with a full stop | |
22 | ERROR | Missing parameter type at position 1 | |
31 | ERROR | Function comment short description must end with a full stop | |
33 | ERROR | Missing parameter type at position 1 | |
42 | ERROR | Whitespace found at end of line | |
44 | ERROR | Inline comments must end in full-stops, exclamation marks, or | |
| | question marks | |
50 | ERROR | Inline comments must end in full-stops, exclamation marks, or | |
| | question marks | |
54 | ERROR | Whitespace found at end of line | |
55 | ERROR | Inline comments must end in full-stops, exclamation marks, or | |
| | question marks | |
58 | ERROR | Whitespace found at end of line | |
59 | ERROR | Inline comments must end in full-stops, exclamation marks, or | |
| | question marks | |
66 | ERROR | Whitespace found at end of line | |
69 | ERROR | Whitespace found at end of line | |
70 | ERROR | Inline comments must end in full-stops, exclamation marks, or | |
| | question marks | |
75 | ERROR | Function comment short description must end with a full stop | |
77 | ERROR | Missing parameter type at position 1 | |
-------------------------------------------------------------------------------- | |
UPGRADE TO PHP_CODESNIFFER 2.0 TO FIX ERRORS AUTOMATICALLY | |
-------------------------------------------------------------------------------- | |
FILE: ...anshu/webapps/drupal7/sites/all/modules/securesite/theme/securesite.css | |
-------------------------------------------------------------------------------- | |
FOUND 6 ERROR(S) AFFECTING 6 LINE(S) | |
-------------------------------------------------------------------------------- | |
1 | ERROR | Additional whitespace found at start of file | |
12 | ERROR | CSS colours must be defined in lowercase; expected #bbb but | |
| | found #BBB | |
17 | ERROR | CSS colours must be defined in lowercase; expected #eee but | |
| | found #EEE | |
25 | ERROR | Multiple selectors should each be on a single line | |
45 | ERROR | Multiple selectors should each be on a single line | |
110 | ERROR | Multiple selectors should each be on a single line | |
-------------------------------------------------------------------------------- | |
UPGRADE TO PHP_CODESNIFFER 2.0 TO FIX ERRORS AUTOMATICALLY | |
-------------------------------------------------------------------------------- | |
FILE: ...apps/drupal7/sites/all/modules/securesite/theme/securesite-page.tpl.php | |
-------------------------------------------------------------------------------- | |
FOUND 1 ERROR(S) AFFECTING 1 LINE(S) | |
-------------------------------------------------------------------------------- | |
27 | ERROR | Files must end in a single new line character | |
-------------------------------------------------------------------------------- | |
UPGRADE TO PHP_CODESNIFFER 2.0 TO FIX ERRORS AUTOMATICALLY | |
-------------------------------------------------------------------------------- | |
FILE: ...hu/webapps/drupal7/sites/all/modules/securesite/securesite.variable.inc | |
-------------------------------------------------------------------------------- | |
FOUND 2 ERROR(S) AFFECTING 2 LINE(S) | |
-------------------------------------------------------------------------------- | |
38 | ERROR | If the line declaring an array spans longer than 80 characters, | |
| | each element should be broken into its own line | |
40 | ERROR | If the line declaring an array spans longer than 80 characters, | |
| | each element should be broken into its own line | |
-------------------------------------------------------------------------------- | |
UPGRADE TO PHP_CODESNIFFER 2.0 TO FIX ERRORS AUTOMATICALLY | |
-------------------------------------------------------------------------------- | |
FILE: /home/shivanshu/webapps/drupal7/sites/all/modules/securesite/README.txt | |
-------------------------------------------------------------------------------- | |
FOUND 0 ERROR(S) AND 3 WARNING(S) AFFECTING 3 LINE(S) | |
-------------------------------------------------------------------------------- | |
159 | WARNING | Line exceeds 80 characters; contains 84 characters | |
160 | WARNING | Line exceeds 80 characters; contains 96 characters | |
161 | WARNING | Line exceeds 80 characters; contains 109 characters | |
-------------------------------------------------------------------------------- | |
UPGRADE TO PHP_CODESNIFFER 2.0 TO FIX ERRORS AUTOMATICALLY | |
-------------------------------------------------------------------------------- | |
FILE: ...e/shivanshu/webapps/drupal7/sites/all/modules/securesite/securesite.inc | |
-------------------------------------------------------------------------------- | |
FOUND 33 ERROR(S) AND 8 WARNING(S) AFFECTING 40 LINE(S) | |
-------------------------------------------------------------------------------- | |
18 | ERROR | Case breaking statements must be followed by a single blank | |
| | line | |
23 | ERROR | Case breaking statements must be followed by a single blank | |
| | line | |
71 | ERROR | If the line declaring an array spans longer than 80 | |
| | characters, each element should be broken into its own line | |
80 | ERROR | Case breaking statements must be followed by a single blank | |
| | line | |
96 | ERROR | Case breaking statements must be followed by a single blank | |
| | line | |
98 | WARNING | Line exceeds 80 characters; contains 97 characters | |
110 | ERROR | Case breaking statements must be followed by a single blank | |
| | line | |
114 | WARNING | Line exceeds 80 characters; contains 94 characters | |
126 | ERROR | Missing parameter type at position 1 | |
128 | ERROR | Last parameter comment requires a blank newline after it | |
128 | ERROR | Missing parameter type at position 2 | |
130 | ERROR | Data type of return value is missing | |
143 | WARNING | Line exceeds 80 characters; contains 114 characters | |
144 | WARNING | Line exceeds 80 characters; contains 106 characters | |
145 | WARNING | Line exceeds 80 characters; contains 114 characters | |
164 | ERROR | Inline comments must end in full-stops, exclamation marks, or | |
| | question marks | |
209 | ERROR | Whitespace found at end of line | |
212 | ERROR | Whitespace found at end of line | |
219 | ERROR | Whitespace found at end of line | |
220 | ERROR | Inline comments must end in full-stops, exclamation marks, or | |
| | question marks | |
242 | ERROR | Expected "if (...) {\n"; found "if(...){\n" | |
275 | WARNING | Line exceeds 80 characters; contains 100 characters | |
280 | ERROR | Whitespace found at end of line | |
281 | ERROR | Inline comments must end in full-stops, exclamation marks, or | |
| | question marks | |
284 | ERROR | Whitespace found at end of line | |
285 | ERROR | Inline comments must end in full-stops, exclamation marks, or | |
| | question marks | |
291 | WARNING | Line exceeds 80 characters; contains 83 characters | |
292 | WARNING | Line exceeds 80 characters; contains 83 characters | |
320 | ERROR | Case breaking statements must be followed by a single blank | |
| | line | |
322 | ERROR | Case breaking statements must be followed by a single blank | |
| | line | |
332 | ERROR | Missing parameter type at position 1 | |
351 | ERROR | Inline comments must end in full-stops, exclamation marks, or | |
| | question marks | |
382 | ERROR | Case breaking statements must be followed by a single blank | |
| | line | |
386 | ERROR | Case breaking statements must be followed by a single blank | |
| | line | |
399 | ERROR | Inline comments must end in full-stops, exclamation marks, or | |
| | question marks | |
401 | ERROR | Whitespace found at end of line | |
402 | ERROR | Inline comments must end in full-stops, exclamation marks, or | |
| | question marks | |
409 | ERROR | Function comment short description must be on a single line, | |
| | further text should be a separate paragraph | |
423 | ERROR | Function comment short description must be on a single line, | |
| | further text should be a separate paragraph | |
450 | ERROR | Function comment short description must be on a single line, | |
| | further text should be a separate paragraph | |
509 | ERROR | Missing function doc comment | |
-------------------------------------------------------------------------------- | |
UPGRADE TO PHP_CODESNIFFER 2.0 TO FIX ERRORS AUTOMATICALLY | |
-------------------------------------------------------------------------------- |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
~/webapps/drupal8 ⮀ ⮀ ⮀ drupalcs8 modules/securesite ⮂ | |
FILE: ...ebapps/drupal8/modules/securesite/digest_md5/digest_md5.conf.php | |
---------------------------------------------------------------------- | |
FOUND 16 ERRORS AFFECTING 11 LINES | |
---------------------------------------------------------------------- | |
6 | ERROR | [ ] Doc comment short description must be on a single | |
| | line, further text should be a separate paragraph | |
11 | ERROR | [ ] Doc comment short description must be on a single | |
| | line, further text should be a separate paragraph | |
16 | ERROR | [x] Doc comment short description must end with a full | |
| | stop | |
39 | ERROR | [ ] Doc comment short description must be on a single | |
| | line, further text should be a separate paragraph | |
41 | ERROR | [x] Perl-style comments are not allowed; use "// | |
| | Comment" instead | |
45 | ERROR | [ ] Doc comment short description must be on a single | |
| | line, further text should be a separate paragraph | |
47 | ERROR | [x] Perl-style comments are not allowed; use "// | |
| | Comment" instead | |
55 | ERROR | [x] No space found before comment text; expected "// | |
| | todo realm should be text, not varchar" but found | |
| | "//todo realm should be text, not varchar" | |
55 | ERROR | [x] Inline comments must start with a capital letter | |
55 | ERROR | [x] Inline comments must end in full-stops, | |
| | exclamation marks, or question marks | |
87 | ERROR | [x] No space found before comment text; expected "// | |
| | todo nonce and realm fields should be text" but | |
| | found "//todo nonce and realm fields should be | |
| | text" | |
87 | ERROR | [x] Inline comments must start with a capital letter | |
87 | ERROR | [x] Inline comments must end in full-stops, | |
| | exclamation marks, or question marks | |
130 | ERROR | [x] Expected 1 space between comma and "100"; 0 found | |
130 | ERROR | [x] Expected one space after the comma, 0 found | |
141 | ERROR | [x] Case breaking statements must be followed by a | |
| | single blank line | |
---------------------------------------------------------------------- | |
PHPCBF CAN FIX THE 12 MARKED SNIFF VIOLATIONS AUTOMATICALLY | |
---------------------------------------------------------------------- | |
FILE: ...shu/webapps/drupal8/modules/securesite/digest_md5/digest_md5.php | |
---------------------------------------------------------------------- | |
FOUND 113 ERRORS AND 3 WARNINGS AFFECTING 61 LINES | |
---------------------------------------------------------------------- | |
8 | ERROR | [ ] Doc comment short description must be on a | |
| | single line, further text should be a separate | |
| | paragraph | |
30 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
30 | ERROR | [ ] String concat is not required here; use a single | |
| | string instead | |
30 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
31 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
32 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
32 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
33 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
33 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
34 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
34 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
35 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
35 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
36 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
36 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
37 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
37 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
38 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
38 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
39 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
39 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
40 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
40 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
41 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
41 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
42 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
42 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
80 | WARNING | [ ] Line exceeds 80 characters; contains 91 | |
| | characters | |
80 | ERROR | [x] Doc comment short description must start with a | |
| | capital letter | |
82 | ERROR | [x] "include_once" is a statement not a function; no | |
| | parentheses are required | |
82 | ERROR | [x] Language constructs must be followed by a single | |
| | space; expected "include_once (" but found | |
| | "include_once(" | |
89 | ERROR | [x] Doc comment short description must start with a | |
| | capital letter | |
121 | ERROR | [ ] If the line declaring an array spans longer than | |
| | 80 characters, each element should be broken | |
| | into its own line | |
123 | ERROR | [x] No space found before comment text; expected "// | |
| | todo removing the use of $edit['method'] for | |
| | now. don't know the use of this" but found | |
| | "//todo removing the use of $edit['method'] for | |
| | now. don't know the use of this" | |
123 | ERROR | [x] Inline comments must start with a capital letter | |
123 | ERROR | [x] Inline comments must end in full-stops, | |
| | exclamation marks, or question marks | |
124 | ERROR | [ ] If the line declaring an array spans longer than | |
| | 80 characters, each element should be broken | |
| | into its own line | |
124 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
124 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
124 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
124 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
124 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
124 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
125 | ERROR | [x] Line indented incorrectly; expected 2 spaces, | |
| | found 0 | |
125 | ERROR | [ ] 2 spaces found before inline comment; expected | |
| | "// if ($edit['method'] != 'AUTHENTICATE') {" | |
| | but found "// if ($edit['method'] != | |
| | 'AUTHENTICATE') {" | |
125 | ERROR | [x] Inline comments must start with a capital letter | |
126 | ERROR | [x] Line indented incorrectly; expected 2 spaces, | |
| | found 4 | |
127 | ERROR | [x] Line indented incorrectly; expected 2 spaces, | |
| | found 4 | |
128 | ERROR | [x] Line indented incorrectly; expected 2 spaces, | |
| | found 4 | |
128 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
128 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
129 | ERROR | [x] Line indented incorrectly; expected 2 spaces, | |
| | found 0 | |
129 | ERROR | [ ] 2 spaces found before inline comment; expected | |
| | "// }" but found "// }" | |
130 | ERROR | [ ] If the line declaring an array spans longer than | |
| | 80 characters, each element should be broken | |
| | into its own line | |
130 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
136 | ERROR | [x] There must be exactly one blank line before the | |
| | tags in a doc comment | |
136 | ERROR | [ ] Missing parameter type | |
140 | ERROR | [x] Separate the @param and @return sections by a | |
| | blank line. | |
140 | ERROR | [ ] Return type missing for @return tag in function | |
| | comment | |
145 | ERROR | [ ] If the line declaring an array spans longer than | |
| | 80 characters, each element should be broken | |
| | into its own line | |
156 | ERROR | [x] No space found before comment text; expected "// | |
| | todo check here" but found "//todo check here" | |
156 | ERROR | [x] Inline comments must start with a capital letter | |
156 | ERROR | [x] Inline comments must end in full-stops, | |
| | exclamation marks, or question marks | |
158 | WARNING | [ ] Line exceeds 80 characters; contains 140 | |
| | characters | |
158 | ERROR | [x] No space found before comment text; expected "// | |
| | db_query("INSERT INTO `securesite_nonce` (". | |
| | implode(', ', array_keys($values)) .") VALUES | |
| | (". implode(', ', $types) .")", $values);" but | |
| | found "//db_query("INSERT INTO | |
| | `securesite_nonce` (". implode(', ', | |
| | array_keys($values)) .") VALUES (". implode(', | |
| | ', $types) .")", $values);" | |
167 | WARNING | [ ] Line exceeds 80 characters; contains 124 | |
| | characters | |
167 | ERROR | [x] No space found before comment text; expected "// | |
| | db_query("UPDATE `securesite_nonce` SET ". | |
| | implode(', ', $fields) ." WHERE nonce = '%s' AND | |
| | realm = '%s'", $values);" but found | |
| | "//db_query("UPDATE `securesite_nonce` SET ". | |
| | implode(', ', $fields) ." WHERE nonce = '%s' AND | |
| | realm = '%s'", $values);" | |
177 | ERROR | [x] There must be exactly one blank line before the | |
| | tags in a doc comment | |
177 | ERROR | [ ] Missing parameter type | |
183 | ERROR | [x] Separate the @param and @return sections by a | |
| | blank line. | |
183 | ERROR | [ ] Return type missing for @return tag in function | |
| | comment | |
187 | ERROR | [ ] global variables should start with a single | |
| | underscore followed by the module and another | |
| | underscore | |
187 | ERROR | [ ] global variables should start with a single | |
| | underscore followed by the module and another | |
| | underscore | |
199 | ERROR | [x] Line indented incorrectly; expected 4 spaces, | |
| | found 0 | |
199 | ERROR | [ ] 4 spaces found before inline comment; expected | |
| | "// if ($edit['method'] != 'AUTHENTICATE') {" | |
| | but found "// if ($edit['method'] != | |
| | 'AUTHENTICATE') {" | |
199 | ERROR | [x] Inline comments must start with a capital letter | |
200 | ERROR | [x] Line indented incorrectly; expected 4 spaces, | |
| | found 6 | |
201 | ERROR | [x] Line indented incorrectly; expected 4 spaces, | |
| | found 0 | |
201 | ERROR | [ ] 4 spaces found before inline comment; expected | |
| | "// }" but found "// }" | |
209 | ERROR | [x] There should be no white space after an opening | |
| | "(" | |
210 | ERROR | [x] There should be no white space after an opening | |
| | "(" | |
219 | ERROR | [x] Case breaking statements must be followed by a | |
| | single blank line | |
249 | ERROR | [x] There should be no white space after an opening | |
| | "(" | |
287 | ERROR | [x] There should be no white space after an opening | |
| | "(" | |
309 | ERROR | [ ] If the line declaring an array spans longer than | |
| | 80 characters, each element should be broken | |
| | into its own line | |
309 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
309 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
309 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
309 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
309 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
309 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
309 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
309 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
314 | ERROR | [ ] If the line declaring an array spans longer than | |
| | 80 characters, each element should be broken | |
| | into its own line | |
316 | ERROR | [ ] If the line declaring an array spans longer than | |
| | 80 characters, each element should be broken | |
| | into its own line | |
321 | ERROR | [x] Case breaking statements must be followed by a | |
| | single blank line | |
325 | ERROR | [ ] If the line declaring an array spans longer than | |
| | 80 characters, each element should be broken | |
| | into its own line | |
331 | ERROR | [x] There should be no white space after an opening | |
| | "(" | |
333 | ERROR | [ ] If the line declaring an array spans longer than | |
| | 80 characters, each element should be broken | |
| | into its own line | |
336 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
336 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
339 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
342 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
342 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
343 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
343 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
344 | ERROR | [x] No space found before comment text; expected "// | |
| | TODO check next line" but found "//TODO check | |
| | next line" | |
344 | ERROR | [x] Inline comments must end in full-stops, | |
| | exclamation marks, or question marks | |
346 | ERROR | [x] Case breaking statements must be followed by a | |
| | single blank line | |
348 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
348 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
349 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
351 | ERROR | [x] Case breaking statements must be followed by a | |
| | single blank line | |
359 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
361 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
361 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
---------------------------------------------------------------------- | |
PHPCBF CAN FIX THE 92 MARKED SNIFF VIOLATIONS AUTOMATICALLY | |
---------------------------------------------------------------------- | |
FILE: ...bapps/drupal8/modules/securesite/digest_md5/stored_passwords.php | |
---------------------------------------------------------------------- | |
FOUND 33 ERRORS AND 2 WARNINGS AFFECTING 21 LINES | |
---------------------------------------------------------------------- | |
7 | ERROR | [ ] Doc comment short description must be on a | |
| | single line, further text should be a separate | |
| | paragraph | |
68 | WARNING | [ ] Line exceeds 80 characters; contains 91 | |
| | characters | |
68 | ERROR | [x] Doc comment short description must start with a | |
| | capital letter | |
70 | ERROR | [x] "include_once" is a statement not a function; no | |
| | parentheses are required | |
70 | ERROR | [x] Language constructs must be followed by a single | |
| | space; expected "include_once (" but found | |
| | "include_once(" | |
76 | ERROR | [x] Doc comment short description must start with a | |
| | capital letter | |
79 | ERROR | [x] No space found before comment text; expected "// | |
| | $conf_path = conf_path();" but found | |
| | "//$conf_path = conf_path();" | |
93 | ERROR | [x] There must be exactly one blank line before the | |
| | tags in a doc comment | |
93 | ERROR | [ ] Missing parameter type | |
98 | WARNING | [ ] Line exceeds 80 characters; contains 93 | |
| | characters | |
99 | ERROR | [x] Separate the @param and @return sections by a | |
| | blank line. | |
99 | ERROR | [ ] Return type missing for @return tag in function | |
| | comment | |
107 | ERROR | [ ] If the line declaring an array spans longer than | |
| | 80 characters, each element should be broken | |
| | into its own line | |
114 | ERROR | [x] Case breaking statements must be followed by a | |
| | single blank line | |
129 | ERROR | [x] Case breaking statements must be followed by a | |
| | single blank line | |
135 | ERROR | [ ] If the line declaring an array spans longer than | |
| | 80 characters, each element should be broken | |
| | into its own line | |
147 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
147 | ERROR | [ ] String concat is not required here; use a single | |
| | string instead | |
147 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
148 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
149 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
149 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
150 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
150 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
151 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
151 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
152 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
152 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
153 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
153 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
154 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
154 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
155 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
155 | ERROR | [ ] String concat is not required here; use a single | |
| | string instead | |
155 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
---------------------------------------------------------------------- | |
PHPCBF CAN FIX THE 26 MARKED SNIFF VIOLATIONS AUTOMATICALLY | |
---------------------------------------------------------------------- | |
FILE: .../shivanshu/webapps/drupal8/modules/securesite/securesite.install | |
---------------------------------------------------------------------- | |
FOUND 2 ERRORS AFFECTING 2 LINES | |
---------------------------------------------------------------------- | |
11 | ERROR | [x] Expected 1 space after closing parenthesis; found 0 | |
34 | ERROR | [x] Expected 1 newline at end of file; 0 found | |
---------------------------------------------------------------------- | |
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY | |
---------------------------------------------------------------------- | |
FILE: ...e/shivanshu/webapps/drupal8/modules/securesite/securesite.module | |
---------------------------------------------------------------------- | |
FOUND 34 ERRORS AND 4 WARNINGS AFFECTING 35 LINES | |
---------------------------------------------------------------------- | |
8 | ERROR | [x] Doc comment short description must end with a | |
| | full stop | |
13 | ERROR | [x] Doc comment short description must end with a | |
| | full stop | |
18 | ERROR | [x] Doc comment short description must end with a | |
| | full stop | |
23 | ERROR | [x] Doc comment short description must end with a | |
| | full stop | |
28 | ERROR | [x] Doc comment short description must end with a | |
| | full stop | |
33 | ERROR | [x] Doc comment short description must end with a | |
| | full stop | |
38 | ERROR | [x] Doc comment short description must end with a | |
| | full stop | |
65 | WARNING | [ ] Line exceeds 80 characters; contains 83 | |
| | characters | |
66 | WARNING | [ ] Line exceeds 80 characters; contains 83 | |
| | characters | |
93 | ERROR | [x] Doc comment short description must end with a | |
| | full stop | |
95 | ERROR | [x] Expected 1 space after closing parenthesis; | |
| | found 0 | |
98 | ERROR | [x] Expected 1 space after IF keyword; 0 found | |
104 | ERROR | [x] Doc comment short description must end with a | |
| | full stop | |
119 | ERROR | [x] Use "elseif" in place of "else if" | |
122 | ERROR | [x] Use "elseif" in place of "else if" | |
125 | ERROR | [x] Use "elseif" in place of "else if" | |
137 | ERROR | [x] Expected one space after the comma, 2 found | |
137 | ERROR | [x] Expected 1 space between comma and type hint | |
| | "FormStateInterface"; 2 found | |
141 | ERROR | [x] Closing parenthesis of array declaration must be | |
| | on a new line | |
169 | ERROR | [x] Line indented incorrectly; expected 4 spaces, | |
| | found 0 | |
170 | ERROR | [x] Line indented incorrectly; expected 4 spaces, | |
| | found 6 | |
176 | WARNING | [ ] Format should be "* Implements hook_foo().", "* | |
| | Implements hook_foo_BAR_ID_bar() for | |
| | xyz_bar().",, "* Implements | |
| | hook_foo_BAR_ID_bar() for xyz-bar.html.twig.", | |
| | or "* Implements hook_foo_BAR_ID_bar() for | |
| | xyz-bar.tpl.php.". | |
177 | ERROR | [x] Doc comment short description must end with a | |
| | full stop | |
177 | ERROR | [ ] Doc comment short description must be on a | |
| | single line, further text should be a separate | |
| | paragraph | |
195 | ERROR | [ ] More than 2 empty lines are not allowed | |
196 | ERROR | [ ] More than 2 empty lines are not allowed | |
197 | ERROR | [ ] More than 2 empty lines are not allowed | |
198 | ERROR | [ ] More than 2 empty lines are not allowed | |
199 | ERROR | [ ] More than 2 empty lines are not allowed | |
230 | ERROR | [ ] Doc comment short description must be on a | |
| | single line, further text should be a separate | |
| | paragraph | |
233 | ERROR | [x] Separate the @ingroup and @see sections by a | |
| | blank line. | |
243 | ERROR | [x] Separate the @ingroup and @see sections by a | |
| | blank line. | |
251 | ERROR | [ ] More than 2 empty lines are not allowed | |
252 | ERROR | [ ] More than 2 empty lines are not allowed | |
254 | WARNING | [ ] Format should be "* Implements hook_foo().", "* | |
| | Implements hook_foo_BAR_ID_bar() for | |
| | xyz_bar().",, "* Implements | |
| | hook_foo_BAR_ID_bar() for xyz-bar.html.twig.", | |
| | or "* Implements hook_foo_BAR_ID_bar() for | |
| | xyz-bar.tpl.php.". | |
255 | ERROR | [x] Doc comment short description must end with a | |
| | full stop | |
255 | ERROR | [ ] Doc comment short description must be on a | |
| | single line, further text should be a separate | |
| | paragraph | |
308 | ERROR | [ ] If the line declaring an array spans longer than | |
| | 80 characters, each element should be broken | |
| | into its own line | |
---------------------------------------------------------------------- | |
PHPCBF CAN FIX THE 23 MARKED SNIFF VIOLATIONS AUTOMATICALLY | |
---------------------------------------------------------------------- | |
FILE: ...hu/webapps/drupal8/modules/securesite/theme/securesite.theme.inc | |
---------------------------------------------------------------------- | |
FOUND 16 ERRORS AFFECTING 16 LINES | |
---------------------------------------------------------------------- | |
12 | ERROR | [x] Doc comment short description must end with a full | |
| | stop | |
14 | ERROR | [ ] Missing parameter type | |
29 | ERROR | [x] Doc comment short description must end with a full | |
| | stop | |
31 | ERROR | [ ] Missing parameter type | |
36 | ERROR | [x] No space found before comment text; expected "// | |
| | $variables['form']['name']['#required'] = FALSE;" | |
| | but found | |
| | "//$variables['form']['name']['#required'] = | |
| | FALSE;" | |
39 | ERROR | [x] No space found before comment text; expected "// | |
| | $variables['children'] = | |
| | drupal_render_children($variables['form']);" but | |
| | found "//$variables['children'] = | |
| | drupal_render_children($variables['form']);" | |
43 | ERROR | [x] Doc comment short description must end with a full | |
| | stop | |
45 | ERROR | [ ] Missing parameter type | |
56 | ERROR | [x] Inline comments must end in full-stops, exclamation | |
| | marks, or question marks | |
62 | ERROR | [x] Inline comments must end in full-stops, exclamation | |
| | marks, or question marks | |
66 | ERROR | [x] Whitespace found at end of line | |
67 | ERROR | [x] Inline comments must end in full-stops, exclamation | |
| | marks, or question marks | |
70 | ERROR | [x] Whitespace found at end of line | |
71 | ERROR | [x] Inline comments must end in full-stops, exclamation | |
| | marks, or question marks | |
80 | ERROR | [x] Inline comments must end in full-stops, exclamation | |
| | marks, or question marks | |
82 | ERROR | [x] Expected 1 newline at end of file; 0 found | |
---------------------------------------------------------------------- | |
PHPCBF CAN FIX THE 13 MARKED SNIFF VIOLATIONS AUTOMATICALLY | |
---------------------------------------------------------------------- | |
FILE: ...hivanshu/webapps/drupal8/modules/securesite/theme/securesite.css | |
---------------------------------------------------------------------- | |
FOUND 6 ERRORS AFFECTING 6 LINES | |
---------------------------------------------------------------------- | |
1 | ERROR | [x] Additional whitespace found at start of file | |
12 | ERROR | [x] CSS colours must be defined in lowercase; expected | |
| | #bbb but found #BBB | |
17 | ERROR | [x] CSS colours must be defined in lowercase; expected | |
| | #eee but found #EEE | |
21 | ERROR | [x] Multiple selectors should each be on a single line | |
41 | ERROR | [x] Multiple selectors should each be on a single line | |
106 | ERROR | [x] Multiple selectors should each be on a single line | |
---------------------------------------------------------------------- | |
PHPCBF CAN FIX THE 6 MARKED SNIFF VIOLATIONS AUTOMATICALLY | |
---------------------------------------------------------------------- | |
FILE: ...ps/drupal8/modules/securesite/src/SecuresiteManagerInterface.php | |
---------------------------------------------------------------------- | |
FOUND 10 ERRORS AFFECTING 8 LINES | |
---------------------------------------------------------------------- | |
14 | ERROR | [x] Doc comment short description must end with a full | |
| | stop | |
19 | ERROR | [x] Missing function doc comment | |
22 | ERROR | [x] Doc comment short description must end with a full | |
| | stop | |
29 | ERROR | [ ] Missing short description in doc comment | |
31 | ERROR | [ ] Parameter comment must start with a capital letter | |
31 | ERROR | [ ] Parameter comment must end with a full stop | |
35 | ERROR | [x] Missing function doc comment | |
37 | ERROR | [x] Missing function doc comment | |
38 | ERROR | [x] Expected 1 newline at end of file; 0 found | |
38 | ERROR | [x] The closing brace for the interface must have an | |
| | empty line before it | |
---------------------------------------------------------------------- | |
PHPCBF CAN FIX THE 7 MARKED SNIFF VIOLATIONS AUTOMATICALLY | |
---------------------------------------------------------------------- | |
FILE: ...shu/webapps/drupal8/modules/securesite/src/SecuresiteManager.php | |
---------------------------------------------------------------------- | |
FOUND 85 ERRORS AND 13 WARNINGS AFFECTING 78 LINES | |
---------------------------------------------------------------------- | |
22 | ERROR | [x] Missing class doc comment | |
52 | ERROR | [x] Missing function doc comment | |
52 | ERROR | [x] Expected 1 space after closing parenthesis; | |
| | found 0 | |
81 | ERROR | [x] Case breaking statements must be followed by a | |
| | single blank line | |
95 | ERROR | [x] TRUE, FALSE and NULL must be uppercase; expected | |
| | "NULL" but found "null" | |
99 | ERROR | [x] Case breaking statements must be followed by a | |
| | single blank line | |
101 | ERROR | [x] TRUE, FALSE and NULL must be uppercase; expected | |
| | "NULL" but found "null" | |
101 | ERROR | [x] TRUE, FALSE and NULL must be uppercase; expected | |
| | "NULL" but found "null" | |
105 | ERROR | [x] Case breaking statements must be followed by a | |
| | single blank line | |
107 | ERROR | [x] There should be no white space after an opening | |
| | "(" | |
107 | ERROR | [x] Expected 1 space before "!="; 0 found | |
107 | ERROR | [x] TRUE, FALSE and NULL must be uppercase; expected | |
| | "NULL" but found "null" | |
122 | ERROR | [ ] Variable "currentUser" is camel caps format. do | |
| | not use mixed case (camelCase), use lower case | |
| | and _ | |
131 | ERROR | [x] Case breaking statements must be followed by a | |
| | single blank line | |
136 | ERROR | [x] Case breaking statements must be followed by a | |
| | single blank line | |
138 | ERROR | [x] No space found before comment text; expected "// | |
| | todo check if openid works" but found "//todo | |
| | check if openid works" | |
138 | ERROR | [x] Inline comments must start with a capital letter | |
138 | ERROR | [x] Inline comments must end in full-stops, | |
| | exclamation marks, or question marks | |
147 | ERROR | [ ] Variable "differentUser" is camel caps format. | |
| | do not use mixed case (camelCase), use lower | |
| | case and _ | |
147 | ERROR | [ ] Variable "currentUser" is camel caps format. do | |
| | not use mixed case (camelCase), use lower case | |
| | and _ | |
147 | ERROR | [ ] Variable "currentUser" is camel caps format. do | |
| | not use mixed case (camelCase), use lower case | |
| | and _ | |
148 | ERROR | [ ] Variable "notGuestLogin" is camel caps format. | |
| | do not use mixed case (camelCase), use lower | |
| | case and _ | |
150 | ERROR | [ ] Variable "differentUser" is camel caps format. | |
| | do not use mixed case (camelCase), use lower | |
| | case and _ | |
150 | ERROR | [ ] Variable "notGuestLogin" is camel caps format. | |
| | do not use mixed case (camelCase), use lower | |
| | case and _ | |
156 | ERROR | [x] Missing function doc comment | |
157 | WARNING | [ ] Line exceeds 80 characters; contains 82 | |
| | characters | |
157 | ERROR | [x] Inline comments must end in full-stops, | |
| | exclamation marks, or question marks | |
168 | ERROR | [x] There should be no white space after an opening | |
| | "(" | |
172 | ERROR | [x] Expected 1 space after IF keyword; 0 found | |
182 | ERROR | [x] There should be no white space after an opening | |
| | "(" | |
197 | ERROR | [x] Missing function doc comment | |
201 | ERROR | [ ] Variable "newUser" is camel caps format. do not | |
| | use mixed case (camelCase), use lower case and _ | |
202 | ERROR | [ ] Variable "newUser" is camel caps format. do not | |
| | use mixed case (camelCase), use lower case and _ | |
227 | ERROR | [x] Missing function doc comment | |
235 | ERROR | [x] Expected 1 space after IF keyword; 0 found | |
235 | ERROR | [x] Expected 1 space after closing parenthesis; | |
| | found "" | |
264 | ERROR | [x] Expected 1 space after closing parenthesis; | |
| | found 0 | |
267 | ERROR | [ ] If the line declaring an array spans longer than | |
| | 80 characters, each element should be broken | |
| | into its own line | |
276 | ERROR | [x] Case breaking statements must be followed by a | |
| | single blank line | |
293 | ERROR | [x] Case breaking statements must be followed by a | |
| | single blank line | |
295 | WARNING | [ ] Line exceeds 80 characters; contains 99 | |
| | characters | |
307 | ERROR | [x] Case breaking statements must be followed by a | |
| | single blank line | |
312 | WARNING | [ ] Line exceeds 80 characters; contains 96 | |
| | characters | |
324 | ERROR | [ ] Missing parameter type | |
326 | ERROR | [ ] Missing parameter type | |
328 | ERROR | [x] Separate the @param and @return sections by a | |
| | blank line. | |
328 | ERROR | [ ] Return type missing for @return tag in function | |
| | comment | |
331 | ERROR | [ ] Public method name | |
| | "SecuresiteManager::_securesite_digest_validate" | |
| | is not in lowerCamel format | |
341 | WARNING | [ ] Line exceeds 80 characters; contains 116 | |
| | characters | |
342 | WARNING | [ ] Line exceeds 80 characters; contains 108 | |
| | characters | |
343 | WARNING | [ ] Line exceeds 80 characters; contains 116 | |
| | characters | |
344 | ERROR | [x] Line indented incorrectly; expected 6 spaces, | |
| | found 0 | |
352 | ERROR | [x] Concat operator must be surrounded by a single | |
| | space | |
359 | ERROR | [x] Missing function doc comment | |
362 | ERROR | [x] Expected 1 space after "="; 2 found | |
364 | WARNING | [ ] Line exceeds 80 characters; contains 94 | |
| | characters | |
364 | ERROR | [x] No space found before comment text; expected "// | |
| | todo what is the use of the following if | |
| | statement? why get the form and not display it?" | |
| | but found "//todo what is the use of the | |
| | following if statement? why get the form and not | |
| | display it?" | |
373 | ERROR | [x] There should be no white space after an opening | |
| | "(" | |
377 | WARNING | [ ] Line exceeds 80 characters; contains 81 | |
| | characters | |
378 | ERROR | [x] Inline comments must end in full-stops, | |
| | exclamation marks, or question marks | |
394 | ERROR | [x] A unary operator statement must not be followed | |
| | by a space | |
408 | ERROR | [x] Case breaking statements must be followed by a | |
| | single blank line | |
419 | ERROR | [x] No space found before comment text; expected "// | |
| | todo fix next line" but found "//todo fix next | |
| | line" | |
419 | ERROR | [x] Inline comments must start with a capital letter | |
420 | ERROR | [x] No space found before comment text; expected "// | |
| | drupal_set_title(t('Authentication required'));" | |
| | but found "//drupal_set_title(t('Authentication | |
| | required'));" | |
425 | ERROR | [x] Inline comments must end in full-stops, | |
| | exclamation marks, or question marks | |
444 | WARNING | [ ] Line exceeds 80 characters; contains 102 | |
| | characters | |
447 | ERROR | [x] Expected 1 space after IF keyword; 0 found | |
447 | ERROR | [x] Expected 1 space after closing parenthesis; | |
| | found "" | |
450 | ERROR | [x] No space found before comment text; expected "// | |
| | todo find alternative" but found "//todo find | |
| | alternative" | |
450 | ERROR | [x] Inline comments must start with a capital letter | |
451 | WARNING | [x] There must be no blank line following an inline | |
| | comment | |
451 | ERROR | [x] No space found before comment text; expected "// | |
| | drupal_set_title(t('Access denied'));" but found | |
| | "//drupal_set_title(t('Access denied'));" | |
456 | ERROR | [x] Inline comments must end in full-stops, | |
| | exclamation marks, or question marks | |
460 | ERROR | [x] Inline comments must end in full-stops, | |
| | exclamation marks, or question marks | |
466 | WARNING | [ ] Line exceeds 80 characters; contains 85 | |
| | characters | |
467 | WARNING | [ ] Line exceeds 80 characters; contains 85 | |
| | characters | |
474 | ERROR | [x] No space found before comment text; expected "// | |
| | todo fix next few lines" but found "//todo fix | |
| | next few lines" | |
474 | ERROR | [x] Inline comments must start with a capital letter | |
474 | ERROR | [x] Inline comments must end in full-stops, | |
| | exclamation marks, or question marks | |
480 | ERROR | [x] Expected 1 space after IF keyword; 0 found | |
483 | ERROR | [x] Inline comments must end in full-stops, | |
| | exclamation marks, or question marks | |
487 | ERROR | [x] Inline comments must end in full-stops, | |
| | exclamation marks, or question marks | |
497 | ERROR | [ ] More than 2 empty lines are not allowed | |
498 | ERROR | [ ] More than 2 empty lines are not allowed | |
500 | WARNING | [ ] Line exceeds 80 characters; contains 81 | |
| | characters | |
502 | ERROR | [ ] Doc comment short description must be on a | |
| | single line, further text should be a separate | |
| | paragraph | |
504 | ERROR | [x] Expected 1 space after closing parenthesis; | |
| | found 0 | |
505 | ERROR | [ ] Variable "formBuilder" is camel caps format. do | |
| | not use mixed case (camelCase), use lower case | |
| | and _ | |
508 | ERROR | [ ] Variable "formBuilder" is camel caps format. do | |
| | not use mixed case (camelCase), use lower case | |
| | and _ | |
511 | ERROR | [ ] Variable "formBuilder" is camel caps format. do | |
| | not use mixed case (camelCase), use lower case | |
| | and _ | |
517 | ERROR | [ ] Variable "formBuilder" is camel caps format. do | |
| | not use mixed case (camelCase), use lower case | |
| | and _ | |
539 | ERROR | [x] Case breaking statements must be followed by a | |
| | single blank line | |
541 | ERROR | [x] Case breaking statements must be followed by a | |
| | single blank line | |
541 | ERROR | [x] Language constructs must be followed by a single | |
| | space; expected "return (" but found "return(" | |
551 | ERROR | [ ] Doc comment short description must be on a | |
| | single line, further text should be a separate | |
| | paragraph | |
563 | ERROR | [x] Missing function doc comment | |
568 | ERROR | [x] Missing function doc comment | |
---------------------------------------------------------------------- | |
PHPCBF CAN FIX THE 64 MARKED SNIFF VIOLATIONS AUTOMATICALLY | |
---------------------------------------------------------------------- | |
FILE: ...es/securesite/src/Tests/ForcedAuth/SecureSiteForceAlwaysTest.php | |
---------------------------------------------------------------------- | |
FOUND 21 ERRORS AND 2 WARNINGS AFFECTING 18 LINES | |
---------------------------------------------------------------------- | |
5 | ERROR | [x] Doc comment short description must end with a | |
| | full stop | |
6 | ERROR | [ ] There must be exactly one blank line after the | |
| | file comment | |
38 | ERROR | [ ] Visibility must be declared on method "setUp" | |
48 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteForceAlwaysNobody" | |
57 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteForceAlwaysUser" | |
69 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteForceAlwaysGuest" | |
70 | ERROR | [x] No space found before comment text; expected "// | |
| | todo why curl?" but found "//todo why curl?" | |
70 | ERROR | [x] Inline comments must start with a capital letter | |
83 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteForceAlwaysResetValid" | |
84 | WARNING | [ ] Line exceeds 80 characters; contains 90 | |
| | characters | |
84 | ERROR | [x] Comments may not appear after statements | |
86 | WARNING | [ ] Line exceeds 80 characters; contains 89 | |
| | characters | |
86 | ERROR | [x] Comments may not appear after statements | |
89 | ERROR | [x] No space found before comment text; expected "// | |
| | todo fix next line" but found "//todo fix next | |
| | line" | |
89 | ERROR | [x] Inline comments must start with a capital letter | |
89 | ERROR | [x] Inline comments must end in full-stops, | |
| | exclamation marks, or question marks | |
96 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteForceAlwaysResetInvalid" | |
105 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteForceAlwaysResetSubmit" | |
114 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteForceAlwaysCronAll" | |
115 | ERROR | [ ] If the line declaring an array spans longer than | |
| | 80 characters, each element should be broken | |
| | into its own line | |
124 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteForceAlwaysCronForm" | |
134 | ERROR | [ ] Visibility must be declared on method "tearDown" | |
138 | ERROR | [x] The closing brace for the class must have an | |
| | empty line before it | |
---------------------------------------------------------------------- | |
PHPCBF CAN FIX THE 9 MARKED SNIFF VIOLATIONS AUTOMATICALLY | |
---------------------------------------------------------------------- | |
FILE: ...s/securesite/src/Tests/ForcedAuth/SecureSiteForceOfflineTest.php | |
---------------------------------------------------------------------- | |
FOUND 8 ERRORS AFFECTING 7 LINES | |
---------------------------------------------------------------------- | |
4 | ERROR | [x] Doc comment short description must end with a full | |
| | stop | |
5 | ERROR | [ ] There must be exactly one blank line after the file | |
| | comment | |
12 | ERROR | [ ] Doc comment short description must be on a single | |
| | line, further text should be a separate paragraph | |
36 | ERROR | [ ] Visibility must be declared on method "setUp" | |
44 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteForceOfflineNormal" | |
52 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteForceOfflineMaintenance" | |
57 | ERROR | [x] Expected 1 newline at end of file; 0 found | |
57 | ERROR | [x] The closing brace for the class must have an empty | |
| | line before it | |
---------------------------------------------------------------------- | |
PHPCBF CAN FIX THE 3 MARKED SNIFF VIOLATIONS AUTOMATICALLY | |
---------------------------------------------------------------------- | |
FILE: ...dules/securesite/src/Tests/ForcedAuth/SecureSiteForce403Test.php | |
---------------------------------------------------------------------- | |
FOUND 8 ERRORS AFFECTING 8 LINES | |
---------------------------------------------------------------------- | |
4 | ERROR | [x] Doc comment short description must end with a full | |
| | stop | |
5 | ERROR | [ ] There must be exactly one blank line after the file | |
| | comment | |
13 | ERROR | [ ] Doc comment short description must be on a single | |
| | line, further text should be a separate paragraph | |
38 | ERROR | [ ] Visibility must be declared on method "setUp" | |
51 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteForce403Normal" | |
59 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteForce403Restricted" | |
67 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteForce403User" | |
72 | ERROR | [x] The closing brace for the class must have an empty | |
| | line before it | |
---------------------------------------------------------------------- | |
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY | |
---------------------------------------------------------------------- | |
FILE: ...ules/securesite/src/Tests/ForcedAuth/SecureSiteConfig403Test.php | |
---------------------------------------------------------------------- | |
FOUND 10 ERRORS AND 1 WARNING AFFECTING 10 LINES | |
---------------------------------------------------------------------- | |
4 | ERROR | [x] Doc comment short description must end with a | |
| | full stop | |
37 | ERROR | [ ] Visibility must be declared on method "setUp" | |
43 | WARNING | [ ] Line exceeds 80 characters; contains 85 | |
| | characters | |
45 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteConfig403Save" | |
47 | ERROR | [x] There should be no white space after an opening | |
| | "(" | |
53 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteConfig403ResetCurrent" | |
62 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteConfig403Page" | |
73 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteConfig403ResetPrevious" | |
82 | ERROR | [ ] Visibility must be declared on method "tearDown" | |
90 | ERROR | [x] Expected 1 newline at end of file; 0 found | |
90 | ERROR | [x] The closing brace for the class must have an | |
| | empty line before it | |
---------------------------------------------------------------------- | |
PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY | |
---------------------------------------------------------------------- | |
FILE: .../securesite/src/Tests/ForcedAuth/SecureSiteForceDisabledTest.php | |
---------------------------------------------------------------------- | |
FOUND 9 ERRORS AFFECTING 6 LINES | |
---------------------------------------------------------------------- | |
4 | ERROR | [x] Doc comment short description must end with a full | |
| | stop | |
5 | ERROR | [ ] There must be exactly one blank line after the file | |
| | comment | |
36 | ERROR | [ ] Visibility must be declared on method "setUp" | |
37 | ERROR | [x] No space found before comment text; expected "// | |
| | todo find a replacement" but found "//todo find a | |
| | replacement" | |
37 | ERROR | [x] Inline comments must start with a capital letter | |
37 | ERROR | [x] Inline comments must end in full-stops, exclamation | |
| | marks, or question marks | |
44 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteForceDisabled" | |
48 | ERROR | [x] Expected 1 newline at end of file; 0 found | |
48 | ERROR | [x] The closing brace for the class must have an empty | |
| | line before it | |
---------------------------------------------------------------------- | |
PHPCBF CAN FIX THE 6 MARKED SNIFF VIOLATIONS AUTOMATICALLY | |
---------------------------------------------------------------------- | |
FILE: ...modules/securesite/src/Tests/FormAuth/SecureSiteFormUserTest.php | |
---------------------------------------------------------------------- | |
FOUND 12 ERRORS AFFECTING 10 LINES | |
---------------------------------------------------------------------- | |
4 | ERROR | [x] Doc comment short description must end with a full | |
| | stop | |
24 | ERROR | [x] No space found before comment text; expected "// | |
| | todo phpdoc comments" but found "//todo phpdoc | |
| | comments" | |
24 | ERROR | [x] Inline comments must start with a capital letter | |
24 | ERROR | [x] Inline comments must end in full-stops, exclamation | |
| | marks, or question marks | |
25 | ERROR | [ ] Class property $normal_user should use lowerCamel | |
| | naming without underscores | |
27 | ERROR | [ ] Class property $access_user should use lowerCamel | |
| | naming without underscores | |
43 | ERROR | [ ] Visibility must be declared on method "setUp" | |
51 | ERROR | [ ] If the line declaring an array spans longer than 80 | |
| | characters, each element should be broken into its | |
| | own line | |
58 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteTypeFormUserWrong" | |
67 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteTypeFormUserNoAccess" | |
76 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteTypeFormUserAccess" | |
83 | ERROR | [x] The closing brace for the class must have an empty | |
| | line before it | |
---------------------------------------------------------------------- | |
PHPCBF CAN FIX THE 5 MARKED SNIFF VIOLATIONS AUTOMATICALLY | |
---------------------------------------------------------------------- | |
FILE: ...modules/securesite/src/Tests/FormAuth/SecureSiteFormNoneTest.php | |
---------------------------------------------------------------------- | |
FOUND 9 ERRORS AFFECTING 7 LINES | |
---------------------------------------------------------------------- | |
4 | ERROR | [x] Doc comment short description must end with a full | |
| | stop | |
5 | ERROR | [ ] There must be exactly one blank line after the file | |
| | comment | |
36 | ERROR | [ ] Visibility must be declared on method "setUp" | |
41 | ERROR | [ ] If the line declaring an array spans longer than 80 | |
| | characters, each element should be broken into its | |
| | own line | |
48 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteTypeFormNone" | |
50 | ERROR | [x] No space found before comment text; expected "// | |
| | todo form id might change" but found "//todo form | |
| | id might change" | |
50 | ERROR | [x] Inline comments must start with a capital letter | |
50 | ERROR | [x] Inline comments must end in full-stops, exclamation | |
| | marks, or question marks | |
53 | ERROR | [x] The closing brace for the class must have an empty | |
| | line before it | |
---------------------------------------------------------------------- | |
PHPCBF CAN FIX THE 5 MARKED SNIFF VIOLATIONS AUTOMATICALLY | |
---------------------------------------------------------------------- | |
FILE: ...odules/securesite/src/Tests/FormAuth/SecureSiteFormGuestTest.php | |
---------------------------------------------------------------------- | |
FOUND 8 ERRORS AFFECTING 7 LINES | |
---------------------------------------------------------------------- | |
4 | ERROR | [x] Doc comment short description must end with a full | |
| | stop | |
38 | ERROR | [ ] Visibility must be declared on method "setUp" | |
43 | ERROR | [ ] If the line declaring an array spans longer than 80 | |
| | characters, each element should be broken into its | |
| | own line | |
50 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteTypeFormGuestUnsetEmptyNoAccess" | |
59 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteTypeFormGuestUnsetRandomNoAccess" | |
68 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteTypeFormGuestUnsetRandomAccess" | |
74 | ERROR | [x] Expected 1 newline at end of file; 2 found | |
74 | ERROR | [x] The closing brace for the class must have an empty | |
| | line before it | |
---------------------------------------------------------------------- | |
PHPCBF CAN FIX THE 3 MARKED SNIFF VIOLATIONS AUTOMATICALLY | |
---------------------------------------------------------------------- | |
FILE: ...resite/src/Tests/DigestAuth/SecureSiteDigestUserUnstoredTest.php | |
---------------------------------------------------------------------- | |
FOUND 15 ERRORS AFFECTING 11 LINES | |
---------------------------------------------------------------------- | |
5 | ERROR | [x] Doc comment short description must end with a full | |
| | stop | |
6 | ERROR | [ ] There must be exactly one blank line after the file | |
| | comment | |
38 | ERROR | [ ] Visibility must be declared on method "setUp" | |
40 | ERROR | [x] No space found before comment text; expected "// | |
| | todo wtf is this function?" but found "//todo wtf | |
| | is this function?" | |
40 | ERROR | [x] Inline comments must start with a capital letter | |
46 | ERROR | [ ] If the line declaring an array spans longer than 80 | |
| | characters, each element should be broken into its | |
| | own line | |
48 | ERROR | [x] No space found before comment text; expected "// | |
| | todo curl options" but found "//todo curl options" | |
48 | ERROR | [x] Inline comments must start with a capital letter | |
48 | ERROR | [x] Inline comments must end in full-stops, exclamation | |
| | marks, or question marks | |
56 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteTypeDigestUserUnstoredBasic" | |
69 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteTypeDigestUserUnstoredForm" | |
79 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteTypeDigestUserUnstoredStore" | |
93 | ERROR | [ ] Visibility must be declared on method "tearDown" | |
97 | ERROR | [x] Expected 1 newline at end of file; 0 found | |
97 | ERROR | [x] The closing brace for the class must have an empty | |
| | line before it | |
---------------------------------------------------------------------- | |
PHPCBF CAN FIX THE 8 MARKED SNIFF VIOLATIONS AUTOMATICALLY | |
---------------------------------------------------------------------- | |
FILE: ...les/securesite/src/Tests/DigestAuth/SecureSiteDigestNoneTest.php | |
---------------------------------------------------------------------- | |
FOUND 9 ERRORS AFFECTING 7 LINES | |
---------------------------------------------------------------------- | |
5 | ERROR | [x] Doc comment short description must end with a full | |
| | stop | |
6 | ERROR | [ ] There must be exactly one blank line after the file | |
| | comment | |
36 | ERROR | [ ] Visibility must be declared on method "setUp" | |
38 | ERROR | [x] No space found before comment text; expected "// | |
| | todo wtf is this function" but found "//todo wtf is | |
| | this function" | |
38 | ERROR | [x] Inline comments must start with a capital letter | |
38 | ERROR | [x] Inline comments must end in full-stops, exclamation | |
| | marks, or question marks | |
43 | ERROR | [ ] If the line declaring an array spans longer than 80 | |
| | characters, each element should be broken into its | |
| | own line | |
50 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteTypeDigestNone" | |
61 | ERROR | [x] The closing brace for the class must have an empty | |
| | line before it | |
---------------------------------------------------------------------- | |
PHPCBF CAN FIX THE 5 MARKED SNIFF VIOLATIONS AUTOMATICALLY | |
---------------------------------------------------------------------- | |
FILE: ...curesite/src/Tests/DigestAuth/SecureSiteDigestUserStoredTest.php | |
---------------------------------------------------------------------- | |
FOUND 14 ERRORS AFFECTING 10 LINES | |
---------------------------------------------------------------------- | |
5 | ERROR | [x] Doc comment short description must end with a full | |
| | stop | |
6 | ERROR | [ ] There must be exactly one blank line after the file | |
| | comment | |
39 | ERROR | [ ] Visibility must be declared on method "setUp" | |
41 | ERROR | [x] No space found before comment text; expected "// | |
| | todo wtf is this function" but found "//todo wtf is | |
| | this function" | |
41 | ERROR | [x] Inline comments must start with a capital letter | |
41 | ERROR | [x] Inline comments must end in full-stops, exclamation | |
| | marks, or question marks | |
46 | ERROR | [ ] If the line declaring an array spans longer than 80 | |
| | characters, each element should be broken into its | |
| | own line | |
49 | ERROR | [x] No space found before comment text; expected "// | |
| | todo curl options" but found "//todo curl options" | |
49 | ERROR | [x] Inline comments must start with a capital letter | |
49 | ERROR | [x] Inline comments must end in full-stops, exclamation | |
| | marks, or question marks | |
56 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteTypeDigestUserStoredWrong" | |
65 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteTypeDigestUserStoredCorrect" | |
79 | ERROR | [ ] Visibility must be declared on method "tearDown" | |
83 | ERROR | [x] The closing brace for the class must have an empty | |
| | line before it | |
---------------------------------------------------------------------- | |
PHPCBF CAN FIX THE 8 MARKED SNIFF VIOLATIONS AUTOMATICALLY | |
---------------------------------------------------------------------- | |
FILE: ...curesite/src/Tests/DigestAuth/SecureSiteDigestGuestUnsetTest.php | |
---------------------------------------------------------------------- | |
FOUND 12 ERRORS AFFECTING 9 LINES | |
---------------------------------------------------------------------- | |
5 | ERROR | [x] Doc comment short description must end with a full | |
| | stop | |
6 | ERROR | [ ] There must be exactly one blank line after the file | |
| | comment | |
37 | ERROR | [ ] Visibility must be declared on method "setUp" | |
39 | ERROR | [x] No space found before comment text; expected "// | |
| | todo wtf is this function?" but found "//todo wtf | |
| | is this function?" | |
39 | ERROR | [x] Inline comments must start with a capital letter | |
45 | ERROR | [ ] If the line declaring an array spans longer than 80 | |
| | characters, each element should be broken into its | |
| | own line | |
47 | ERROR | [x] No space found before comment text; expected "// | |
| | todo curl options" but found "//todo curl options" | |
47 | ERROR | [x] Inline comments must start with a capital letter | |
47 | ERROR | [x] Inline comments must end in full-stops, exclamation | |
| | marks, or question marks | |
54 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteTypeDigestGuestUnsetEmpty" | |
64 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteTypeDigestGuestUnsetRandom" | |
70 | ERROR | [x] The closing brace for the class must have an empty | |
| | line before it | |
---------------------------------------------------------------------- | |
PHPCBF CAN FIX THE 7 MARKED SNIFF VIOLATIONS AUTOMATICALLY | |
---------------------------------------------------------------------- | |
FILE: ...securesite/src/Tests/DigestAuth/SecureSiteDigestGuestSetTest.php | |
---------------------------------------------------------------------- | |
FOUND 20 ERRORS AFFECTING 15 LINES | |
---------------------------------------------------------------------- | |
4 | ERROR | [x] Doc comment short description must end with a full | |
| | stop | |
5 | ERROR | [ ] There must be exactly one blank line after the | |
| | file comment | |
22 | ERROR | [x] No space found before comment text; expected "// | |
| | todo phpdoc" but found "//todo phpdoc" | |
22 | ERROR | [x] Inline comments must start with a capital letter | |
22 | ERROR | [x] Inline comments must end in full-stops, | |
| | exclamation marks, or question marks | |
24 | ERROR | [ ] Class property $guest_name should use lowerCamel | |
| | naming without underscores | |
25 | ERROR | [ ] Class property $guest_pass should use lowerCamel | |
| | naming without underscores | |
40 | ERROR | [ ] Visibility must be declared on method "setUp" | |
42 | ERROR | [x] No space found before comment text; expected "// | |
| | todo wtf is this function?" but found "//todo wtf | |
| | is this function?" | |
42 | ERROR | [x] Inline comments must start with a capital letter | |
48 | ERROR | [ ] If the line declaring an array spans longer than | |
| | 80 characters, each element should be broken into | |
| | its own line | |
50 | ERROR | [x] No space found before comment text; expected "// | |
| | todo curl options" but found "//todo curl options" | |
50 | ERROR | [x] Inline comments must start with a capital letter | |
50 | ERROR | [x] Inline comments must end in full-stops, | |
| | exclamation marks, or question marks | |
57 | ERROR | [ ] If the line declaring an array spans longer than | |
| | 80 characters, each element should be broken into | |
| | its own line | |
64 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteDigestGuestSetEmpty" | |
73 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteDigestGuestSetRandom" | |
82 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteDigestGuestSetCorrect" | |
94 | ERROR | [ ] Visibility must be declared on method "tearDown" | |
102 | ERROR | [x] The closing brace for the class must have an empty | |
| | line before it | |
---------------------------------------------------------------------- | |
PHPCBF CAN FIX THE 10 MARKED SNIFF VIOLATIONS AUTOMATICALLY | |
---------------------------------------------------------------------- | |
FILE: ...securesite/src/Tests/BasicAuth/SecureSiteBasicGuestUnsetTest.php | |
---------------------------------------------------------------------- | |
FOUND 16 ERRORS AFFECTING 13 LINES | |
---------------------------------------------------------------------- | |
4 | ERROR | [x] Doc comment short description must end with a full | |
| | stop | |
5 | ERROR | [ ] There must be exactly one blank line after the | |
| | file comment | |
11 | ERROR | [ ] More than 2 empty lines are not allowed | |
38 | ERROR | [ ] Visibility must be declared on method "setUp" | |
42 | ERROR | [ ] If the line declaring an array spans longer than | |
| | 80 characters, each element should be broken into | |
| | its own line | |
43 | ERROR | [x] No space found before comment text; expected "// | |
| | todo curl options" but found "//todo curl options" | |
43 | ERROR | [x] Inline comments must start with a capital letter | |
43 | ERROR | [x] Inline comments must end in full-stops, | |
| | exclamation marks, or question marks | |
50 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteTypeBasicGuestUnsetEmptyNoAccess" | |
61 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteTypeBasicGuestUnsetEmptyAccess" | |
69 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteTypeBasicGuestUnsetRandomNoAccess" | |
79 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteTypeBasicGuestUnsetRandomAccess" | |
88 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteTypeBasicGuestUnsetChange" | |
99 | ERROR | [ ] Visibility must be declared on method "tearDown" | |
103 | ERROR | [x] Expected 1 newline at end of file; 0 found | |
103 | ERROR | [x] The closing brace for the class must have an empty | |
| | line before it | |
---------------------------------------------------------------------- | |
PHPCBF CAN FIX THE 6 MARKED SNIFF VIOLATIONS AUTOMATICALLY | |
---------------------------------------------------------------------- | |
FILE: ...dules/securesite/src/Tests/BasicAuth/SecureSiteBasicNoneTest.php | |
---------------------------------------------------------------------- | |
FOUND 5 ERRORS AFFECTING 5 LINES | |
---------------------------------------------------------------------- | |
4 | ERROR | [x] Doc comment short description must end with a full | |
| | stop | |
5 | ERROR | [ ] There must be exactly one blank line after the file | |
| | comment | |
36 | ERROR | [ ] Visibility must be declared on method "setUp" | |
44 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteTypeBasicNone" | |
53 | ERROR | [x] The closing brace for the class must have an empty | |
| | line before it | |
---------------------------------------------------------------------- | |
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY | |
---------------------------------------------------------------------- | |
FILE: ...s/securesite/src/Tests/BasicAuth/SecureSiteBasicGuestSetTest.php | |
---------------------------------------------------------------------- | |
FOUND 17 ERRORS AFFECTING 12 LINES | |
---------------------------------------------------------------------- | |
5 | ERROR | [x] Doc comment short description must end with a full | |
| | stop | |
6 | ERROR | [ ] There must be exactly one blank line after the file | |
| | comment | |
12 | ERROR | [ ] More than 2 empty lines are not allowed | |
25 | ERROR | [x] No space found before comment text; expected "// | |
| | todo phpdoc comments" but found "//todo phpdoc | |
| | comments" | |
25 | ERROR | [x] Inline comments must start with a capital letter | |
25 | ERROR | [x] Inline comments must end in full-stops, exclamation | |
| | marks, or question marks | |
45 | ERROR | [ ] Visibility must be declared on method "setUp" | |
54 | ERROR | [ ] If the line declaring an array spans longer than 80 | |
| | characters, each element should be broken into its | |
| | own line | |
61 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteTypeBasicGuestSetEmpty" | |
62 | ERROR | [x] No space found before comment text; expected "// | |
| | todo curl options" but found "//todo curl options" | |
62 | ERROR | [x] Inline comments must start with a capital letter | |
62 | ERROR | [x] Inline comments must end in full-stops, exclamation | |
| | marks, or question marks | |
73 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteTypeBasicGuestSetWrong" | |
82 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteTypeBasicGuestSetCorrect" | |
91 | ERROR | [ ] Visibility must be declared on method "tearDown" | |
95 | ERROR | [x] Expected 1 newline at end of file; 0 found | |
95 | ERROR | [x] The closing brace for the class must have an empty | |
| | line before it | |
---------------------------------------------------------------------- | |
PHPCBF CAN FIX THE 9 MARKED SNIFF VIOLATIONS AUTOMATICALLY | |
---------------------------------------------------------------------- | |
FILE: ...dules/securesite/src/Tests/BasicAuth/SecureSiteBasicUserTest.php | |
---------------------------------------------------------------------- | |
FOUND 19 ERRORS AFFECTING 14 LINES | |
---------------------------------------------------------------------- | |
4 | ERROR | [x] Doc comment short description must end with a full | |
| | stop | |
5 | ERROR | [ ] There must be exactly one blank line after the | |
| | file comment | |
22 | ERROR | [x] No space found before comment text; expected "// | |
| | todo phpdoc comments" but found "//todo phpdoc | |
| | comments" | |
22 | ERROR | [x] Inline comments must start with a capital letter | |
22 | ERROR | [x] Inline comments must end in full-stops, | |
| | exclamation marks, or question marks | |
23 | ERROR | [ ] Class property $normal_user should use lowerCamel | |
| | naming without underscores | |
25 | ERROR | [ ] Class property $access_user should use lowerCamel | |
| | naming without underscores | |
41 | ERROR | [ ] Visibility must be declared on method "setUp" | |
47 | ERROR | [ ] If the line declaring an array spans longer than | |
| | 80 characters, each element should be broken into | |
| | its own line | |
53 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteTypeBasicUserWrong" | |
54 | ERROR | [x] No space found before comment text; expected "// | |
| | todo - curl options" but found "//todo - curl | |
| | options" | |
54 | ERROR | [x] Inline comments must start with a capital letter | |
54 | ERROR | [x] Inline comments must end in full-stops, | |
| | exclamation marks, or question marks | |
63 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteTypeBasicUserNoAccess" | |
74 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteTypeBasicUserAccess" | |
88 | ERROR | [ ] Visibility must be declared on method | |
| | "testSecureSiteTypeBasicUserChange" | |
99 | ERROR | [ ] Visibility must be declared on method "tearDown" | |
103 | ERROR | [x] Expected 1 newline at end of file; 0 found | |
103 | ERROR | [x] The closing brace for the class must have an empty | |
| | line before it | |
---------------------------------------------------------------------- | |
PHPCBF CAN FIX THE 9 MARKED SNIFF VIOLATIONS AUTOMATICALLY | |
---------------------------------------------------------------------- | |
FILE: .../modules/securesite/src/EventSubscriber/SecuresiteSubscriber.php | |
---------------------------------------------------------------------- | |
FOUND 13 ERRORS AND 4 WARNINGS AFFECTING 11 LINES | |
---------------------------------------------------------------------- | |
17 | WARNING | [ ] Line exceeds 80 characters; contains 85 | |
| | characters | |
17 | ERROR | [x] Doc comment short description must end with a | |
| | full stop | |
28 | WARNING | [x] There must be no blank line following an inline | |
| | comment | |
28 | ERROR | [x] Inline comments must start with a capital letter | |
36 | ERROR | [x] Expected 1 space after closing parenthesis; | |
| | found 0 | |
41 | ERROR | [x] Doc comment short description must end with a | |
| | full stop | |
50 | WARNING | [ ] Line exceeds 80 characters; contains 101 | |
| | characters | |
50 | ERROR | [x] No space found before comment text; expected "// | |
| | creating an array of headers to be added to the | |
| | response. This array will be populated later on" | |
| | but found "//creating an array of headers to be | |
| | added to the response. This array will be | |
| | populated later on" | |
50 | ERROR | [x] Inline comments must start with a capital letter | |
50 | ERROR | [x] Inline comments must end in full-stops, | |
| | exclamation marks, or question marks | |
60 | WARNING | [ ] Line exceeds 80 characters; contains 85 | |
| | characters | |
74 | ERROR | [x] Doc comment short description must end with a | |
| | full stop | |
83 | ERROR | [x] Expected 1 space after IF keyword; 0 found | |
85 | ERROR | [x] Expected 1 space after IF keyword; 0 found | |
85 | ERROR | [x] Inline control structures are not allowed | |
95 | ERROR | [ ] More than 2 empty lines are not allowed | |
102 | ERROR | [ ] Visibility must be declared on method | |
| | "getSubscribedEvents" | |
---------------------------------------------------------------------- | |
PHPCBF CAN FIX THE 12 MARKED SNIFF VIOLATIONS AUTOMATICALLY | |
---------------------------------------------------------------------- | |
FILE: ...apps/drupal8/modules/securesite/src/Form/SecuresiteLoginForm.php | |
---------------------------------------------------------------------- | |
FOUND 3 ERRORS AFFECTING 3 LINES | |
---------------------------------------------------------------------- | |
28 | ERROR | [ ] There must be no blank lines after the function | |
| | comment | |
66 | ERROR | [x] Functions must not contain multiple empty lines in | |
| | a row; found 2 empty lines | |
70 | ERROR | [x] Expected 1 newline at end of file; 0 found | |
---------------------------------------------------------------------- | |
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY | |
---------------------------------------------------------------------- | |
FILE: ...s/drupal8/modules/securesite/src/Form/SecuresiteSettingsForm.php | |
---------------------------------------------------------------------- | |
FOUND 11 ERRORS AND 4 WARNINGS AFFECTING 12 LINES | |
---------------------------------------------------------------------- | |
15 | ERROR | [x] Missing class doc comment | |
33 | ERROR | [ ] If the line declaring an array spans longer than | |
| | 80 characters, each element should be broken | |
| | into its own line | |
33 | WARNING | [x] A comma should follow the last multiline array | |
| | item. Found: ) | |
58 | ERROR | [ ] If the line declaring an array spans longer than | |
| | 80 characters, each element should be broken | |
| | into its own line | |
60 | ERROR | [ ] If the line declaring an array spans longer than | |
| | 80 characters, each element should be broken | |
| | into its own line | |
87 | ERROR | [ ] If the line declaring an array spans longer than | |
| | 80 characters, each element should be broken | |
| | into its own line | |
111 | WARNING | [x] A comma should follow the last multiline array | |
| | item. Found: ) | |
135 | ERROR | [x] Expected 1 space after closing parenthesis; | |
| | found 0 | |
151 | ERROR | [x] Doc comment short description must start with a | |
| | capital letter | |
152 | ERROR | [ ] Doc comment short description must be on a | |
| | single line, further text should be a separate | |
| | paragraph | |
187 | WARNING | [ ] Line exceeds 80 characters; contains 113 | |
| | characters | |
187 | ERROR | [x] Comments may not appear after statements | |
188 | WARNING | [ ] Line exceeds 80 characters; contains 111 | |
| | characters | |
188 | ERROR | [x] Comments may not appear after statements | |
189 | ERROR | [x] Comments may not appear after statements | |
---------------------------------------------------------------------- | |
PHPCBF CAN FIX THE 8 MARKED SNIFF VIOLATIONS AUTOMATICALLY | |
---------------------------------------------------------------------- | |
FILE: /home/shivanshu/webapps/drupal8/modules/securesite/README.txt | |
---------------------------------------------------------------------- | |
FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 3 LINES | |
---------------------------------------------------------------------- | |
159 | WARNING | Line exceeds 80 characters; contains 84 characters | |
160 | WARNING | Line exceeds 80 characters; contains 96 characters | |
161 | WARNING | Line exceeds 80 characters; contains 109 characters | |
---------------------------------------------------------------------- | |
Time: 2.15 secs; Memory: 16.5Mb |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment