Contrib.social

Fix the issues reported by phpcs

Open on Drupal.org →
Created on 9 May 2023, over 1 year ago
Updated 24 July 2024, 4 months ago

Problem/Motivation

FILE: ...ackendassignment\web\modules\contrib\po_translations_report\config\install\po_translations_report.admin_config.yml
------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
------------------------------------------------------------------------------------------------------------------------
5 | ERROR | [x] Expected 1 newline at end of file; 0 found
------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------

FILE: ...tdocs\backendassignment\web\modules\contrib\po_translations_report\config\schema\po_translations_report.schema.yml
------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
------------------------------------------------------------------------------------------------------------------------
21 | ERROR | [x] Expected 1 newline at end of file; 0 found
------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------

FILE: C:\xampp\htdocs\backendassignment\web\modules\contrib\po_translations_report\po_translations_report.info.yml
------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AND 1 WARNING AFFECTING 1 LINE
------------------------------------------------------------------------------------------------------------------
8 | WARNING | [ ] All dependencies must be prefixed with the project name, for example "drupal:"
8 | ERROR | [x] Expected 1 newline at end of file; 0 found
------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------

FILE: C:\xampp\htdocs\backendassignment\web\modules\contrib\po_translations_report\po_translations_report.links.menu.yml
------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
------------------------------------------------------------------------------------------------------------------------
5 | ERROR | [x] Expected 1 newline at end of file; 0 found
------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------

FILE: C:\xampp\htdocs\backendassignment\web\modules\contrib\po_translations_report\po_translations_report.module
----------------------------------------------------------------------------------------------------------------
FOUND 4 ERRORS AFFECTING 3 LINES
----------------------------------------------------------------------------------------------------------------
1 | ERROR | [x] Missing file doc comment
1 | ERROR | [x] End of line character is invalid; expected "\n" but found "\r\n"
3 | ERROR | [x] There must be one blank line after the last USE statement; 0 found;
11 | ERROR | [x] Expected 1 blank line before function; 0 found
----------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------------------

FILE: C:\xampp\htdocs\backendassignment\web\modules\contrib\po_translations_report\po_translations_report.services.yml
----------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------------------------------------------------------
11 | ERROR | [x] Expected 1 newline at end of file; 0 found
----------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------------------------

FILE: ...ckendassignment\web\modules\contrib\po_translations_report\src\Annotation\PoTranslationsReportDetailsDisplayer.php
------------------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
------------------------------------------------------------------------------------------------------------------------
1 | ERROR | [x] End of line character is invalid; expected "\n" but found "\r\n"
3 | ERROR | [x] Namespaced classes, interfaces and traits should not begin with a file doc comment
------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------

FILE: ...docs\backendassignment\web\modules\contrib\po_translations_report\src\Annotation\PoTranslationsReportDisplayer.php
------------------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
------------------------------------------------------------------------------------------------------------------------
1 | ERROR | [x] End of line character is invalid; expected "\n" but found "\r\n"
3 | ERROR | [x] Namespaced classes, interfaces and traits should not begin with a file doc comment
------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------

FILE: ...ocs\backendassignment\web\modules\contrib\po_translations_report\src\Controller\PoTranslationsReportController.php
------------------------------------------------------------------------------------------------------------------------
FOUND 20 ERRORS AND 9 WARNINGS AFFECTING 25 LINES
------------------------------------------------------------------------------------------------------------------------
1 | ERROR | [x] End of line character is invalid; expected "\n" but found "\r\n"
3 | ERROR | [x] Namespaced classes, interfaces and traits should not begin with a file doc comment
22 | WARNING | [ ] The class short comment should describe what the class does and not simply repeat the class name
30 | ERROR | [x] Short array syntax must be used to define arrays
72 | ERROR | [ ] Parameter $translationManager is not described in comment
75 | ERROR | [x] Data types in @param tags need to be fully namespaced
77 | ERROR | [x] Data types in @param tags need to be fully namespaced
79 | ERROR | [x] Data types in @param tags need to be fully namespaced
81 | ERROR | [x] Data types in @param tags need to be fully namespaced
116 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait
| | and $this->t() instead
117 | ERROR | [x] Short array syntax must be used to define arrays
119 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait
| | and $this->t() instead
119 | ERROR | [x] Short array syntax must be used to define arrays
135 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait
| | and $this->t() instead
135 | ERROR | [x] Short array syntax must be used to define arrays
154 | ERROR | [x] Short array syntax must be used to define arrays
230 | ERROR | [x] Short array syntax must be used to define arrays
282 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait
| | and $this->t() instead
282 | ERROR | [x] Short array syntax must be used to define arrays
284 | ERROR | [x] Short array syntax must be used to define arrays
290 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait
| | and $this->t() instead
290 | ERROR | [x] Short array syntax must be used to define arrays
292 | ERROR | [x] Short array syntax must be used to define arrays
299 | ERROR | [x] Short array syntax must be used to define arrays
315 | ERROR | [x] Short array syntax must be used to define arrays
329 | ERROR | [x] Short array syntax must be used to define arrays
330 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait
| | and $this->t() instead
331 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait
| | and $this->t() instead
332 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait
| | and $this->t() instead
------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 19 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------

