As per coding standard documentation at 'https://www.drupal.org/coding-standards/docs', @file tag docblock should not be there in the files that contain a namespaced class/interface/trait, whose file name is the class name with a .php extension.

CommentFileSizeAuthor
#6 After patch apply 2847945.png137.74 KBriddhi.addweb
#5 removed-file-docblock-2847945-5.patch640 bytesDeepthi kumari
#2 removed-file-docblock-2847945-2.patch703 bytesaks22

Comments

aks22 created an issue. See original summary.

aks22’s picture

aks22 commented
Assigned: aks22 » Unassigned
Status: Active » Needs review
StatusFileSize
new removed-file-docblock-2847945-2.patch703 bytes
sam152’s picture

sam152 commented
Status: Needs review » Needs work 
FILE: ...mbed_metacafe/src/Plugin/video_embed_field/Provider/Metacafe.php
----------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------
 7 | ERROR | Missing short description in doc comment
----------------------------------------------------------------------

Time: 73ms; Memory: 6Mb

I prefer not to do these one at a time.

sam152’s picture

sam152 commented
Title: PHP files modification as per drupal standards » Fix all coding standards issues in module
Deepthi kumari’s picture

Deepthi kumari commented
Status: Needs work » Needs review
StatusFileSize
new removed-file-docblock-2847945-5.patch640 bytes

Updated Patch. Please review.

riddhi.addweb’s picture

riddhi.addweb commented
Status: Needs review » Needs work
StatusFileSize
new After patch apply 2847945.png137.74 KB

I have applied the patch cleanly but still, there are some errors.
Please check the screenshot for the same.

FILE: /var/www/html/web/drupal_8.9/web/modules/contrib/video_embed_metacafe/src/Plugin/video_embed_field/Provider/Metacafe.php
------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AND 1 WARNING AFFECTING 2 LINES
------------------------------------------------------------------------------------------------------------------------------
 14 | WARNING | [ ] The class short comment should describe what the class does and not simply repeat the class name
 46 | ERROR   | [x] Use null coalesce operator instead of ternary operator.
------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------------


FILE: /var/www/html/web/drupal_8.9/web/modules/contrib/video_embed_metacafe/video_embed_metacafe.info.yml
---------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
---------------------------------------------------------------------------------------------------------
 7 | WARNING | All dependencies must be prefixed with the project name, for example "drupal:"
---------------------------------------------------------------------------------------------------------