Skip to content
This repository has been archived by the owner on Jan 5, 2018. It is now read-only.

Rendered Video psuedo-field #23

Open
wants to merge 2 commits into
base: 8.x-1.x
Choose a base branch
from

Conversation

NickDickinsonWilde
Copy link

Provides a pseudo-field to render the video. Really it needs a selection of options but works right now. (at least for YouTube vids - haven't test it with the other videos)
see also https://www.drupal.org/node/2570261

* Implements hook_entity_extra_field_info().
*/
function media_entity_embeddable_video_media_view(array &$build, EntityInterface $entity, EntityViewDisplayInterface $display, $view_mode) {
//if ($entity)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This can be remove I guess?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, removed in added commit.

@slashrsm
Copy link
Member

slashrsm commented Feb 9, 2016

Great, this looks good. Please create an issue in d.o. issue queue and cross-link it with this PR. It would also be great to sqush commits and reword commit message in accordance with drupal.org standards: https://www.drupal.org/node/52287

Thank you!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants