Problem/Motivation

Many sites leverage the Entity Usage contributed module to track entities and their relationship to other entities. The Entity Usage module provides a configuration page (/admin/config/entity-usage/settings) where users can opt certain elements/entities into tracking.

While Entity Usage provides support out of the box to track media added by Media Directories via the core media embed button or the link button, there is not coverage for the "Insert file link" CKEditor button provided through the media_directories_file_link submodule. Therefore, media added through this button is not tracked. This can impact user workflows that leverage media tracking through the entity_usage table.

Proposed resolution

Provide an EntityUsageTrack plugin for media embedded with tags so users can opt to include that media in the entity_usage table.

Remaining tasks

- [ ] Create Entity Usage tracking plugin
- [ ] Test plugin

User interface changes

  • Adds a new tracking plugin on the Entity Usage configuration page for users leveraging the Entity Usage contrib module

API changes

NA

Data model changes

NA

CommentFileSizeAuthor
#3 media-directories-file-link-tracking-plugin-3612560-3.patch2.01 KBinregards2pluto
#3 media-directories-file-link-tracking-plugin-3612560.png133.43 KBinregards2pluto

Issue fork media_directories-3612560

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

inregards2pluto created an issue. See original summary.

inregards2pluto opened merge request !31

inregards2pluto’s picture

inregards2pluto
she/her
commented
StatusFileSize
new media-directories-file-link-tracking-plugin-3612560.png133.43 KB
new media-directories-file-link-tracking-plugin-3612560-3.patch2.01 KB

Added tracking plugin! I based it off of the existing plugin for the core CKEditor Media Embed button. Tested and it appears to work correctly, but would appreciate review from someone else. I created an MR and a corresponding patch.

inregards2pluto’s picture

inregards2pluto
she/her
commented
Status: Active » Needs review

rang501 made their first commit to this issue’s fork.

rang501’s picture

rang501 commented

Thanks for the detailed issue description and the fix.

It looks fine to me, so I'll merge it.

rang501’s picture

rang501 commented
Status: Needs review » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.