FILE: C:\xampp\htdocs\backendassignment\web\modules\contrib\po_translations_report\src\DetailsDisplayerPluginBase.php
---------------------------------------------------------------------------------------------------------------------
FOUND 6 ERRORS AFFECTING 6 LINES
---------------------------------------------------------------------------------------------------------------------
1 | ERROR | [x] End of line character is invalid; expected "\n" but found "\r\n"
3 | ERROR | [x] Namespaced classes, interfaces and traits should not begin with a file doc comment
59 | ERROR | [x] Short array syntax must be used to define arrays
61 | ERROR | [x] Short array syntax must be used to define arrays
85 | ERROR | [x] Short array syntax must be used to define arrays
109 | ERROR | [x] Short array syntax must be used to define arrays
---------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 6 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------------------------------

FILE: ...\xampp\htdocs\backendassignment\web\modules\contrib\po_translations_report\src\DetailsDisplayerPluginInterface.php
------------------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
------------------------------------------------------------------------------------------------------------------------
1 | ERROR | [x] End of line character is invalid; expected "\n" but found "\r\n"
3 | ERROR | [x] Namespaced classes, interfaces and traits should not begin with a file doc comment
------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------

FILE: C:\xampp\htdocs\backendassignment\web\modules\contrib\po_translations_report\src\DetailsDisplayerPluginManager.php
------------------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
------------------------------------------------------------------------------------------------------------------------
1 | ERROR | [x] End of line character is invalid; expected "\n" but found "\r\n"
3 | ERROR | [x] Namespaced classes, interfaces and traits should not begin with a file doc comment
------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------

FILE: C:\xampp\htdocs\backendassignment\web\modules\contrib\po_translations_report\src\DisplayerPluginBase.php
--------------------------------------------------------------------------------------------------------------
FOUND 6 ERRORS AFFECTING 6 LINES
--------------------------------------------------------------------------------------------------------------
1 | ERROR | [x] End of line character is invalid; expected "\n" but found "\r\n"
3 | ERROR | [x] Namespaced classes, interfaces and traits should not begin with a file doc comment
59 | ERROR | [x] Short array syntax must be used to define arrays
61 | ERROR | [x] Short array syntax must be used to define arrays
85 | ERROR | [x] Short array syntax must be used to define arrays
109 | ERROR | [x] Short array syntax must be used to define arrays
--------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 6 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------------------------------------

FILE: C:\xampp\htdocs\backendassignment\web\modules\contrib\po_translations_report\src\DisplayerPluginInterface.php
-------------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
-------------------------------------------------------------------------------------------------------------------
1 | ERROR | [x] End of line character is invalid; expected "\n" but found "\r\n"
3 | ERROR | [x] Namespaced classes, interfaces and traits should not begin with a file doc comment
-------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------------------------------------------

FILE: C:\xampp\htdocs\backendassignment\web\modules\contrib\po_translations_report\src\DisplayerPluginManager.php
-----------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
-----------------------------------------------------------------------------------------------------------------
1 | ERROR | [x] End of line character is invalid; expected "\n" but found "\r\n"
3 | ERROR | [x] Namespaced classes, interfaces and traits should not begin with a file doc comment
-----------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------------

