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.

CommentFileSizeAuthor
#6 2759007-4.patch51.44 KBrajeshwari10
#2 2759007-2.patch7.56 KBrajeshwari10
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rajeshwari10 created an issue. See original summary.

rajeshwari10’s picture

rajeshwari10 CreditAttribution: rajeshwari10 as a volunteer and at Blisstering Solutions commented
Assigned: rajeshwari10 » Unassigned
Status: Active » Needs review
FileSize
2759007-2.patch7.56 KB

Removed @file docblock.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner as a volunteer and at Chapter Three commented
Status: Needs review » Needs work

Please use phpbcs and phpcbf to fix all issues at once.

rajeshwari10’s picture

rajeshwari10 CreditAttribution: rajeshwari10 as a volunteer and at Blisstering Solutions commented
Assigned: Unassigned » rajeshwari10
dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner as a volunteer and at Chapter Three commented

IMHO the right way to deal with those is to use phpcs to fix them all.

rajeshwari10’s picture

rajeshwari10 CreditAttribution: rajeshwari10 as a volunteer and at Blisstering Solutions commented
Status: Needs work » Needs review
FileSize
2759007-4.patch51.44 KB

Solved all the phpcs issues.

Thanks!!

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner as a volunteer and at Chapter Three commented

@rajeshwari10
Did you run phpbcf?

rajeshwari10’s picture

rajeshwari10 CreditAttribution: rajeshwari10 as a volunteer and at Blisstering Solutions commented

@dawehner, yes I have run phpcbf.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner as a volunteer commented
Status: Needs review » Fixed

Thank you @rajeshwari10!

  • dawehner committed b2e2e7b on 8.x-1.x authored by rajeshwari10 
    Issue #2759007 by rajeshwari10: Remove @file tag docblock from all the ....
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Assigned: rajeshwari10 » Unassigned
Status: Needs review » Fixed

Reverting the issue status change.

Status: Fixed » Closed (fixed)

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