Closed (fixed)
Project:
Basic Cart
Version:
8.x-8.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
20 Oct 2025 at 21:29 UTC
Updated:
9 Nov 2025 at 22:04 UTC
Jump to comment: Most recent
FILE: /Users/norman/Sites/drupal11/web/modules/contrib/basic_cart/basic_cart.install
---------------------------------------------------------------------------------
FOUND 14 ERRORS AFFECTING 14 LINES
---------------------------------------------------------------------------------
19 | ERROR | [x] Short array syntax must be used to define arrays
21 | ERROR | [x] Short array syntax must be used to define arrays
22 | ERROR | [x] Short array syntax must be used to define arrays
28 | ERROR | [x] Short array syntax must be used to define arrays
34 | ERROR | [x] Short array syntax must be used to define arrays
41 | ERROR | [x] Short array syntax must be used to define arrays
48 | ERROR | [x] Short array syntax must be used to define arrays
51 | ERROR | [x] Short array syntax must be used to define arrays
53 | ERROR | [x] Short array syntax must be used to define arrays
54 | ERROR | [x] Short array syntax must be used to define arrays
60 | ERROR | [x] Short array syntax must be used to define arrays
66 | ERROR | [x] Short array syntax must be used to define arrays
73 | ERROR | [x] Short array syntax must be used to define arrays
80 | ERROR | [x] Short array syntax must be used to define arrays
---------------------------------------------------------------------------------
PHPCBF CAN FIX THE 14 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------
FILE: /Users/norman/Sites/drupal11/web/modules/contrib/basic_cart/basic_cart.module
---------------------------------------------------------------------------------------------
FOUND 33 ERRORS AND 1 WARNING AFFECTING 29 LINES
---------------------------------------------------------------------------------------------
10 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is
| | Drupal\Core\Entity\EntityInterface.
24 | ERROR | [x] Short array syntax must be used to define arrays
25 | ERROR | [x] Short array syntax must be used to define arrays
26 | ERROR | [x] Short array syntax must be used to define arrays
28 | ERROR | [x] Short array syntax must be used to define arrays
29 | ERROR | [x] Short array syntax must be used to define arrays
31 | ERROR | [x] Short array syntax must be used to define arrays
34 | ERROR | [x] Short array syntax must be used to define arrays
35 | ERROR | [x] Short array syntax must be used to define arrays
37 | ERROR | [x] Short array syntax must be used to define arrays
38 | ERROR | [x] Short array syntax must be used to define arrays
40 | ERROR | [x] Short array syntax must be used to define arrays
41 | ERROR | [x] Short array syntax must be used to define arrays
43 | ERROR | [x] Short array syntax must be used to define arrays
44 | ERROR | [x] Short array syntax must be used to define arrays
82 | ERROR | [x] Short array syntax must be used to define arrays
89 | ERROR | [x] Expected 1 space before "!="; 0 found
89 | ERROR | [x] Expected 1 space after "!="; 0 found
89 | ERROR | [x] Expected 1 space after "?"; 0 found
89 | ERROR | [x] Expected 1 space before ":"; 0 found
89 | ERROR | [x] Expected 1 space after ":"; 0 found
134 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 1
135 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
166 | ERROR | [x] Short array syntax must be used to define arrays
167 | ERROR | [x] Short array syntax must be used to define arrays
213 | ERROR | [x] Short array syntax must be used to define arrays
214 | ERROR | [x] Short array syntax must be used to define arrays
265 | ERROR | [x] Short array syntax must be used to define arrays
267 | ERROR | [x] Use null coalesce operator instead of ternary operator.
267 | ERROR | [x] Short array syntax must be used to define arrays
313 | ERROR | [x] Use null coalesce operator instead of ternary operator.
324 | ERROR | [x] Use null coalesce operator instead of ternary operator.
337 | ERROR | [x] Short array syntax must be used to define arrays
342 | WARNING | [x] A comma should follow the last multiline array item. Found: )
---------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 34 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------
FILE: /Users/norman/Sites/drupal11/web/modules/contrib/basic_cart/src/Form/CartSettingsForm.php
---------------------------------------------------------------------------------------------
FOUND 56 ERRORS AND 44 WARNINGS AFFECTING 89 LINES
---------------------------------------------------------------------------------------------
8 | ERROR | [x] When importing a class with "use", do not include a leading \
35 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection
| | instead
36 | WARNING | [ ] NodeType::loadMultiple calls should be avoided in classes, use
| | dependency injection instead
41 | ERROR | [x] Expected 1 space after IF keyword; 0 found
41 | ERROR | [x] Expected 1 space before "=="; 0 found
41 | ERROR | [x] Expected 1 space after "=="; 0 found
41 | ERROR | [x] Expected 1 space after closing parenthesis; found 0
41 | ERROR | [x] Whitespace found at end of line
42 | ERROR | [x] Whitespace found at end of line
43 | ERROR | [x] Line indented incorrectly; expected 6 spaces, found 16
44 | ERROR | [x] Line indented incorrectly; expected 8 spaces, found 6
44 | ERROR | [x] Object operator not indented correctly; expected 18 spaces but found 6
44 | ERROR | [x] Expected one space after the comma, 0 found
46 | ERROR | [x] Expected 1 space after IF keyword; 0 found
46 | ERROR | [x] Expected 1 space before "=="; 0 found
46 | ERROR | [x] Expected 1 space after "=="; 0 found
46 | ERROR | [x] Expected 1 space after closing parenthesis; found 0
47 | ERROR | [x] Line indented incorrectly; expected 6 spaces, found 5
48 | ERROR | [x] Line indented incorrectly; expected 8 spaces, found 6
48 | ERROR | [x] Object operator not indented correctly; expected 7 spaces but found 6
48 | ERROR | [x] Expected one space after the comma, 0 found
49 | ERROR | [x] Whitespace found at end of line
51 | ERROR | [x] Functions must not contain multiple empty lines in a row; found 2 empty
| | lines
52 | ERROR | [x] Whitespace found at end of line
53 | ERROR | [x] Short array syntax must be used to define arrays
61 | ERROR | [x] Short array syntax must be used to define arrays
62 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
64 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
67 | ERROR | [x] Short array syntax must be used to define arrays
68 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
74 | ERROR | [x] Short array syntax must be used to define arrays
79 | ERROR | [x] Short array syntax must be used to define arrays
80 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
82 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
85 | ERROR | [x] Short array syntax must be used to define arrays
86 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
88 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
92 | ERROR | [x] Short array syntax must be used to define arrays
93 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
95 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
98 | ERROR | [x] Short array syntax must be used to define arrays
99 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
101 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
105 | ERROR | [x] Short array syntax must be used to define arrays
106 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
108 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
112 | ERROR | [x] Short array syntax must be used to define arrays
113 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
116 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
120 | ERROR | [x] Short array syntax must be used to define arrays
124 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
127 | ERROR | [x] Short array syntax must be used to define arrays
131 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
134 | ERROR | [x] Short array syntax must be used to define arrays
138 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
141 | ERROR | [x] Short array syntax must be used to define arrays
142 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
146 | ERROR | [x] Short array syntax must be used to define arrays
147 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
152 | ERROR | [x] Short array syntax must be used to define arrays
153 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
155 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
161 | ERROR | [x] Short array syntax must be used to define arrays
162 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
166 | ERROR | [x] Short array syntax must be used to define arrays
167 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
172 | ERROR | [x] Short array syntax must be used to define arrays
173 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
177 | ERROR | [x] Short array syntax must be used to define arrays
178 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
180 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
184 | ERROR | [x] Short array syntax must be used to define arrays
185 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
190 | ERROR | [x] Short array syntax must be used to define arrays
194 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
197 | ERROR | [x] Short array syntax must be used to define arrays
201 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
204 | ERROR | [x] Short array syntax must be used to define arrays
208 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
211 | ERROR | [x] Short array syntax must be used to define arrays
215 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
218 | ERROR | [x] Short array syntax must be used to define arrays
222 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
225 | ERROR | [x] Short array syntax must be used to define arrays
229 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
232 | ERROR | [x] Short array syntax must be used to define arrays
236 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
239 | ERROR | [x] Short array syntax must be used to define arrays
243 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
246 | ERROR | [x] Short array syntax must be used to define arrays
250 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
253 | ERROR | [x] Short array syntax must be used to define arrays
257 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
260 | ERROR | [x] Short array syntax must be used to define arrays
264 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
267 | ERROR | [x] Short array syntax must be used to define arrays
271 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
274 | ERROR | [x] Short array syntax must be used to define arrays
278 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
317 | WARNING | [ ] Unused variable $key.
---------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 56 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------
FILE: /Users/norman/Sites/drupal11/web/modules/contrib/basic_cart/src/Form/CartForm.php
---------------------------------------------------------------------------------------------
FOUND 18 ERRORS AND 6 WARNINGS AFFECTING 15 LINES
---------------------------------------------------------------------------------------------
8 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is
| | Drupal\Core\Url.
28 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection
| | instead
32 | ERROR | [x] No space found before comment text; expected "// $langcode =
| | \Drupal::languageManager()->getCurrentLanguage()->getId();" but found
| | "//$langcode =
| | \Drupal::languageManager()->getCurrentLanguage()->getId();"
33 | ERROR | [x] No space found before comment text; expected "// $price =
| | Utility::getTotalPrice();" but found "//$price =
| | Utility::getTotalPrice();"
34 | ERROR | [x] No space found before comment text; expected "// $total =
| | Utility::formatPrice($price->total);" but found "//$total =
| | Utility::formatPrice($price->total);"
35 | ERROR | [x] No space found before comment text; expected "// $vat_is_enabled = (int)
| | $config->get('vat_state');" but found "//$vat_is_enabled = (int)
| | $config->get('vat_state');"
36 | WARNING | [ ] Line exceeds 80 characters; contains 102 characters
36 | WARNING | [x] There must be no blank line following an inline comment
36 | WARNING | [ ] There must be no blank line following an inline comment
36 | ERROR | [x] Line indented incorrectly; expected 4 spaces, found 3
39 | ERROR | [x] Short array syntax must be used to define arrays
46 | ERROR | [x] Short array syntax must be used to define arrays
55 | ERROR | [x] Short array syntax must be used to define arrays
60 | ERROR | [x] Short array syntax must be used to define arrays
64 | ERROR | [x] Short array syntax must be used to define arrays
66 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
66 | WARNING | [ ] Only string literals should be passed to t() where possible
71 | ERROR | [x] Short array syntax must be used to define arrays
73 | ERROR | [x] Expected 1 space after "=>"; 0 found
73 | ERROR | [x] Expected 1 space before "!="; 0 found
73 | ERROR | [x] Expected 1 space after "!="; 0 found
73 | ERROR | [x] Expected 1 space after "?"; 0 found
73 | ERROR | [x] Expected 1 space before ":"; 0 found
73 | ERROR | [x] Expected 1 space after ":"; 0 found
---------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 19 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------
FILE: ...ers/norman/Sites/drupal11/web/modules/contrib/basic_cart/src/Form/CheckOutSettingsForm.php
---------------------------------------------------------------------------------------------
FOUND 16 ERRORS AND 21 WARNINGS AFFECTING 35 LINES
---------------------------------------------------------------------------------------------
34 | ERROR | [x] Short array syntax must be used to define arrays
35 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
37 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
40 | ERROR | [x] Short array syntax must be used to define arrays
41 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
43 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
44 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection
| | instead
47 | ERROR | [x] Short array syntax must be used to define arrays
48 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
50 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
54 | ERROR | [x] Short array syntax must be used to define arrays
55 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
57 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
61 | ERROR | [x] Short array syntax must be used to define arrays
67 | ERROR | [x] Short array syntax must be used to define arrays
68 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
70 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
74 | ERROR | [x] Short array syntax must be used to define arrays
75 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
77 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
81 | ERROR | [x] Short array syntax must be used to define arrays
82 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
84 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
87 | ERROR | [x] Short array syntax must be used to define arrays
88 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
90 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
94 | ERROR | [x] Short array syntax must be used to define arrays
95 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
97 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
101 | ERROR | [x] Short array syntax must be used to define arrays
102 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
104 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
115 | ERROR | [x] Short array syntax must be used to define arrays
115 | ERROR | [ ] The array declaration extends to column 148 (the limit is 120). The
| | array content should be split up over multiple lines
116 | ERROR | [x] Short array syntax must be used to define arrays
116 | ERROR | [ ] The array declaration extends to column 140 (the limit is 120). The
| | array content should be split up over multiple lines
117 | ERROR | [x] Short array syntax must be used to define arrays
---------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 14 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------
FILE: /Users/norman/Sites/drupal11/web/modules/contrib/basic_cart/src/CartStorageSelect.php
----------------------------------------------------------------------------------------
FOUND 7 ERRORS AFFECTING 6 LINES
----------------------------------------------------------------------------------------
10 | ERROR | [ ] Missing member variable doc comment
11 | ERROR | [ ] Missing member variable doc comment
44 | ERROR | [x] Scope keyword "public" must be followed by a single space; found 2
57 | ERROR | [x] Scope keyword "public" must be followed by a single space; found 2
67 | ERROR | [x] Scope keyword "public" must be followed by a single space; found 2
82 | ERROR | [x] Scope keyword "public" must be followed by a single space; found 2
82 | ERROR | [x] Short array syntax must be used to define arrays
----------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 5 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------
FILE: /Users/norman/Sites/drupal11/web/modules/contrib/basic_cart/src/Settings.php
----------------------------------------------------------------------------------------
FOUND 14 ERRORS AND 1 WARNING AFFECTING 15 LINES
----------------------------------------------------------------------------------------
9 | ERROR | [ ] Missing member variable doc comment
10 | ERROR | [ ] Missing member variable doc comment
19 | ERROR | [x] Scope keyword "public" must be followed by a single space; found 2
38 | ERROR | [x] Scope keyword "static" must be followed by a single space; found 2
40 | WARNING | [ ] Only string literals should be passed to t() where possible
107 | ERROR | [x] Short array syntax must be used to define arrays
108 | ERROR | [x] Short array syntax must be used to define arrays
109 | ERROR | [x] Short array syntax must be used to define arrays
110 | ERROR | [x] Short array syntax must be used to define arrays
111 | ERROR | [x] Short array syntax must be used to define arrays
113 | ERROR | [x] Short array syntax must be used to define arrays
114 | ERROR | [x] Short array syntax must be used to define arrays
115 | ERROR | [x] Short array syntax must be used to define arrays
116 | ERROR | [x] Short array syntax must be used to define arrays
130 | ERROR | [x] Short array syntax must be used to define arrays
----------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 12 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------
FILE: /Users/norman/Sites/drupal11/web/modules/contrib/basic_cart/src/CartStorage.php
---------------------------------------------------------------------------------------------
FOUND 3 ERRORS AND 13 WARNINGS AFFECTING 12 LINES
---------------------------------------------------------------------------------------------
7 | WARNING | [ ] The class short comment should describe what the class does and not
| | simply repeat the class name
29 | WARNING | [ ] Line exceeds 80 characters; contains 113 characters
29 | WARNING | [x] 'TODO: Drupal Rector Notice: Please delete the following comment after
| | you've made any necessary changes.' should match the format '@todo Fix
| | problem X here.'
30 | WARNING | [ ] Line exceeds 80 characters; contains 132 characters
36 | ERROR | [x] Short array syntax must be used to define arrays
59 | WARNING | [ ] Line exceeds 80 characters; contains 113 characters
59 | WARNING | [x] 'TODO: Drupal Rector Notice: Please delete the following comment after
| | you've made any necessary changes.' should match the format '@todo Fix
| | problem X here.'
60 | WARNING | [ ] Line exceeds 80 characters; contains 132 characters
69 | ERROR | [x] Short array syntax must be used to define arrays
88 | WARNING | [ ] Line exceeds 80 characters; contains 111 characters
88 | WARNING | [x] 'TODO: Drupal Rector Notice: Please delete the following comment after
| | you've made any necessary changes.' should match the format '@todo Fix
| | problem X here.'
89 | WARNING | [ ] Line exceeds 80 characters; contains 130 characters
116 | ERROR | [x] Short array syntax must be used to define arrays
118 | WARNING | [ ] Line exceeds 80 characters; contains 111 characters
118 | WARNING | [x] 'TODO: Drupal Rector Notice: Please delete the following comment after
| | you've made any necessary changes.' should match the format '@todo Fix
| | problem X here.'
119 | WARNING | [ ] Line exceeds 80 characters; contains 130 characters
---------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 7 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------
FILE: /Users/norman/Sites/drupal11/web/modules/contrib/basic_cart/src/Utility.php
---------------------------------------------------------------------------------------------
FOUND 38 ERRORS AND 5 WARNINGS AFFECTING 40 LINES
---------------------------------------------------------------------------------------------
6 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is
| | Drupal\Core\Link.
7 | WARNING | [x] Unused use statement
17 | ERROR | [ ] Missing member variable doc comment
99 | ERROR | [x] Short array syntax must be used to define arrays
133 | ERROR | [x] Short array syntax must be used to define arrays
136 | ERROR | [x] Short array syntax must be used to define arrays
137 | ERROR | [x] Short array syntax must be used to define arrays
146 | ERROR | [x] Short array syntax must be used to define arrays
147 | ERROR | [x] Short array syntax must be used to define arrays
155 | ERROR | [x] Short array syntax must be used to define arrays
159 | ERROR | [x] Short array syntax must be used to define arrays
161 | ERROR | [x] Short array syntax must be used to define arrays
170 | ERROR | [x] Short array syntax must be used to define arrays
171 | ERROR | [x] Short array syntax must be used to define arrays
174 | WARNING | [ ] Only string literals should be passed to t() where possible
175 | WARNING | [ ] Only string literals should be passed to t() where possible
179 | ERROR | [x] Short array syntax must be used to define arrays
180 | ERROR | [x] Short array syntax must be used to define arrays
181 | ERROR | [x] Short array syntax must be used to define arrays
185 | ERROR | [x] Each index in a multi-line array must be on a new line
189 | ERROR | [x] Short array syntax must be used to define arrays
192 | WARNING | [ ] Only string literals should be passed to t() where possible
193 | WARNING | [ ] Only string literals should be passed to t() where possible
197 | ERROR | [x] Short array syntax must be used to define arrays
198 | ERROR | [x] Short array syntax must be used to define arrays
199 | ERROR | [x] Short array syntax must be used to define arrays
203 | ERROR | [x] Each index in a multi-line array must be on a new line
203 | ERROR | [x] Short array syntax must be used to define arrays
207 | ERROR | [x] Each index in a multi-line array must be on a new line
207 | ERROR | [x] Short array syntax must be used to define arrays
235 | ERROR | [x] Short array syntax must be used to define arrays
256 | ERROR | [x] Short array syntax must be used to define arrays
351 | ERROR | [ ] Parameter $variable is not described in comment
368 | ERROR | [x] No space found before comment text; expected "// $quantity_enabled =
| | $config->get('quantity_status');" but found "//$quantity_enabled =
| | $config->get('quantity_status');"
370 | ERROR | [x] Use null coalesce operator instead of ternary operator.
370 | ERROR | [x] Short array syntax must be used to define arrays
372 | ERROR | [x] Short array syntax must be used to define arrays
425 | ERROR | [x] Short array syntax must be used to define arrays
443 | ERROR | [x] Short array syntax must be used to define arrays
445 | ERROR | [x] Short array syntax must be used to define arrays
456 | ERROR | [x] Use null coalesce operator instead of ternary operator.
459 | ERROR | [x] Short array syntax must be used to define arrays
461 | ERROR | [x] Use null coalesce operator instead of ternary operator.
---------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 37 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------
FILE: ...tes/drupal11/web/modules/contrib/basic_cart/src/Plugin/Field/FieldWidget/AddToCartWidget.php
---------------------------------------------------------------------------------------------
FOUND 4 ERRORS AFFECTING 4 LINES
---------------------------------------------------------------------------------------------
27 | ERROR | [x] Use null coalesce operator instead of ternary operator.
28 | ERROR | [x] Short array syntax must be used to define arrays
34 | ERROR | [x] Short array syntax must be used to define arrays
36 | ERROR | [x] Whitespace found at end of line
---------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------
FILE: ...x/Sites/drupal11/web/modules/contrib/basic_cart/src/Plugin/Field/FieldType/AddToCartItem.php
---------------------------------------------------------------------------------------------
FOUND 3 ERRORS AND 1 WARNING AFFECTING 4 LINES
---------------------------------------------------------------------------------------------
29 | ERROR | [x] Short array syntax must be used to define arrays
30 | ERROR | [x] Short array syntax must be used to define arrays
31 | ERROR | [x] Short array syntax must be used to define arrays
54 | WARNING | [ ] Only string literals should be passed to t() where possible
---------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 3 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------
FILE: ...o/web/modules/contrib/basic_cart/src/Plugin/Field/FieldFormatter/AddToCartFormatter.php
---------------------------------------------------------------------------------------------
FOUND 9 ERRORS AND 8 WARNINGS AFFECTING 17 LINES
---------------------------------------------------------------------------------------------
6 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is
| | Drupal\Core\Field\FieldItemListInterface.
29 | ERROR | [x] Short array syntax must be used to define arrays
48 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection
| | instead
49 | ERROR | [x] Short array syntax must be used to define arrays
56 | WARNING | [ ] Only string literals should be passed to t() where possible
60 | WARNING | [ ] Only string literals should be passed to t() where possible
65 | ERROR | [ ] The array declaration extends to column 152 (the limit is 120). The
| | array content should be split up over multiple lines
81 | ERROR | [x] Short array syntax must be used to define arrays
82 | ERROR | [x] Short array syntax must be used to define arrays
84 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
87 | ERROR | [x] Short array syntax must be used to define arrays
89 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
92 | ERROR | [x] Short array syntax must be used to define arrays
94 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
97 | ERROR | [x] Short array syntax must be used to define arrays
99 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
110 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
---------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 8 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------
FILE: ...s/contrib/basic_cart/src/Plugin/Field/FieldFormatter/AddToCartWithQuantityFormatter.php
---------------------------------------------------------------------------------------------
FOUND 13 ERRORS AND 15 WARNINGS AFFECTING 27 LINES
---------------------------------------------------------------------------------------------
6 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is
| | Drupal\Core\Field\FieldItemListInterface.
29 | ERROR | [x] Short array syntax must be used to define arrays
46 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection
| | instead
54 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
54 | WARNING | [ ] Only string literals should be passed to t() where possible
57 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection
| | instead
58 | ERROR | [x] Short array syntax must be used to define arrays
67 | WARNING | [ ] Only string literals should be passed to t() where possible
71 | WARNING | [ ] Only string literals should be passed to t() where possible
79 | ERROR | [ ] The array declaration extends to column 150 (the limit is 120). The
| | array content should be split up over multiple lines
97 | ERROR | [x] Short array syntax must be used to define arrays
98 | ERROR | [x] Short array syntax must be used to define arrays
100 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
103 | ERROR | [x] Short array syntax must be used to define arrays
105 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
108 | ERROR | [x] Short array syntax must be used to define arrays
110 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
113 | ERROR | [x] Short array syntax must be used to define arrays
115 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
118 | ERROR | [x] Short array syntax must be used to define arrays
120 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
123 | ERROR | [x] Short array syntax must be used to define arrays
125 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
128 | ERROR | [x] Short array syntax must be used to define arrays
130 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
133 | ERROR | [x] Short array syntax must be used to define arrays
135 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
146 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
---------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 12 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------
FILE: /Users/norman/Sites/drupal11/web/modules/contrib/basic_cart/src/Plugin/Block/CartBlock.php
---------------------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
---------------------------------------------------------------------------------------------
23 | ERROR | [x] Short array syntax must be used to define arrays
28 | ERROR | [x] Short array syntax must be used to define arrays
---------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------
FILE: ...Users/norman/Sites/drupal11/web/modules/contrib/basic_cart/src/Plugin/Block/CountBlock.php
---------------------------------------------------------------------------------------------
FOUND 15 ERRORS AND 6 WARNINGS AFFECTING 20 LINES
---------------------------------------------------------------------------------------------
7 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is
| | Drupal\Core\Form\FormStateInterface.
26 | ERROR | [x] Short array syntax must be used to define arrays
29 | ERROR | [x] Short array syntax must be used to define arrays
67 | ERROR | [x] Short array syntax must be used to define arrays
70 | WARNING | [ ] #options values usually have to run through t() for translation
71 | WARNING | [ ] #options values usually have to run through t() for translation
72 | WARNING | [ ] #options values usually have to run through t() for translation
73 | WARNING | [ ] #options values usually have to run through t() for translation
77 | ERROR | [x] Use null coalesce operator instead of ternary operator.
80 | ERROR | [x] Short array syntax must be used to define arrays
82 | WARNING | [ ] #options values usually have to run through t() for translation
82 | WARNING | [ ] #options values usually have to run through t() for translation
85 | ERROR | [x] Use null coalesce operator instead of ternary operator.
87 | ERROR | [x] Short array syntax must be used to define arrays
91 | ERROR | [x] Use null coalesce operator instead of ternary operator.
94 | ERROR | [x] Short array syntax must be used to define arrays
98 | ERROR | [x] Use null coalesce operator instead of ternary operator.
101 | ERROR | [x] Short array syntax must be used to define arrays
105 | ERROR | [x] Use null coalesce operator instead of ternary operator.
108 | ERROR | [x] Short array syntax must be used to define arrays
112 | ERROR | [x] Use null coalesce operator instead of ternary operator.
---------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 15 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------
FILE: /Users/norman/Sites/drupal11/web/modules/contrib/basic_cart/src/CartTable.php
---------------------------------------------------------------------------------------------
FOUND 15 ERRORS AND 1 WARNING AFFECTING 11 LINES
---------------------------------------------------------------------------------------------
7 | WARNING | [ ] The class short comment should describe what the class does and not
| | simply repeat the class name
11 | ERROR | [ ] Missing member variable doc comment
12 | ERROR | [ ] Missing member variable doc comment
13 | ERROR | [ ] Missing member variable doc comment
15 | ERROR | [ ] Parameter $user is not described in comment
39 | ERROR | [x] Short array syntax must be used to define arrays
39 | ERROR | [ ] The array declaration extends to column 134 (the limit is 120). The
| | array content should be split up over multiple lines
42 | ERROR | [x] Short array syntax must be used to define arrays
42 | ERROR | [ ] The array declaration extends to column 122 (the limit is 120). The
| | array content should be split up over multiple lines
45 | ERROR | [x] Short array syntax must be used to define arrays
45 | ERROR | [x] Short array syntax must be used to define arrays
45 | ERROR | [x] Short array syntax must be used to define arrays
87 | ERROR | [x] Scope keyword "public" must be followed by a single space; found 2
87 | ERROR | [x] Short array syntax must be used to define arrays
124 | ERROR | [x] Short array syntax must be used to define arrays
139 | ERROR | [x] Short array syntax must be used to define arrays
---------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 9 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------
FILE: ...ers/norman/Sites/drupal11/web/modules/contrib/basic_cart/src/Controller/CartController.php
---------------------------------------------------------------------------------------------
FOUND 9 ERRORS AND 22 WARNINGS AFFECTING 23 LINES
---------------------------------------------------------------------------------------------
6 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is
| | Drupal\Core\Controller\ControllerBase.
11 | ERROR | [x] When importing a class with "use", do not include a leading \
27 | WARNING | [ ] Only string literals should be passed to t() where possible
38 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection
| | instead
42 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection
| | instead
45 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
45 | WARNING | [ ] Only string literals should be passed to t() where possible
48 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection
| | instead
48 | ERROR | [x] Short array syntax must be used to define arrays
48 | ERROR | [ ] The array declaration extends to column 175 (the limit is 120). The
| | array content should be split up over multiple lines
48 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
48 | WARNING | [ ] Only string literals should be passed to t() where possible
62 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection
| | instead
76 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection
| | instead
77 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection
| | instead
90 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
| | instead
90 | WARNING | [ ] Only string literals should be passed to t() where possible
108 | WARNING | [ ] NodeType::load calls should be avoided in classes, use dependency
| | injection instead
109 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection
| | instead
109 | ERROR | [x] Short array syntax must be used to define arrays
115 | ERROR | [x] Short array syntax must be used to define arrays
117 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection
| | instead
134 | WARNING | [ ] NodeType::load calls should be avoided in classes, use dependency
| | injection instead
135 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection
| | instead
135 | ERROR | [x] Short array syntax must be used to define arrays
141 | ERROR | [x] Short array syntax must be used to define arrays
143 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection
| | instead
157 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection
| | instead
158 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection
| | instead
176 | WARNING | [ ] Only string literals should be passed to t() where possible
188 | ERROR | [x] Short array syntax must be used to define arrays
---------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 8 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------
FILE: /Users/norman/Sites/drupal11/web/modules/contrib/basic_cart/src/CartInterface.php
------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
------------------------------------------------------------------------------------
39 | ERROR | [x] Short array syntax must be used to define arrays
------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------
FILE: /Users/norman/Sites/drupal11/web/modules/contrib/basic_cart/src/CartSession.php
---------------------------------------------------------------------------------------------
FOUND 11 ERRORS AND 1 WARNING AFFECTING 7 LINES
---------------------------------------------------------------------------------------------
7 | WARNING | [ ] The class short comment should describe what the class does and not
| | simply repeat the class name
10 | ERROR | [ ] Missing member variable doc comment
11 | ERROR | [ ] Missing member variable doc comment
37 | ERROR | [x] Short array syntax must be used to define arrays
37 | ERROR | [ ] The array declaration extends to column 134 (the limit is 120). The
| | array content should be split up over multiple lines
40 | ERROR | [x] Short array syntax must be used to define arrays
40 | ERROR | [ ] The array declaration extends to column 122 (the limit is 120). The
| | array content should be split up over multiple lines
43 | ERROR | [x] Short array syntax must be used to define arrays
43 | ERROR | [x] Short array syntax must be used to define arrays
43 | ERROR | [x] Short array syntax must be used to define arrays
76 | ERROR | [x] Scope keyword "public" must be followed by a single space; found 2
76 | ERROR | [x] Short array syntax must be used to define arrays
---------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 7 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------Issue fork basic_cart-3553487
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
changes
Comments
Comment #4
norman.lol