- Issue created by @urvashi_vora
- Status changed to RTBC
over 1 year ago 12:57pm 28 September 2023 - 🇮🇳India ashutosh ahirwal India
I have tried to applied provided patch and it get applied cleanly.
FILE: /var/www/html/vb/d_cont/scrolling_news_ticker/scrolling_news_ticker.module
--------------------------------------------------------------------------------
FOUND 11 ERRORS AND 1 WARNING AFFECTING 12 LINES
--------------------------------------------------------------------------------
1 | ERROR | [x] End of line character is invalid; expected "\n" but found
| | "\r\n"
5 | WARNING | [ ] Line exceeds 80 characters; contains 81 characters
13 | ERROR | [x] Line indented incorrectly; expected 0 spaces, found 1
14 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 3
16 | ERROR | [x] Line indented incorrectly; expected 4 spaces, found 5
17 | ERROR | [x] Line indented incorrectly; expected 6 spaces, found 7
18 | ERROR | [x] Line indented incorrectly; expected 6 spaces, found 7
19 | ERROR | [x] Line indented incorrectly; expected 6 spaces, found 7
21 | ERROR | [x] Whitespace found at end of line
22 | ERROR | [x] Line indented incorrectly; expected 4 spaces, found 5
23 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 3
24 | ERROR | [x] Line indented incorrectly; expected 0 spaces, found 1
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 11 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------
FILE: ...l/vb/d_cont/scrolling_news_ticker/src/Plugin/Block/ScrollingNewsTicker.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 5 WARNINGS AFFECTING 5 LINES
--------------------------------------------------------------------------------
85 | WARNING | t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and
| | $this->t() instead
87 | 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
89 | 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
--------------------------------------------------------------------------------
Time: 97ms; Memory: 10MB
phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig scrolling_news_ticker/
Patch review
1.0
Code
It involves compliance with, or the content of coding standards. Requires broad community agreement.
I have tried to applied provided patch and it get applied cleanly.