- Issue created by @renatog
- @renatog opened merge request.
- Status changed to Needs review
over 2 years ago 2:25am 8 March 2023 - Status changed to Fixed
over 2 years ago 8:33am 8 March 2023 - Status changed to Fixed
over 2 years ago 8:33am 8 March 2023
Inside of EbtCoreSettingsForm we're using element_validate which is working fine but the array exceeds 80 chars so drupal coding standards is asking to use multiline instead
FILE: src/Form/EbtCoreSettingsForm.php
-------------------------------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AND 1 WARNING AFFECTING 2 LINES
-------------------------------------------------------------------------------------------------------------------------------------
104 | ERROR | [ ] The array declaration extends to column 125 (the limit is 80). The array content should be split up over
| | multiple lines
104 | ERROR | [ ] The array declaration extends to column 124 (the limit is 80). The array content should be split up over
| | multiple lines
228 | WARNING | [x] A comma should follow the last multiline array item. Found: $ebt_core_xxlarge_width
-------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------------------------------------------------------------Time: 564ms; Memory: 12MB
We can just fix that using multilines approach and it'll works fine
Fixed
1.4
Code