Problem/Motivation

FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/metatag_mobile/tests/src/Functional/MetatagMobileTagsTest.php
----------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 25 WARNINGS AFFECTING 25 LINES
----------------------------------------------------------------------------------------------------------------------------
  58 | WARNING | Unused variable $modules.
  65 | WARNING | Unused private method getTestTagName()
  79 | WARNING | Unused private method alternateHandheldTestOutputXpath()
  86 | WARNING | Unused private method alternateHandheldTestValueAttribute()
  93 | WARNING | Unused private method amphtmlTestOutputXpath()
 100 | WARNING | Unused private method amphtmlTestValueAttribute()
 107 | WARNING | Unused private method androidAppLinkAlternativeTestValue()
 114 | WARNING | Unused private method androidAppLinkAlternativeTestOutputXpath()
 123 | WARNING | Unused private method androidAppLinkAlternativeTestValueAttribute()
 130 | WARNING | Unused private method androidManifestTestOutputXpath()
 137 | WARNING | Unused private method androidManifestTestValueAttribute()
 144 | WARNING | Unused private method cleartypeTestNameAttribute()
 151 | WARNING | Unused private method handheldfriendlyTestOutputXpath()
 158 | WARNING | Unused private method iosAppLinkAlternativeTestValue()
 165 | WARNING | Unused private method iosAppLinkAlternativeTestOutputXpath()
 172 | WARNING | Unused private method iosAppLinkAlternativeTestValueAttribute()
 179 | WARNING | Unused private method mobileoptimizedTestOutputXpath()
 186 | WARNING | Unused private method msapplicationSquare150x150logoTestValue()
 193 | WARNING | Unused private method msapplicationSquare310x310logoTestValue()
 200 | WARNING | Unused private method msapplicationSquare70x70logoTestValue()
 207 | WARNING | Unused private method msapplicationTileimageTestValue()
 214 | WARNING | Unused private method msapplicationWide310x150logoTestValue()
 221 | WARNING | Unused private method webManifestTestOutputXpath()
 228 | WARNING | Unused private method webManifestTestValueAttribute()
 235 | WARNING | Unused private method xUaCompatibleTestNameAttribute()
----------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/metatag_favicons/tests/src/Functional/MetatagFaviconsTagsTest.php
--------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 24 WARNINGS AFFECTING 24 LINES
--------------------------------------------------------------------------------------------------------------------------------
  61 | WARNING | Unused variable $modules.
  68 | WARNING | Unused private method shortcutIconTestValueAttribute()
  75 | WARNING | Unused private method icon16x16TestOutputXpath()
  82 | WARNING | Unused private method icon192x192TestOutputXpath()
  89 | WARNING | Unused private method icon32x32TestOutputXpath()
  96 | WARNING | Unused private method icon96x96TestOutputXpath()
 103 | WARNING | Unused private method appleTouchIconPrecomposedTestOutputXpath()
 112 | WARNING | Unused private method appleTouchIconPrecomposed114x114TestOutputXpath()
 121 | WARNING | Unused private method appleTouchIconPrecomposed120x120TestOutputXpath()
 130 | WARNING | Unused private method appleTouchIconPrecomposed144x144TestOutputXpath()
 139 | WARNING | Unused private method appleTouchIconPrecomposed152x152TestOutputXpath()
 148 | WARNING | Unused private method appleTouchIconPrecomposed180x180TestOutputXpath()
 157 | WARNING | Unused private method appleTouchIconPrecomposed72x72TestOutputXpath()
 166 | WARNING | Unused private method appleTouchIconPrecomposed76x76TestOutputXpath()
 173 | WARNING | Unused private method appleTouchIconTestOutputXpath()
 180 | WARNING | Unused private method appleTouchIcon114x114TestOutputXpath()
 187 | WARNING | Unused private method appleTouchIcon120x120TestOutputXpath()
 194 | WARNING | Unused private method appleTouchIcon144x144TestOutputXpath()
 201 | WARNING | Unused private method appleTouchIcon152x152TestOutputXpath()
 208 | WARNING | Unused private method appleTouchIcon180x180TestOutputXpath()
 215 | WARNING | Unused private method appleTouchIcon72x72TestOutputXpath()
 222 | WARNING | Unused private method appleTouchIcon76x76TestOutputXpath()
 229 | WARNING | Unused private method maskIconTestTagName()
 236 | WARNING | Unused private method shortcutIconTestTagName()
--------------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/metatag_open_graph/tests/src/Functional/MetatagOpenGraphTagsTest.php
-----------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
-----------------------------------------------------------------------------------------------------------------------------------
 78 | WARNING | Unused variable $modules.
 85 | WARNING | Unused private method getTestTagName()
-----------------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/metatag_pinterest/tests/src/Functional/MetatagPinterestTagsTest.php
----------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 5 WARNINGS AFFECTING 5 LINES
----------------------------------------------------------------------------------------------------------------------------------
 41 | WARNING | Unused variable $modules.
 48 | WARNING | Unused private method getTestTagName()
 63 | WARNING | Unused private method pinterestTestFieldXpath()
 70 | WARNING | Unused private method pinterestTestKey()
 77 | WARNING | Unused private method pinterestTestValue()
----------------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/metatag_google_plus/tests/src/Functional/MetatagGooglePlusTagsTest.php
-------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 6 WARNINGS AFFECTING 6 LINES
-------------------------------------------------------------------------------------------------------------------------------------
 34 | WARNING | Unused variable $modules.
 41 | WARNING | Unused private method getTestTagName()
 52 | WARNING | Unused private method googlePlusAuthorTestOutputXpath()
 59 | WARNING | Unused private method googlePlusAuthorTestValueAttribute()
 66 | WARNING | Unused private method googlePlusPublisherTestOutputXpath()
 73 | WARNING | Unused private method googlePlusPublisherTestValueAttribute()
-------------------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/metatag_verification/tests/src/Functional/MetatagVerificationTagsTest.php
----------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 3 LINES
----------------------------------------------------------------------------------------------------------------------------------------
 25 | WARNING | [x] A comma should follow the last multiline array item. Found: 'zoom_domain_verification'
 32 | WARNING | [ ] Unused variable $modules.
 39 | WARNING | [ ] Unused private method getTestTagName()
----------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/metatag_app_links/tests/src/Functional/MetatagAppLinksTagsTest.php
---------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
---------------------------------------------------------------------------------------------------------------------------------
 53 | WARNING | Unused variable $modules.
 60 | WARNING | Unused private method getTestTagName()
---------------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/metatag_hreflang/src/Plugin/Derivative/HreflangDeriver.php
-----------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-----------------------------------------------------------------------------------------------------------------------------------------
 43 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
-----------------------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/metatag_hreflang/tests/src/Functional/MetatagHreflangTagsTest.php
--------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 6 WARNINGS AFFECTING 6 LINES
--------------------------------------------------------------------------------------------------------------------------------
 45 | WARNING | Unused variable $modules.
 59 | WARNING | Unused private method getTestTagName()
 66 | WARNING | Unused private method hreflangXdefaultTestOutputXpath()
 73 | WARNING | Unused private method hreflangEnTestOutputXpath()
 80 | WARNING | Unused private method hreflangEsTestOutputXpath()
 87 | WARNING | Unused private method hreflangFrTestOutputXpath()
--------------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/metatag_google_cse/tests/src/Functional/MetatagGoogleCSETagsTest.php
-----------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
-----------------------------------------------------------------------------------------------------------------------------------
 29 | WARNING | Unused variable $modules.
 36 | WARNING | Unused private method googleRatingTestTagName()
-----------------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/metatag_dc/tests/src/Functional/MetatagDublinCoreTagsTest.php
----------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
----------------------------------------------------------------------------------------------------------------------------
 39 | WARNING | Unused variable $modules.
 46 | WARNING | Unused private method getTestTagName()
----------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/src/Form/MetatagDefaultsForm.php
-----------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 8 WARNINGS AFFECTING 8 LINES
-----------------------------------------------------------------------------------------------
  25 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
  46 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
 170 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
 176 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
 181 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
 200 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
 243 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
 245 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