FILE: ...:\xampp\htdocs\backendassignment\web\modules\contrib\po_translations_report\src\Form\PoTranslationsReportAdmin.php
------------------------------------------------------------------------------------------------------------------------
FOUND 35 ERRORS AFFECTING 34 LINES
------------------------------------------------------------------------------------------------------------------------
1 | ERROR | [x] End of line character is invalid; expected "\n" but found "\r\n"
3 | ERROR | [x] Namespaced classes, interfaces and traits should not begin with a file doc comment
30 | ERROR | [ ] Missing @var tag in member variable comment
35 | ERROR | [ ] Missing @var tag in member variable comment
62 | ERROR | [x] Short array syntax must be used to define arrays
77 | ERROR | [x] Short array syntax must be used to define arrays
84 | ERROR | [x] Short array syntax must be used to define arrays
92 | ERROR | [x] Short array syntax must be used to define arrays
93 | ERROR | [x] Short array syntax must be used to define arrays
101 | ERROR | [x] Short array syntax must be used to define arrays
109 | ERROR | [x] Short array syntax must be used to define arrays
110 | ERROR | [ ] The array declaration extends to column 82 (the limit is 80). The array content should be split up
| | over multiple lines
110 | ERROR | [x] Short array syntax must be used to define arrays
119 | ERROR | [x] Short array syntax must be used to define arrays
134 | ERROR | [x] Short array syntax must be used to define arrays
135 | ERROR | [x] Short array syntax must be used to define arrays
136 | ERROR | [x] Short array syntax must be used to define arrays
153 | ERROR | [x] Short array syntax must be used to define arrays
154 | ERROR | [x] Short array syntax must be used to define arrays
155 | ERROR | [x] Short array syntax must be used to define arrays
172 | ERROR | [x] Data types in @param tags need to be fully namespaced
176 | ERROR | [x] Short array syntax must be used to define arrays
178 | ERROR | [x] Short array syntax must be used to define arrays
193 | ERROR | [x] Short array syntax must be used to define arrays
207 | ERROR | [x] Short array syntax must be used to define arrays
221 | ERROR | [x] Data types in @param tags need to be fully namespaced
225 | ERROR | [x] Short array syntax must be used to define arrays
227 | ERROR | [x] Short array syntax must be used to define arrays
242 | ERROR | [x] Short array syntax must be used to define arrays
256 | ERROR | [x] Short array syntax must be used to define arrays
272 | ERROR | [x] Short array syntax must be used to define arrays
275 | ERROR | [x] Short array syntax must be used to define arrays
300 | ERROR | [x] Object operator not indented correctly; expected 6 spaces but found 8
343 | ERROR | [x] Data types in @param tags need to be fully namespaced
359 | ERROR | [x] Data types in @param tags need to be fully namespaced
------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 32 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------

FILE: ...ssignment\web\modules\contrib\po_translations_report\src\Plugin\PoTranslationsReportDetailsDisplayer\HtmlTable.php
------------------------------------------------------------------------------------------------------------------------
FOUND 8 ERRORS AND 6 WARNINGS AFFECTING 12 LINES
------------------------------------------------------------------------------------------------------------------------
1 | ERROR | [x] End of line character is invalid; expected "\n" but found "\r\n"
3 | ERROR | [x] Namespaced classes, interfaces and traits should not begin with a file doc comment
22 | WARNING | [ ] The class short comment should describe what the class does and not simply repeat the class name
36 | ERROR | [x] Short array syntax must be used to define arrays
37 | ERROR | [x] Short array syntax must be used to define arrays
37 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait
| | and $this->t() instead
38 | ERROR | [x] Short array syntax must be used to define arrays
38 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait
| | and $this->t() instead
51 | ERROR | [x] Short array syntax must be used to define arrays
64 | ERROR | [x] Short array syntax must be used to define arrays
80 | ERROR | [x] Short array syntax must be used to define arrays
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
83 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait
| | and $this->t() instead
------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 8 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------

