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
#8 AfterPatch_Nivo_Slider.png24.74 KBAanal.addweb
#5 remove@file-2758503-5.patch1.53 KBneha.gangwar
#2 2758503-2.patch1.69 KBvinay15

Comments

Vinay15 created an issue. See original summary.

vinay15’s picture

vinay15
he/him
Primary language Marathi
Location Navi Mumbai
commented
Assigned: vinay15 » Unassigned
Status: Active » Needs review
StatusFileSize
new 2758503-2.patch1.69 KB

Removed @file docblock.

Status: Needs review » Needs work

The last submitted patch, 2: 2758503-2.patch, failed testing.

vinay15’s picture

vinay15
he/him
Primary language Marathi
Location Navi Mumbai
commented
Status: Needs work » Needs review
neha.gangwar’s picture

neha.gangwar commented
StatusFileSize
new remove@file-2758503-5.patch1.53 KB

Retain the defintion only removes @file and contains

Status: Needs review » Needs work

The last submitted patch, 5: remove@file-2758503-5.patch, failed testing.

vinay15’s picture

vinay15
he/him
Primary language Marathi
Location Navi Mumbai
commented
Status: Needs work » Needs review
Aanal.addweb’s picture

Aanal.addweb commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new AfterPatch_Nivo_Slider.png24.74 KB

@neha.gangwar, Thanks for the patch it works well as expected. As i tested it manually by applying in module file. PFA as Result.

andrtroe’s picture

andrtroe commented
Assigned: Unassigned » andrtroe
andrtroe’s picture

andrtroe commented
Assigned: andrtroe » Unassigned
Status: Reviewed & tested by the community » Fixed

It was removed in https://www.drupal.org/node/2886707

Status: Fixed » Closed (fixed)

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