-----------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/src/Form/MetatagSettingsForm.php
-----------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
-----------------------------------------------------------------------------------------------
 5 | WARNING | [x] Unused use statement
 8 | WARNING | [x] Unused use statement
-----------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/src/MetatagDefaultsListBuilder.php
------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
------------------------------------------------------------------------------------------------------------------------------------------
  82 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 115 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
------------------------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/src/Plugin/metatag/Tag/Robots.php
-----------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 10 WARNINGS AFFECTING 10 LINES
-----------------------------------------------------------------------------------------------------------------------------------------
 54 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 55 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 56 | 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
 58 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 59 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 60 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 61 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 62 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 63 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
-----------------------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/src/Plugin/metatag/Tag/Referrer.php
-----------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 9 WARNINGS AFFECTING 9 LINES
-----------------------------------------------------------------------------------------------------------------------------------------
 31 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 32 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 33 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 34 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 35 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 36 | 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
 38 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 40 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
-----------------------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/src/Plugin/metatag/Tag/Title.php
-----------------------------------------------------------------------------------------------------------
FOUND 5 ERRORS AND 2 WARNINGS AFFECTING 6 LINES
-----------------------------------------------------------------------------------------------------------
 22 | ERROR   | [ ] Inline doc block comments are not allowed; use "/* Comment */" or "// Comment" instead
 29 | ERROR   | [ ] Comment indentation error, expected only 1 spaces
 30 | ERROR   | [ ] Comment indentation error, expected only 3 spaces
 34 | ERROR   | [ ] Comment indentation error, expected only 3 spaces
 35 | ERROR   | [ ] Comment indentation error, expected only 5 spaces
 43 | WARNING | [x] There must be no blank line following an inline comment
 43 | WARNING | [ ] There must be no blank line following an inline comment
-----------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/src/Plugin/migrate/process/d7/MetatagEntities.php
-------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AND 6 WARNINGS AFFECTING 4 LINES
-------------------------------------------------------------------------------------------------------------------------------
 256 | ERROR   | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
 352 | WARNING | [x] There must be no blank line following an inline comment
 352 | WARNING | [ ] There must be no blank line following an inline comment
 381 | WARNING | [x] There must be no blank line following an inline comment
 381 | WARNING | [ ] There must be no blank line following an inline comment
 391 | WARNING | [x] There must be no blank line following an inline comment
 391 | WARNING | [ ] There must be no blank line following an inline comment
-------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/src/Plugin/migrate/source/d7/MetatagFieldInstance.php
--------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------------------------------------------
 5 | WARNING | [x] Unused use statement
--------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/src/Command/GenerateTagCommand.php
---------------------------------------------------------------------------------------------------
FOUND 17 ERRORS AND 2 WARNINGS AFFECTING 18 LINES
---------------------------------------------------------------------------------------------------
   8 | WARNING | [x] Unused use statement
   9 | WARNING | [x] Unused use statement
  69 | ERROR   | [ ] Missing short description in doc comment
  70 | ERROR   | [x] Data types in @var tags need to be fully namespaced
  87 | ERROR   | [ ] Missing parameter comment
  87 | ERROR   | [x] Data types in @param tags need to be fully namespaced
 115 | ERROR   | [x] TRUE, FALSE and NULL must be uppercase; expected "NULL" but found "null"
 117 | ERROR   | [x] TRUE, FALSE and NULL must be uppercase; expected "NULL" but found "null"
 119 | ERROR   | [x] TRUE, FALSE and NULL must be uppercase; expected "NULL" but found "null"
 121 | ERROR   | [x] TRUE, FALSE and NULL must be uppercase; expected "NULL" but found "null"
 123 | ERROR   | [x] TRUE, FALSE and NULL must be uppercase; expected "NULL" but found "null"
 125 | ERROR   | [x] TRUE, FALSE and NULL must be uppercase; expected "NULL" but found "null"
 127 | ERROR   | [x] TRUE, FALSE and NULL must be uppercase; expected "NULL" but found "null"
 129 | ERROR   | [x] TRUE, FALSE and NULL must be uppercase; expected "NULL" but found "null"
 131 | ERROR   | [x] TRUE, FALSE and NULL must be uppercase; expected "NULL" but found "null"
 133 | ERROR   | [x] TRUE, FALSE and NULL must be uppercase; expected "NULL" but found "null"
 135 | ERROR   | [x] TRUE, FALSE and NULL must be uppercase; expected "NULL" but found "null"
 137 | ERROR   | [x] TRUE, FALSE and NULL must be uppercase; expected "NULL" but found "null"
 249 | ERROR   | [x] Functions must not contain multiple empty lines in a row; found 2 empty lines