FILE: ...nt\web\modules\contrib\po_translations_report\src\Plugin\PoTranslationsReportDisplayer\SortableLinkedDataTable.php
------------------------------------------------------------------------------------------------------------------------
FOUND 15 ERRORS AND 6 WARNINGS AFFECTING 21 LINES
------------------------------------------------------------------------------------------------------------------------
1 | ERROR | [x] End of line character is invalid; expected "\n" but found "\r\n"
3 | ERROR | [x] Namespaced classes, interfaces and traits should not begin with a file doc comment
23 | WARNING | [ ] The class short comment should describe what the class does and not simply repeat the class name
42 | ERROR | [x] Short array syntax must be used to define arrays
43 | ERROR | [x] Short array syntax must be used to define arrays
44 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait
| | and $this->t() instead
48 | ERROR | [x] Short array syntax must be used to define arrays
52 | ERROR | [x] Short array syntax must be used to define arrays
56 | ERROR | [x] Short array syntax must be used to define arrays
60 | ERROR | [x] Short array syntax must be used to define arrays
61 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait
| | and $this->t() instead
78 | ERROR | [x] Short array syntax must be used to define arrays
90 | ERROR | [x] Short array syntax must be used to define arrays
106 | ERROR | [x] Short array syntax must be used to define arrays
107 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait
| | and $this->t() instead
108 | 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
160 | ERROR | [x] Short array syntax must be used to define arrays
167 | ERROR | [x] Short array syntax must be used to define arrays
174 | ERROR | [x] Short array syntax must be used to define arrays
196 | ERROR | [x] Short array syntax must be used to define arrays
------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 15 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------

FILE: C:\xampp\htdocs\backendassignment\web\modules\contrib\po_translations_report\src\PoDetailsReporter.php
---------------------------------------------------------------------------------------------------------------------
FOUND 9 ERRORS AND 1 WARNING AFFECTING 10 LINES
---------------------------------------------------------------------------------------------------------------------
1 | ERROR | [x] End of line character is invalid; expected "\n" but found "\r\n"
3 | ERROR | [x] Namespaced classes, interfaces and traits should not begin with a file doc comment
15 | WARNING | [ ] The class short comment should describe what the class does and not simply repeat the class name
26 | ERROR | [x] Return type must not contain variable name "$results"
33 | ERROR | [x] Short array syntax must be used to define arrays
65 | ERROR | [x] Return type must not contain variable name "$results"
69 | ERROR | [x] Short array syntax must be used to define arrays
75 | ERROR | [x] Short array syntax must be used to define arrays
85 | ERROR | [x] Short array syntax must be used to define arrays
94 | ERROR | [x] Short array syntax must be used to define arrays
---------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 9 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------------------------------

FILE: C:\xampp\htdocs\backendassignment\web\modules\contrib\po_translations_report\src\PoReporter.php
---------------------------------------------------------------------------------------------------------------------
FOUND 4 ERRORS AND 1 WARNING AFFECTING 5 LINES
---------------------------------------------------------------------------------------------------------------------
1 | ERROR | [x] End of line character is invalid; expected "\n" but found "\r\n"
3 | ERROR | [x] Namespaced classes, interfaces and traits should not begin with a file doc comment
14 | WARNING | [ ] The class short comment should describe what the class does and not simply repeat the class name
62 | ERROR | [ ] Throws comment indentation must be 3 spaces, found 2 spaces
89 | ERROR | [x] Short array syntax must be used to define arrays
---------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 3 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------------------------------

