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
#4 customfilter_file_docblock_search_results.png113.27 KBdhruveshdtripathi
#3 interdiff.txt277 bytesPavan B S
#3 remove@file_2845172_3.patch3.62 KBPavan B S
#2 remove@file_2845172_1.patch3.27 KBnehakhadke
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

nehakhadke created an issue. See original summary.

nehakhadke’s picture

nehakhadke CreditAttribution: nehakhadke at Invezza Technologies commented
FileSize
remove@file_2845172_1.patch3.27 KB
Pavan B S’s picture

Pavan B S CreditAttribution: Pavan B S at Valuebound commented
FileSize
remove@file_2845172_3.patch3.62 KB
interdiff.txt277 bytes

@nehakhadke, thanks for the patch, i checked the patch manually , i found one more instance of @file

dhruveshdtripathi’s picture

dhruveshdtripathi CreditAttribution: dhruveshdtripathi at DevsAdda commented
Status: Needs review » Reviewed & tested by the community
FileSize
customfilter_file_docblock_search_results.png113.27 KB

Patch applied cleanly. All the @file docblocks removed from .php file. As you can see in the attached screenshot, there is 1 @file docblock left. That is from .module file, so that won't be removed.

Thank you.

  • yukare committed 4b72f0a on 8.x-1.x authored by nehakhadke 
    Issue #2845172 by Pavan B S, nehakhadke, dhruveshdtripathi: Remove @file...
yukare’s picture

yukare CreditAttribution: yukare commented
Status: Reviewed & tested by the community » Fixed

Commited to git, thanks.

Status: Fixed » Closed (fixed)

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