---------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 17 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/src/Command/GenerateGroupCommand.php
---------------------------------------------------------------------------------------------------
FOUND 11 ERRORS AND 3 WARNINGS AFFECTING 13 LINES
---------------------------------------------------------------------------------------------------
   8 | WARNING | [x] Unused use statement
  10 | WARNING | [x] Unused use statement
  12 | WARNING | [x] Unused use statement
  52 | ERROR   | [ ] Missing short description in doc comment
  53 | ERROR   | [x] Data types in @var tags need to be fully namespaced
  66 | ERROR   | [ ] Missing parameter comment
  66 | ERROR   | [x] Data types in @param tags need to be fully namespaced
  92 | ERROR   | [x] TRUE, FALSE and NULL must be uppercase; expected "NULL" but found "null"
  98 | ERROR   | [x] TRUE, FALSE and NULL must be uppercase; expected "NULL" but found "null"
 104 | ERROR   | [x] TRUE, FALSE and NULL must be uppercase; expected "NULL" but found "null"
 110 | ERROR   | [x] TRUE, FALSE and NULL must be uppercase; expected "NULL" but found "null"
 116 | ERROR   | [x] TRUE, FALSE and NULL must be uppercase; expected "NULL" but found "null"
 122 | ERROR   | [x] TRUE, FALSE and NULL must be uppercase; expected "NULL" but found "null"
 128 | ERROR   | [x] TRUE, FALSE and NULL must be uppercase; expected "NULL" but found "null"
---------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 12 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/src/MetatagManager.php
-------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-------------------------------------------------------------------------------------
 365 | ERROR | Function return type is not void, but function is returning void here
-------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/src/MetatagToken.php
-----------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-----------------------------------------------------------------------------------------------------------------------------------------
 75 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
-----------------------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/metatag_dc_advanced/tests/src/Functional/MetatagDublinCoreAdvancedTagsTest.php
---------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
---------------------------------------------------------------------------------------------------------------------------------------------
 74 | WARNING | Unused variable $modules.
---------------------------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/metatag_facebook/tests/src/Functional/MetatagFacebookTagsTest.php
--------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
--------------------------------------------------------------------------------------------------------------------------------
 32 | WARNING | Unused variable $modules.
 39 | WARNING | Unused private method getTestTagName()
--------------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/metatag_views/src/Form/MetatagViewsTranslationForm.php
------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 7 WARNINGS AFFECTING 7 LINES
------------------------------------------------------------------------------------------------------------------------------------------
 139 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
 147 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
 161 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
 162 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
 163 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
 180 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 185 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
------------------------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/metatag_views/src/Form/MetatagViewsEditForm.php
--------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
--------------------------------------------------------------------------------------------------------------
 79 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
 80 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
--------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/metatag_views/src/Controller/MetatagViewsTranslationController.php
---------------------------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
---------------------------------------------------------------------------------------------------------------------------------
  39 | ERROR | [x] Whitespace found at end of line
 166 | ERROR | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
---------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/metatag_views/src/Controller/MetatagViewsController.php
----------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
----------------------------------------------------------------------------------------------------------------------
 104 | WARNING | Only string literals should be passed to t() where possible