FILE: ...:\xampp\htdocs\backendassignment\web\modules\contrib\po_translations_report\src\Tests\PoTranslationsReportTest.php
------------------------------------------------------------------------------------------------------------------------
FOUND 38 ERRORS AND 5 WARNINGS AFFECTING 40 LINES
------------------------------------------------------------------------------------------------------------------------
1 | ERROR | [x] End of line character is invalid; expected "\n" but found "\r\n"
3 | ERROR | [x] Namespaced classes, interfaces and traits should not begin with a file doc comment
31 | ERROR | [x] Short array syntax must be used to define arrays
38 | ERROR | [x] Short array syntax must be used to define arrays
41 | ERROR | [x] Object operator not indented correctly; expected 6 spaces but found 8
63 | ERROR | [x] Short array syntax must be used to define arrays
69 | ERROR | [x] Short array syntax must be used to define arrays
71 | ERROR | [x] Array closing indentation error, expected 4 spaces but found 8
71 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait
| | and $this->t() instead
74 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait
| | and $this->t() instead
83 | ERROR | [x] Short array syntax must be used to define arrays
86 | ERROR | [x] Object operator not indented correctly; expected 6 spaces but found 8
103 | ERROR | [x] Short array syntax must be used to define arrays
106 | ERROR | [x] Object operator not indented correctly; expected 6 spaces but found 8
121 | ERROR | [x] Short array syntax must be used to define arrays
124 | ERROR | [x] Object operator not indented correctly; expected 6 spaces but found 8
142 | ERROR | [x] Short array syntax must be used to define arrays
147 | 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 | ERROR | [x] Short array syntax must be used to define arrays
157 | ERROR | [x] Short array syntax must be used to define arrays
162 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait
| | and $this->t() instead
162 | ERROR | [x] Short array syntax must be used to define arrays
207 | ERROR | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing
| | parentheses
220 | ERROR | [x] Short array syntax must be used to define arrays
221 | ERROR | [x] Short array syntax must be used to define arrays
222 | ERROR | [x] Short array syntax must be used to define arrays
225 | ERROR | [x] Short array syntax must be used to define arrays
229 | ERROR | [x] Short array syntax must be used to define arrays
232 | ERROR | [x] Short array syntax must be used to define arrays
236 | ERROR | [x] Short array syntax must be used to define arrays
240 | ERROR | [x] Short array syntax must be used to define arrays
241 | ERROR | [x] Short array syntax must be used to define arrays
244 | ERROR | [x] Short array syntax must be used to define arrays
248 | ERROR | [x] Short array syntax must be used to define arrays
252 | ERROR | [x] Short array syntax must be used to define arrays
255 | ERROR | [x] Short array syntax must be used to define arrays
259 | ERROR | [x] Short array syntax must be used to define arrays
260 | ERROR | [x] Short array syntax must be used to define arrays
263 | ERROR | [x] Short array syntax must be used to define arrays
266 | ERROR | [x] Short array syntax must be used to define arrays
269 | ERROR | [x] Short array syntax must be used to define arrays
272 | ERROR | [x] Short array syntax must be used to define arrays
------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 38 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------

Time: 8.21 secs; Memory: 6MB

Steps to reproduce

Run this command:
phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig po_translations_report

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

📌 Task
Status

RTBC

Version

1.0

Component

Code

Created by

🇮🇳India sidharth_soman Bangalore

Live updates comments and jobs are added and updated live.
  • Coding standards

    It involves compliance with, or the content of coding standards. Requires broad community agreement.

Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @sidharth_soman
  • Status changed to Needs work over 1 year ago
  • Comment over 1 year ago →
    🇮🇳India sidharth_soman Bangalore

    I will work on the issues and provide an MR.

  • Merge request !3Fixed the phpcs issues. → (Open) created by sidharth_soman
  • Issue was unassigned.
  • Status changed to Needs review over 1 year ago
  • Comment over 1 year ago →
    🇮🇳India sidharth_soman Bangalore

    I've fixed all the issues except for those relating to documentation... which should be ideally resolved by the maintainers.

    Please review the above MR.

  • Status changed to Needs work 5 months ago
  • Comment 5 months ago →
    🇵🇭Philippines cleavinjosh

    Hi @sidharth_soman,

    I applied MR!3 and it was applied smoothly. However, I still encountered some issues after I ran phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml.

    ➜  po_translations_report git:(8.x-1.x) curl https://git.drupalcode.org/project/po_translations_report/-/merge_requests/3.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 48019    0 48019    0     0    99k      0 --:--:-- --:--:-- --:--:--   99k
patching file config/install/po_translations_report.admin_config.yml
patching file config/schema/po_translations_report.schema.yml
patching file po_translations_report.info.yml
patching file po_translations_report.links.menu.yml
patching file po_translations_report.module
patching file po_translations_report.services.yml
patching file src/Annotation/PoTranslationsReportDetailsDisplayer.php
patching file src/Annotation/PoTranslationsReportDisplayer.php
patching file src/Controller/PoTranslationsReportController.php
patching file src/DetailsDisplayerPluginBase.php
patching file src/DetailsDisplayerPluginInterface.php
patching file src/DetailsDisplayerPluginManager.php
patching file src/DisplayerPluginBase.php
patching file src/DisplayerPluginInterface.php
patching file src/DisplayerPluginManager.php
patching file src/Form/PoTranslationsReportAdmin.php
patching file src/Plugin/PoTranslationsReportDetailsDisplayer/HtmlTable.php
patching file src/Plugin/PoTranslationsReportDisplayer/SortableLinkedDataTable.php
patching file src/PoDetailsReporter.php
patching file src/PoReporter.php
patching file src/Tests/PoTranslationsReportTest.php
➜  po_translations_report git:(8.x-1.x) ✗ ..
➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml po_translations_report

FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/po_translations_report/po_translations_report.info.yml
---------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
---------------------------------------------------------------------------------------------------------------------------
 8 | WARNING | All dependencies must be prefixed with the project name, for example "drupal:"
---------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/po_translations_report/src/PoReporter.php
----------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
----------------------------------------------------------------------------------------------------------------
 9 | WARNING | The class short comment should describe what the class does and not simply repeat the class name
----------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/po_translations_report/src/Form/PoTranslationsReportAdmin.php
----------------------------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
----------------------------------------------------------------------------------------------------------------------------------
 25 | ERROR | Missing @var tag in member variable comment
 30 | ERROR | Missing @var tag in member variable comment
----------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/po_translations_report/src/DisplayerPluginInterface.php
---------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
---------------------------------------------------------------------------------------------------------------------------------------
 6 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Component\Plugin\ConfigurableInterface.
---------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/po_translations_report/src/Tests/PoTranslationsReportTest.php
-------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------------------------------------------
 6 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Extension\ExtensionPathResolver.
-------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/po_translations_report/src/Plugin/PoTranslationsReportDisplayer/SortableLinkedDataTable.php
----------------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
----------------------------------------------------------------------------------------------------------------------------------------------------------------
 19 | WARNING | The class short comment should describe what the class does and not simply repeat the class name
----------------------------------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/po_translations_report/src/Plugin/PoTranslationsReportDetailsDisplayer/HtmlTable.php
---------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
---------------------------------------------------------------------------------------------------------------------------------------------------------
 18 | WARNING | The class short comment should describe what the class does and not simply repeat the class name
---------------------------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/po_translations_report/src/Controller/PoTranslationsReportController.php
---------------------------------------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AND 1 WARNING AFFECTING 3 LINES
---------------------------------------------------------------------------------------------------------------------------------------------
  7 | ERROR   | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Component\Utility\Xss.
 18 | WARNING | [ ] The class short comment should describe what the class does and not simply repeat the class name
 70 | ERROR   | [ ] Parameter $translationManager is not described in comment
---------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/po_translations_report/src/DetailsDisplayerPluginInterface.php
---------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
---------------------------------------------------------------------------------------------------------------------------------------
 6 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Component\Plugin\ConfigurableInterface.
---------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/po_translations_report/src/PoDetailsReporter.php
---------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
---------------------------------------------------------------------------------------------------------------------
 10 | WARNING | The class short comment should describe what the class does and not simply repeat the class name
---------------------------------------------------------------------------------------------------------------------

Time: 473ms; Memory: 12MB

➜  contrib git:(main) ✗

    Please check and advise.
    Thank you.

  • Assigned to hetal.solanki
  • Comment 5 months ago →
    🇮🇳India hetal.solanki
  • Comment 5 months ago →
    🇮🇳India hetal.solanki

    Hetal.Solanki → changed the visibility of the branch 3358998-dev to hidden.

  • Comment 5 months ago →
    🇮🇳India hetal.solanki

    Hetal.Solanki → changed the visibility of the branch 3358998-dev to active.

  • Issue was unassigned.
  • Comment 5 months ago →
    🇮🇳India hetal.solanki
  • Status changed to Needs review 4 months ago
  • Comment 4 months ago →
    🇵🇭Philippines cleavinjosh

    Hi,

    I fixed the remaining phpcs issues.

    Please review. Thank you.

  • Status changed to RTBC 4 months ago
  • Comment 4 months ago →
    🇮🇳India vinmayiswamy

    Hi,
    I've tested MR !3 on Drupal 10.3.x.

    The MR is applied Cleanly...

    Test Result:
The phpcs issues were fixed after the updated changes.

    RTBC+1

    Thanks!

contrib.social Blog FAQ Discussions
Production build 0.71.5 2024