- Issue created by @samit.310@gmail.com
- Status changed to Needs review
over 1 year ago 4:26pm 24 April 2023 - Status changed to Needs work
over 1 year ago 2:40am 25 April 2023 - 🇵🇭Philippines paraderojether
I reviewed patch #2, and confirmed it fixes the errors/warnings that you reported. However, I'm getting these additional errors/warnings when I run the command against Drupal core version 9.5.7, Auto Node Translate 8.x-1.x-dev.
FILE: /Users/studenttrainees/Drupal2/drupalsite/docroot/modules/contrib/auto_node_translate/src/Form/ConfigForm.php
-------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 4 WARNINGS AFFECTING 4 LINES
-------------------------------------------------------------------------------------------------------------------
10 | WARNING | The class short comment should describe what the class does and not simply repeat the class name
88 | WARNING | #options values usually have to run through t() for translation
89 | WARNING | #options values usually have to run through t() for translation
90 | WARNING | #options values usually have to run through t() for translation
-------------------------------------------------------------------------------------------------------------------FILE: /Users/studenttrainees/Drupal2/drupalsite/docroot/modules/contrib/auto_node_translate/src/GoogleTranslationApi.php
------------------------------------------------------------------------------------------------------------------------
FOUND 3 ERRORS AFFECTING 3 LINES
------------------------------------------------------------------------------------------------------------------------
20 | ERROR | [x] Expected 1 blank line before function; 0 found
29 | ERROR | [x] Expected 1 space after closing parenthesis; found 0
30 | ERROR | [x] Whitespace found at end of line
------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 3 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------FILE: /Users/studenttrainees/Drupal2/drupalsite/docroot/modules/contrib/auto_node_translate/src/TranslationApiInterface.php
---------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
---------------------------------------------------------------------------------------------------------------------------
7 | WARNING | The class short comment should describe what the class does and not simply repeat the class name
---------------------------------------------------------------------------------------------------------------------------FILE: /Users/studenttrainees/Drupal2/drupalsite/docroot/modules/contrib/auto_node_translate/src/IbmWatsonTranslationApi.php
--------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------------------------------------------------------------------
49 | ERROR | The array declaration extends to column 177 (the limit is 80). The array content should be split up over multiple lines
--------------------------------------------------------------------------------------------------------------------------------------Time: 414ms; Memory: 10MB
Please check.
Thank You - Status changed to Needs review
over 1 year ago 5:45am 25 April 2023 - Status changed to Needs work
over 1 year ago 12:29am 26 April 2023 - 🇵🇭Philippines paraderojether
Hi chanderbhushan
Patch #4 is failed to apply.
Please check.
Thank You - Status changed to Closed: won't fix
over 1 year ago 12:04pm 18 May 2023 - 🇵🇹Portugal jmauricio
The 8.x-1.x-dev will be descontinued, so this will not be fixed.