----------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/metatag_views/src/Plugin/views/display_extender/MetatagDisplayExtender.php
------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 5 WARNINGS AFFECTING 4 LINES
------------------------------------------------------------------------------------------------------------------------------------------
  55 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 109 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 114 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 115 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 115 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
------------------------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/metatag_twitter_cards/src/Plugin/metatag/Tag/TwitterCardsType.php
-----------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 8 WARNINGS AFFECTING 8 LINES
-----------------------------------------------------------------------------------------------------------------------------------------
 33 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 34 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 35 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 36 | 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
 38 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 39 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 41 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
-----------------------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/metatag_twitter_cards/tests/src/Functional/MetatagTwitterCardsTagsTest.php
-----------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 4 WARNINGS AFFECTING 4 LINES
-----------------------------------------------------------------------------------------------------------------------------------------
  60 | WARNING | Unused variable $modules.
  69 | WARNING | Unused private method getTestTagName()
  95 | WARNING | Unused private method twitterCardsTypeTestFieldXpath()
 102 | WARNING | Unused private method twitterCardsTypeTestValue()
-----------------------------------------------------------------------------------------------------------------------------------------


FILE: ...e/thallesvf/Desktop/lando/d8-contrib/modules/metatag/metatag_open_graph_products/tests/src/Functional/MetatagOpenGraphProductsTagsTest.php
------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
------------------------------------------------------------------------------------------------------------------------------------------------
 36 | WARNING | Unused variable $modules.
 43 | WARNING | Unused private method getTestTagName()
------------------------------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/tests/src/Unit/Migrate/d7/MetatagFieldInstanceTest.php
---------------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
---------------------------------------------------------------------------------------------------------------------
 16 | ERROR | Missing member variable doc comment
 23 | ERROR | Missing member variable doc comment
---------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/tests/src/Unit/Migrate/d7/MetatagFieldTest.php
-------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
-------------------------------------------------------------------------------------------------------------
 16 | ERROR | Missing member variable doc comment
 23 | ERROR | Missing member variable doc comment
-------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/tests/src/Kernel/MetatagManagerTest.php
------------------------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
------------------------------------------------------------------------------------------------------------------------------
 17 | ERROR | If the line declaring an array spans longer than 80 characters, each element should be broken into its own line
 42 | ERROR | If the line declaring an array spans longer than 80 characters, each element should be broken into its own line
------------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/tests/src/Functional/MetatagTagTypesTest.php
------------------------------------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AND 11 WARNINGS AFFECTING 13 LINES
------------------------------------------------------------------------------------------------------------------------------------------
  85 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
  86 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 112 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 113 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 114 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 138 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 139 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 167 | WARNING | Line exceeds 80 characters; contains 93 characters
 172 | ERROR   | Comment indentation error, expected only 1 spaces
 174 | WARNING | Line exceeds 80 characters; contains 89 characters
 178 | ERROR   | Comment indentation error, expected only 1 spaces
 185 | WARNING | Line exceeds 80 characters; contains 96 characters
 186 | WARNING | Line exceeds 80 characters; contains 114 characters
------------------------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/tests/src/Functional/MetatagStringTest.php
------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 10 WARNINGS AFFECTING 7 LINES
------------------------------------------------------------------------------------------------------------------------------------------
  70 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
  71 | 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
 155 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 188 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 188 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 209 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 259 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 259 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 278 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
------------------------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/tests/src/Functional/MetatagTagsTestBase.php
----------------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AND 3 WARNINGS AFFECTING 4 LINES
----------------------------------------------------------------------------------------------------------------------------------------------
 115 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 185 | ERROR   | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
 300 | WARNING | [ ] Unused private method getTestTagValue()
 310 | WARNING | [ ] Unused private method randomImageUrl()
----------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/tests/src/Functional/MetatagForumTest.php
-----------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 4 WARNINGS AFFECTING 2 LINES
-----------------------------------------------------------------------------------------------------------------------------------------
 72 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 72 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 75 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 75 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
-----------------------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/tests/src/Functional/MetatagNodeTranslationTest.php
------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 8 WARNINGS AFFECTING 8 LINES
------------------------------------------------------------------------------------------------------------------------------------------
  81 | 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 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
  86 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 103 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 113 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 115 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 120 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
