In FileSizeFormatter.php, line 14, the documentation comment says:

Plugin implementation of the 'Youtube_code' formatter.

That file really contains the implementation of the File_size. That should be reflected in the documentation comment.

CommentFileSizeAuthor
#6 field_formatter_file_size_fix_plugin_comment_3265264_5.patch557 bytesavpaderno
#3 3265264-3.patch572 byteselber

Comments

Pasqualle created an issue. See original summary.

elber’s picture

elber
Primary language Portuguese, Brazil
Location Brazil
commented
Assigned: Unassigned » elber
elber’s picture

elber
Primary language Portuguese, Brazil
Location Brazil
commented
Assigned: elber » Unassigned
Status: Active » Needs review
StatusFileSize
new 3265264-3.patch572 bytes
avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Title: Wrong code comment » The plugin documentation comment describes it as "implementation of the the 'Youtube_code' formatter" when it's a different formatter
Issue summary: View changes
pasqualle’s picture

pasqualle
Location 🇭🇺 Budapest
commented
Status: Needs review » Reviewed & tested by the community

Thank you

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
StatusFileSize
new field_formatter_file_size_fix_plugin_comment_3265264_5.patch557 bytes

Actually, that comment can be removed, since plugins don't have a similar comment, when the plugin ID and its label are already in the plugin annotation.

  • Guietc committed 829803b7 on 2.0.x authored by apaderno 
    Issue #3265264 by elber, apaderno: The plugin documentation comment...
guietc’s picture

guietc commented
Status: Reviewed & tested by the community » Fixed
guietc’s picture

guietc commented
Status: Fixed » Closed (fixed)