Dave Reid authored Apr 04, 2013 and Derek Wright committed Apr 04, 2013

No need to hardcode LANGUAGE_NONE when langcode is passed in for us.

- Do not reload entities if the display is set to 'exclude'
- Added a comment to see [#993728] about fixing this upstream in core

Jeremy Thorson authored Apr 04, 2013 and Derek Wright committed Apr 04, 2013

Unfortunately, once a core file field is configured to support the
display bit, the field completely unsets all files marked as hidden in
its field_prepare_view() callback, so our formatter never gets a
chance to do anything with hidden files. To work-around this, we
reload the entity in our formatter so we get the full list of files.
See [#993728] about fixing this upstream in core.

Issue #1867236 by jthorson: Allow other modules to define their own file_metadata display formatters

Issues #1867102, #1867094 by jthorson: Improve rendering of hidden files table, add more  to alter() hooks.

- Added newlines after break; inside switch() statements.
- Code comments are always supposed to be complete sentences.
- Added types to some @param comments.
- @return can define the type but shouldn't specify a variable name.
- Added missing spaces after 'foreach'.
- Fixed a few typos or duplicate words in comments.
- Removed " when ' would work.