------------------------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/tests/src/Functional/MetatagAdminTest.php
------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 24 WARNINGS AFFECTING 24 LINES
------------------------------------------------------------------------------------------------------------------------------------------
  78 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
  81 | 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
  88 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 102 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 119 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 138 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 206 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 219 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 232 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 265 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 273 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 305 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 311 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 323 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 327 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 334 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 341 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 373 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 374 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 375 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 376 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 377 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 395 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
------------------------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/tests/src/Functional/MetatagTagsTest.php
-------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 22 WARNINGS AFFECTING 22 LINES
-------------------------------------------------------------------------------------------------------
  50 | WARNING | Unused private method getTestTagName()
  76 | WARNING | Unused private method abstractTestFieldXpath()
  83 | WARNING | Unused private method authorTestOutputXpath()
  90 | WARNING | Unused private method authorTestValueAttribute()
  97 | WARNING | Unused private method canonicalUrlTestOutputXpath()
 104 | WARNING | Unused private method canonicalUrlTestValueAttribute()
 111 | WARNING | Unused private method contentLanguageTestNameAttribute()
 118 | WARNING | Unused private method setCookieTestNameAttribute()
 125 | WARNING | Unused private method descriptionTestFieldXpath()
 132 | WARNING | Unused private method imageSrcTestOutputXpath()
 139 | WARNING | Unused private method imageSrcTestValueAttribute()
 146 | WARNING | Unused private method nextUrlTestOutputXpath()
 153 | WARNING | Unused private method nextUrlTestValueAttribute()
 160 | WARNING | Unused private method prevUrlTestOutputXpath()
 167 | WARNING | Unused private method prevUrlTestValueAttribute()
 174 | WARNING | Unused private method referrerTestFieldXpath()
 181 | WARNING | Unused private method robotsTestFieldXpath()
 188 | WARNING | Unused private method referrerTestValue()
 195 | WARNING | Unused private method robotsTestKey()
 202 | WARNING | Unused private method robotsTestValue()
 209 | WARNING | Unused private method shortlinkTestOutputXpath()
 216 | WARNING | Unused private method shortlinkTestValueAttribute()
-------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/tests/src/Functional/MetatagFrontpageTest.php
------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 15 WARNINGS AFFECTING 15 LINES
------------------------------------------------------------------------------------------------------------------------------------------
  66 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
  68 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 111 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 112 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 116 | WARNING | Code after the RETURN statement on line 113 cannot be executed
 117 | WARNING | Code after the RETURN statement on line 113 cannot be executed
 118 | WARNING | Code after the RETURN statement on line 113 cannot be executed
 119 | WARNING | Code after the RETURN statement on line 113 cannot be executed
 120 | WARNING | Code after the RETURN statement on line 113 cannot be executed
 121 | WARNING | Code after the RETURN statement on line 113 cannot be executed
 122 | WARNING | Code after the RETURN statement on line 113 cannot be executed
 133 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 135 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 168 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 169 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
------------------------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/tests/src/Functional/MetatagFieldTermTest.php
-----------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-----------------------------------------------------------------------------------------------------------------------------------------
 95 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
-----------------------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/tests/src/Functional/MetatagXssTest.php
----------------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AND 9 WARNINGS AFFECTING 7 LINES
----------------------------------------------------------------------------------------------------------------------------------------------
 106 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 107 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 128 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 148 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 148 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 178 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 178 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 198 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 198 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 211 | ERROR   | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
----------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/tests/src/Functional/MetatagFieldTestBase.php
------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 25 WARNINGS AFFECTING 14 LINES
------------------------------------------------------------------------------------------------------------------------------------------
 189 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 190 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 218 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 237 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 237 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 246 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 246 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 246 | WARNING | Only string literals should be passed to t() where possible
 287 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 317 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 317 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 327 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 327 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 327 | WARNING | Only string literals should be passed to t() where possible
 345 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 345 | WARNING | Only string literals should be passed to t() where possible
 386 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 386 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 396 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 396 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 396 | WARNING | Only string literals should be passed to t() where possible
 427 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 427 | WARNING | Only string literals should be passed to t() where possible
 465 | WARNING | Variable $title is undefined.
 468 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
