Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
Currently the module requires you to always upload the file via Drupal. This covers most usecases, but there are usecases where the video is already transcoded on a external server.
Proposed resolution
For this usecase add a 'video_url' field, per default hidden, which shows the corresponding URL.
Remaining tasks
Review
Commit
User interface changes
API changes
Data model changes
Comment | File | Size | Author |
---|---|---|---|
#2 | 2728241-2.patch | 4.19 KB | dawehner |
Comments
Comment #2
dawehnerHere is the patch
Comment #3
BoobaaThe patch accepts just about anything as the URL. It'd be better to accept only correct URLs. I think it's OK to introduce dependency on core's link.module for this.
Comment #4
dawehnerGood point! The field type we want here though is not a link, but a URL, so we can leverage
\Drupal\Core\Field\Plugin\Field\FieldType\UriItem
which is in core. No requirement to the link module is needed.Comment #6
yce CreditAttribution: yce at Pronovix for BrightCove commentedThanks for the patch! I had to tune it a little though, so it would be more user friendly.
Commited.
Comment #7
yce CreditAttribution: yce at Pronovix for BrightCove commentedComment #8
dawehner<3 Thank you a lot!