FILE: ...oject/modules/contrib/user_statistics/src/Form/UserStatsForm.php
----------------------------------------------------------------------
FOUND 5 ERRORS AFFECTING 5 LINES
----------------------------------------------------------------------
8 | ERROR | Doc comment is empty
163 | ERROR | Doc comment is empty
172 | ERROR | Doc comment is empty
181 | ERROR | Doc comment is empty
184 | ERROR | Public method name "UserStatsForm::build_options" is
| | not in lowerCamel format
----------------------------------------------------------------------

Comments

mahtab_alam created an issue. See original summary.

mahtab_alam’s picture

Status: Active » Needs review
StatusFileSize
new2.89 KB
nishantkumar155’s picture

Status: Needs review » Needs work

I am getting below error.

Checking patch composer.json...
error: composer.json: No such file or directory
Checking patch composer.lock...
error: composer.lock: No such file or directory

mahtab_alam’s picture

StatusFileSize
new20.72 KB
nishantkumar155’s picture

please use phpcs --standard=DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md user_statistics

for DrupalPractice.

i am getting below error.
=======================================================
FILE: .../drupal-8.5.3/modules/user_statistics/src/Form/UserStatsForm.php
----------------------------------------------------------------------
FOUND 0 ERRORS AND 26 WARNINGS AFFECTING 26 LINES
----------------------------------------------------------------------
37 | WARNING | t() calls should be avoided in classes, use
| | dependency injection and $this->t() instead
43 | WARNING | t() calls should be avoided in classes, use
| | dependency injection and $this->t() instead
44 | WARNING | t() calls should be avoided in classes, use
| | dependency injection and $this->t() instead
50 | WARNING | t() calls should be avoided in classes, use
| | dependency injection and $this->t() instead
51 | WARNING | t() calls should be avoided in classes, use
| | dependency injection and $this->t() instead
54 | WARNING | \Drupal calls should be avoided in classes, use
| | dependency injection instead
63 | WARNING | t() calls should be avoided in classes, use
| | dependency injection and $this->t() instead
64 | WARNING | t() calls should be avoided in classes, use
| | dependency injection and $this->t() instead
73 | WARNING | t() calls should be avoided in classes, use
| | dependency injection and $this->t() instead
86 | WARNING | t() calls should be avoided in classes, use
| | dependency injection and $this->t() instead
92 | WARNING | t() calls should be avoided in classes, use
| | dependency injection and $this->t() instead
98 | WARNING | t() calls should be avoided in classes, use
| | dependency injection and $this->t() instead
104 | WARNING | t() calls should be avoided in classes, use
| | dependency injection and $this->t() instead
105 | WARNING | t() calls should be avoided in classes, use
| | dependency injection and $this->t() instead
111 | WARNING | t() calls should be avoided in classes, use
| | dependency injection and $this->t() instead
117 | WARNING | t() calls should be avoided in classes, use
| | dependency injection and $this->t() instead
123 | WARNING | t() calls should be avoided in classes, use
| | dependency injection and $this->t() instead
129 | WARNING | t() calls should be avoided in classes, use
| | dependency injection and $this->t() instead
130 | WARNING | t() calls should be avoided in classes, use
| | dependency injection and $this->t() instead
138 | WARNING | t() calls should be avoided in classes, use
| | dependency injection and $this->t() instead
139 | WARNING | t() calls should be avoided in classes, use
| | dependency injection and $this->t() instead
167 | WARNING | \Drupal calls should be avoided in classes, use
| | dependency injection instead
169 | WARNING | t() calls should be avoided in classes, use
| | dependency injection and $this->t() instead
176 | WARNING | \Drupal calls should be avoided in classes, use
| | dependency injection instead
178 | WARNING | t() calls should be avoided in classes, use
| | dependency injection and $this->t() instead
186 | WARNING | \Drupal calls should be avoided in classes, use
| | dependency injection instead
----------------------------------------------------------------------

Time: 45ms; Memory: 6Mb
======================================================================

mahtab_alam’s picture

StatusFileSize
new22.45 KB
mahtab_alam’s picture

Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.