------------------------------------------------------------------------------------------------------------------------------------------


FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/metatag/tests/src/Functional/MetatagConfigTranslationTest.php
------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 9 WARNINGS AFFECTING 9 LINES
------------------------------------------------------------------------------------------------------------------------------------------
  72 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
  73 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
  75 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
  86 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
  92 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 130 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 132 | 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
 157 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
------------------------------------------------------------------------------------------------------------------------------------------

Proposed resolution

Fix Drupal standards in metatag\Command\GenerateGroupCommand

CommentFileSizeAuthor
#15 metatag-n3113761-15.patch39.74 KBdamienmckenna
#15 metatag-n3113761-15.interdiff.txt10.78 KBdamienmckenna
#14 metatag-n3113761-14.patch31.19 KBdamienmckenna
#14 metatag-n3113761-14.interdiff.txt18.15 KBdamienmckenna
#13 3113761-13.patch22.83 KBurvashi_vora
#11 3113761-11.patch50.55 KBdiego_mow
#8 3113761-8.patch29.35 KBrokzabukovec
#2 metatag-Fix_Drupal_standards_in_metatag_Command_GenerateGroupCommand-3113761-2-D8.patch2.79 KBthalles

Comments

thalles created an issue. See original summary.

thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
commented
Status: Needs work » Needs review
StatusFileSize
new metatag-Fix_Drupal_standards_in_metatag_Command_GenerateGroupCommand-3113761-2-D8.patch2.79 KB

Follow a patch

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Title: Fix Drupal standards in metatag\Command\GenerateGroupCommand » Coding standards cleanup

Thank you for working on this.

Let's expand the scope to do another general cleanup of the codebase rather than just a single file.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Needs work
thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
commented

Ok!

thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
commented
Issue summary: View changes
rokzabukovec’s picture

rokzabukovec commented
Assigned: Unassigned » rokzabukovec
rokzabukovec’s picture

rokzabukovec commented
Status: Needs work » Needs review
StatusFileSize
new 3113761-8.patch29.35 KB

Hi. I fixed all the coding standard issues found with a code sniffer tool so that it returns 0 errors. But a lot of issues were introduced through commented out code and @TODO blocks so I am not sure how to handle these. The attached patch fixes all of the issues.

Status: Needs review » Needs work

The last submitted patch, 8: 3113761-8.patch, failed testing. View results

rokzabukovec’s picture

rokzabukovec commented
Assigned: rokzabukovec » Unassigned
diego_mow’s picture

diego_mow commented
Status: Needs work » Needs review
StatusFileSize
new 3113761-11.patch50.55 KB

Updating patch.

Status: Needs review » Needs work

The last submitted patch, 11: 3113761-11.patch, failed testing. View results

urvashi_vora’s picture

urvashi_vora commented
Status: Needs work » Needs review
StatusFileSize
new 3113761-13.patch22.83 KB

Please review the patch. Thanks

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
StatusFileSize
new metatag-n3113761-14.interdiff.txt18.15 KB
new metatag-n3113761-14.patch31.19 KB

Rerolled and made some additional improvements.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Parent issue: » #3257295: Plan for Metatag 8.x-1.20
StatusFileSize
new metatag-n3113761-15.interdiff.txt10.78 KB
new metatag-n3113761-15.patch39.74 KB

Some additional improvements.

Status: Needs review » Needs work

The last submitted patch, 15: metatag-n3113761-15.patch, failed testing. View results

  • DamienMcKenna committed f802057 on 8.x-1.x 
    Issue #3113761 by DamienMcKenna, thalles, rokzabukovec, Diego_Mow,...
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs work » Fixed

Committed, because I know the test failures are from PHP 8.1 compatibility problems we're dealing with elsewhere. Thanks everyone!

Status: Fixed » Closed (fixed)

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