civictheme_media_get_variables not returning anything for civictheme_remote_video media type

Marked https://www.drupal.org/project/civictheme/issues/3475985 🐛 No remote media displayed for field_c_p_attachments Active as a duplicate.

@sime

This is our proposed fix.

Update civictheme_media_get_variables to handle the OEmbed (remote video).

We want to only return the properties that are useful for remote video:

<ul>
  <li>name</li>
  <li>url</li>
  <li>created</li>
  <li>updated</li>
  <li></li>
</ul>

Do not return other properties.

Does this align with your thinking?

OEmbed (remote video) is already handled similarly to other media components, with:

civictheme_preprocess_media__civictheme_remote_video for Remote Video
civictheme_preprocess_media__civictheme_video for Video.

Is there a specific reason to use civictheme_media_get_variables() for this case?
In the current setup, this function is used only for attachments (Documents), and an image which always include an associated file.

Yes the purpose is for this to be a utility to get values for a variety of different field types. As you say it only handles file type of media it would be good if it could be used for video