🇮🇳India @Asheesh kumar Pal

New Delhi
Account created on 27 June 2022, almost 2 years ago
#

Recent comments

🇮🇳India Asheesh kumar Pal New Delhi

Everything is working fine cannot hence marking this issue as closed .

🇮🇳India Asheesh kumar Pal New Delhi

this issue is fixed in the recent release.

🇮🇳India Asheesh kumar Pal New Delhi

Thanks for the patch Aakasnha your patch is applied successfully Hence marking it as done.

🇮🇳India Asheesh kumar Pal New Delhi

Thanks for the patch Kritika marking it fixed.

🇮🇳India Asheesh kumar Pal New Delhi

Thank you all of you guys i have applied your patch.

🇮🇳India Asheesh kumar Pal New Delhi

Hi Rassoni theme is not supported for Drupal 10 because it is made only for Drupal 9 this issue is irrelevant so making this issue closed

🇮🇳India Asheesh kumar Pal New Delhi

Hi Vinni_Kadyan

I applied your patch #2 to shorten version 8.x-2.x-dev and confirmed that the PHPCS issues were fixed.

Please look at the screenshot attached for your reference

Thank you.

🇮🇳India Asheesh kumar Pal New Delhi

I have applied the patch for the above issue, please review it

🇮🇳India Asheesh kumar Pal New Delhi

FILE: /var/www/html/drupal_cms/newtheme/web/modules/contrib/schemadata/README.txt
---------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
---------------------------------------------------------------------------------
33 | ERROR | [x] Expected 1 newline at end of file; 0 found
---------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------

FILE: /var/www/html/drupal_cms/newtheme/web/modules/contrib/schemadata/schemadata.info.yml
------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
------------------------------------------------------------------------------------------
7 | ERROR | [x] Expected 1 newline at end of file; 0 found
------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------

FILE: /var/www/html/drupal_cms/newtheme/web/modules/contrib/schemadata/src/Controller/SchemadataController.php
----------------------------------------------------------------------------------------------------------------------------------------------
FOUND 31 ERRORS AND 12 WARNINGS AFFECTING 38 LINES
----------------------------------------------------------------------------------------------------------------------------------------------
9 | ERROR | [x] There must be one blank line after the last USE statement; 0 found;
14 | ERROR | [ ] Missing short description in doc comment
15 | ERROR | [x] Whitespace found at end of line
19 | ERROR | [ ] Public method name "SchemadataController::show_all_tables" is not in lowerCamel format
19 | ERROR | [x] Expected 1 blank line before function; 0 found
21 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
23 | ERROR | [x] Short array syntax must be used to define arrays
24 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
24 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
26 | ERROR | [x] Short array syntax must be used to define arrays
27 | ERROR | [x] Short array syntax must be used to define arrays
28 | ERROR | [x] Short array syntax must be used to define arrays
33 | ERROR | [x] Short array syntax must be used to define arrays
34 | ERROR | [x] Array indentation error, expected 8 spaces but found 14
35 | ERROR | [x] Array closing indentation error, expected 6 spaces but found 18
38 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
38 | WARNING | [ ] Only string literals should be passed to t() where possible
40 | ERROR | [x] Short array syntax must be used to define arrays
49 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
55 | ERROR | [x] Whitespace found at end of line
56 | ERROR | [ ] Missing short description in doc comment
57 | ERROR | [x] Whitespace found at end of line
60 | ERROR | [x] Whitespace found at end of line
63 | ERROR | [x] Data types in @throws tags need to be fully namespaced
65 | ERROR | [ ] Public method name "SchemadataController::explain_table" is not in lowerCamel format
67 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
68 | ERROR | [x] Namespaced classes/interfaces/traits should be referenced with use statements
71 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
77 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
78 | ERROR | [x] Short array syntax must be used to define arrays
81 | ERROR | [x] Short array syntax must be used to define arrays
81 | ERROR | [ ] The array declaration extends to column 107 (the limit is 80). The array content should be split up over multiple lines
81 | ERROR | [x] Short array syntax must be used to define arrays
83 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
89 | ERROR | [x] Short array syntax must be used to define arrays
91 | ERROR | [x] Short array syntax must be used to define arrays
98 | ERROR | [x] Short array syntax must be used to define arrays
102 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
105 | ERROR | [x] Short array syntax must be used to define arrays
106 | WARNING | [x] A comma should follow the last multiline array item. Found: 'pager'
110 | ERROR | [x] Expected newline after closing brace
113 | ERROR | [x] Expected 1 blank line after function; 0 found
114 | ERROR | [x] The closing brace for the class must have an empty line before it
----------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 27 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/drupal_cms/newtheme/web/modules/contrib/schemadata/schemadata.routing.yml
---------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
---------------------------------------------------------------------------------------------
13 | ERROR | [x] Expected 1 newline at end of file; 0 found
---------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------

FILE: /var/www/html/drupal_cms/newtheme/web/modules/contrib/schemadata/schemadata.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: /var/www/html/drupal_cms/newtheme/web/modules/contrib/schemadata/schemadata.module
----------------------------------------------------------------------------------------
FOUND 6 ERRORS AND 3 WARNINGS AFFECTING 6 LINES
----------------------------------------------------------------------------------------
1 | ERROR | [x] Missing file doc comment
8 | WARNING | [ ] Hook implementations should not duplicate @param documentation
8 | ERROR | [ ] Missing parameter name
10 | WARNING | [ ] Hook implementations should not duplicate @param documentation
10 | ERROR | [ ] Missing parameter name
20 | ERROR | [x] Case breaking statements must be followed by a single blank line
21 | WARNING | [ ] Code after the RETURN statement on line 20 cannot be executed
21 | ERROR | [x] Line indented incorrectly; expected 4 spaces, found 6
23 | ERROR | [x] Expected 1 newline at end of file; 0 found
----------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY

🇮🇳India Asheesh kumar Pal New Delhi

I have applied a patch for the above issue, please review it.

Production build 0.69.0 2024