phpcs coding standards fix

Created on 14 January 2023, about 2 years ago
Updated 19 January 2023, about 2 years ago

Basic and auto fixes are in branch. Additional fixes required:

FILE: /Users/tesboss/_PROJECTS/drupal/addtoany/addtoany.admin.inc
-------------------------------------------------------------------------------------------------------
FOUND 7 ERRORS AND 1 WARNING AFFECTING 8 LINES
-------------------------------------------------------------------------------------------------------
  14 | ERROR   | Return type missing for @return tag in function comment
  30 | ERROR   | No key specified for array entry; first entry specifies key
  39 | ERROR   | No key specified for array entry; first entry specifies key
  48 | ERROR   | No key specified for array entry; first entry specifies key
  57 | ERROR   | Return type missing for @return tag in function comment
  83 | WARNING | Avoid backslash escaping in translatable strings when possible, use "" quotes instead
 153 | ERROR   | Return type missing for @return tag in function comment
 217 | ERROR   | Return type missing for @return tag in function comment
-------------------------------------------------------------------------------------------------------


FILE: /Users/tesboss/_PROJECTS/drupal/addtoany/addtoany.info
------------------------------------------------------------------------------------------
FOUND 4 ERRORS AFFECTING 4 LINES
------------------------------------------------------------------------------------------
 5 | ERROR | It's only necessary to declare files[] if they declare a class or interface.
 6 | ERROR | It's only necessary to declare files[] if they declare a class or interface.
 7 | ERROR | It's only necessary to declare files[] if they declare a class or interface.
 9 | ERROR | It's only necessary to declare files[] if they declare a class or interface.
------------------------------------------------------------------------------------------


FILE: /Users/tesboss/_PROJECTS/drupal/addtoany/INSTALL.txt
----------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
----------------------------------------------------------------------
 2 | WARNING | Line exceeds 80 characters; contains 111 characters
----------------------------------------------------------------------


FILE: /Users/tesboss/_PROJECTS/drupal/addtoany/views/addtoany_handler_field_addtoany_link.inc
---------------------------------------------------------------------------------------------
FOUND 3 ERRORS AFFECTING 2 LINES
---------------------------------------------------------------------------------------------
 11 | ERROR | Class name must begin with a capital letter
 11 | ERROR | Class name must use UpperCamel naming without underscores
 13 | ERROR | Doc comment is empty
---------------------------------------------------------------------------------------------


FILE: /Users/tesboss/_PROJECTS/drupal/addtoany/views/addtoany.views.inc
-----------------------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-----------------------------------------------------------------------------------------------------------------------------------------------------------------------
 9 | 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.", "* Implements hook_foo_BAR_ID_bar() for xyz-bar.tpl.php.", or "* Implements hook_foo_BAR_ID_bar() for block templates."
-----------------------------------------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/tesboss/_PROJECTS/drupal/addtoany/addtoany.module
---------------------------------------------------------------------------------------------------------------------
FOUND 47 ERRORS AND 4 WARNINGS AFFECTING 50 LINES
---------------------------------------------------------------------------------------------------------------------
  11 | ERROR   | addtoany_install() is an installation hook and must be declared in an install file
  13 | ERROR   | The $text argument to l() should be enclosed within t() so that it is translatable
  24 | WARNING | Do not pass empty strings to t()
  82 | ERROR   | Do not use t() in hook_menu()
  83 | ERROR   | Do not use t() in hook_menu()
 135 | WARNING | Hook implementations should not duplicate @return documentation
 135 | ERROR   | Return type missing for @return tag in function comment
 152 | ERROR   | String concat is not required here; use a single string instead
 153 | ERROR   | String concat is not required here; use a single string instead
 154 | ERROR   | String concat is not required here; use a single string instead
 155 | ERROR   | String concat is not required here; use a single string instead
 160 | ERROR   | String concat is not required here; use a single string instead
 161 | ERROR   | String concat is not required here; use a single string instead
 162 | ERROR   | String concat is not required here; use a single string instead
 163 | ERROR   | String concat is not required here; use a single string instead
 164 | ERROR   | String concat is not required here; use a single string instead
 165 | ERROR   | String concat is not required here; use a single string instead
 166 | ERROR   | String concat is not required here; use a single string instead
 169 | ERROR   | String concat is not required here; use a single string instead
 170 | ERROR   | String concat is not required here; use a single string instead
 171 | ERROR   | String concat is not required here; use a single string instead
 172 | ERROR   | String concat is not required here; use a single string instead
 173 | ERROR   | String concat is not required here; use a single string instead
 176 | ERROR   | String concat is not required here; use a single string instead
 177 | ERROR   | String concat is not required here; use a single string instead
 178 | ERROR   | String concat is not required here; use a single string instead
 181 | ERROR   | String concat is not required here; use a single string instead
 182 | ERROR   | String concat is not required here; use a single string instead
 185 | ERROR   | String concat is not required here; use a single string instead
 186 | ERROR   | String concat is not required here; use a single string instead
 193 | ERROR   | String concat is not required here; use a single string instead
 194 | ERROR   | String concat is not required here; use a single string instead
 195 | ERROR   | String concat is not required here; use a single string instead
 196 | ERROR   | String concat is not required here; use a single string instead
 197 | ERROR   | String concat is not required here; use a single string instead
 207 | ERROR   | String concat is not required here; use a single string instead
 208 | WARNING | Line exceeds 80 characters; contains 239 characters
 210 | WARNING | Line exceeds 80 characters; contains 179 characters
 225 | ERROR   | Inline doc block comments are not allowed; use "/* Comment */" or "// Comment" instead
 237 | ERROR   | Doc comment short description must be on a single line, further text should be a separate paragraph
 239 | ERROR   | Missing parameter comment
 240 | ERROR   | Missing parameter comment
 241 | ERROR   | Missing parameter comment
 243 | ERROR   | Return type missing for @return tag in function comment
 255 | ERROR   | String concat is not required here; use a single string instead
 262 | ERROR   | String concat is not required here; use a single string instead
 263 | ERROR   | String concat is not required here; use a single string instead
 264 | ERROR   | String concat is not required here; use a single string instead
 281 | ERROR   | Parameter tags must be grouped together in a doc comment
 302 | ERROR   | Parameter tags must be grouped together in a doc comment
 310 | ERROR   | Type hint "array" missing for $url
---------------------------------------------------------------------------------------------------------------------

📌 Task
Status

Closed: won't fix

Version

4.0

Component

Code

Created by

🇦🇺Australia VladimirAus Brisbane, Australia

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • Status changed to Closed: won't fix about 2 years ago
Production build 0.71.5 2024