This issue is a follow up for #2043541: Show file usage in summary where the usage column was introduced. After having another look at admin/content/file I came to the conclusion that the table could be cleaned up a bit by merging the the usage operation link with the usage count in the usage column. This is also where you would expect a link to a page which provides more information about the usage of a file, thus also increasing UX.

Attached patch fixes this issue.

CommentFileSizeAuthor
#5 2067491-usage-sync-with-d8.patch3.66 KBdave reid
#4 file_entity-merge-usage-operation-link-with-usage-count-2067491-4.patch3.25 KBlegolasbo
file_entity-merge-usage-operation-with-usage-count.patch1.61 KBlegolasbo

Comments

aaron’s picture

aaron commented
Status: Needs review » Reviewed & tested by the community

Brilliant!

aaron’s picture

aaron commented

file_entity-merge-usage-operation-with-usage-count.patch queued for re-testing.

devin carlson’s picture

devin carlson commented
Status: Reviewed & tested by the community » Needs work

This looks good but can the wording be made consistent with the usage operations link already in Drupal 8? :)

legolasbo’s picture

legolasbo
he/him
Primary language Dutch
Location Middelburg
commented
Status: Needs work » Needs review
StatusFileSize
new file_entity-merge-usage-operation-link-with-usage-count-2067491-4.patch3.25 KB

I've altered the wording to be consistent with the usage operation link in Drupal 8 and while i was at it I altered the usage page to be more consistent with Drupal 8 as well.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
StatusFileSize
new 2067491-usage-sync-with-d8.patch3.66 KB

Revised patch that reverted the user_load_multiple() removal and uses format_plural to format the usage count.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs review » Fixed

Committed #5 to 7.x-2.x. http://drupalcode.org/project/file_entity.git/commit/6ff1d60

Status: Fixed » Closed (fixed)

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