Coming from #2113325: file_entity.install coding standards cleanup.

FILE: ...al/media_dev/profiles/media_dev/modules/file_entity/file_entity.install
--------------------------------------------------------------------------------
FOUND 6 ERROR(S) AFFECTING 4 LINE(S)
--------------------------------------------------------------------------------
 223 | ERROR | Do not use t() or st() in installation phase hooks, use $t =
     |       | get_t() to retrieve the appropriate localization function name
 311 | ERROR | If the line declaring an array spans longer than 80 characters,
     |       | each element should be broken into its own line
 332 | ERROR | There should be no white space after an opening "{"
 332 | ERROR | There should be no white space before a closing "}"
 332 | ERROR | Closing brace must be on a line by itself
 816 | ERROR | If the line declaring an array spans longer than 80 characters,
     |       | each element should be broken into its own line
--------------------------------------------------------------------------------
CommentFileSizeAuthor
#2 2114293_file_entity_cs_fix.patch1.9 KBastonvictor

Comments

bneil’s picture

I'm wondering if there needs to be white space in the empty update function or not?

astonvictor’s picture

Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new1.9 KB
joseph.olstad’s picture

Status: Needs review » Reviewed & tested by the community

Looks good

joseph.olstad’s picture

Status: Reviewed & tested by the community » Fixed

Thanks

Status: Fixed » Closed (fixed)

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