As per coding standard documentation at https://www.drupal.org/coding-standards/docs#file, the @file 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. And config link is missing in the module

CommentFileSizeAuthor
#3 legacy_redirect_file_docblock_search_results.png49.87 KBdhruveshdtripathi
#2 2876473-2.patch868 bytesPavan B S
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Pavan B S created an issue. See original summary.

Pavan B S’s picture

Pavan B S CreditAttribution: Pavan B S at Valuebound commented
Status: Active » Needs review
FileSize
2876473-2.patch868 bytes

Applying the patch, please review

dhruveshdtripathi’s picture

dhruveshdtripathi CreditAttribution: dhruveshdtripathi at DevsAdda commented
Status: Needs review » Reviewed & tested by the community
FileSize
legacy_redirect_file_docblock_search_results.png49.87 KB

Patch applied cleanly. All the @file dockblocks removed from .php file. Checked it manually. As you can see in the attached png file, earlier there were 3 @file docblocks, after applying patch there is 1. That 1 is from .module file.

Good work!

  • kscheirer committed 2d1235a on 8.x-1.x authored by Pavan B S 
    Issue #2876473 by Pavan B S, dhruveshdtripathi: Remove @file tag...
kscheirer’s picture

kscheirer
Primary language English
Location Vallejo
CreditAttribution: kscheirer commented
Status: Reviewed & tested by the community » Fixed

Thanks for the contribution!

Status: Fixed » Closed (fixed)

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