Allow editing of the URI for Youtube etc. The URI/embed code for Youtube and similar is just a string and is sometimes entered with the wrong value. Let the admin change the value.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

chrisgross’s picture

Version: 7.x-2.0-alpha3 » 7.x-2.0-beta1
Component: User interface » Media Internet
Related issues: +#2673692: Change File URI on edit page

It looks like this is not currently supported. I started writing a patch for media_youtube, and then I realized that it was a lot more complicated due to the fact that media_internet is what handles this.

chrisgross’s picture

Title: Edit URI » Edit URI for remote files
chrisgross’s picture

Here's a patch that adds this feature. From my limited testing, I've yet to see any ill effects.

Okay, so this actually adds a new file. Needs work.

chrisgross’s picture

Here's one that actually works.

chrisgross’s picture

and here's one with exception handling.

chrisgross’s picture

chrisgross’s picture

...and here's one that's actually diff'd against an UNPATCHED beta1. Testing is hard.

chrisgross’s picture

Status: Active » Needs review
chrisgross’s picture

Here's an updated patch with better condition checking.

joseph.olstad’s picture

Nice work, leaning more towards putting this new functionality into the 7.x-3.x branch of media

Needs review+

joseph.olstad’s picture

Version: 7.x-2.0-beta1 » 7.x-2.14
Status: Needs review » Reviewed & tested by the community

looks good for both 7.x-2.x and 7.x-3.x

joseph.olstad’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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

joseph.olstad’s picture

Version: 7.x-2.14 » 7.x-2.15
Status: Closed (fixed) » Needs work
Related issues: +#2961667: Editing an image media results in an error when media_internet is enabled

above patch caused regression in media_internet

joseph.olstad’s picture

Version: 7.x-2.15 » 7.x-3.0-rc4
Chris Matthews’s picture

Version: 7.x-3.0-rc4 » 7.x-3.x-dev
Status: Needs work » Closed (outdated)

Recent versions of media have resolved most of peoples concerns and is compatible with entity translation, multilingual and various advanced configurations. Due to the high volume of inactive and most often irrelevant issues we are Closing this as (outdated). If for whatever reason this issue is important to you AND you still have issues after checking the media recipe documentation, then let us know and we will review your concerns.

Otherwise, see the recipe documentation for how to configure media and for troubleshooting tips OR refer to the media_dev distribution if you want to see a working media setup.

As mentioned, feel free to make some noise in this issue if you still feel it is important to you or someone else.

Thanks,

Media team

sirtet’s picture

Status: Closed (outdated) » Active

I just had to replace a youtube video.
Currently you have to add a new file entity and replace it whereever u have used the old one. Thankfully there's the USAGE tab on every file, which shows you the places you have to edit.

For that use-case, it would be much easyer if the remote video's url was editable.
That would be the equivalent workflow as with local image-files, where you can replace the file.

I am surprised there are not more requests for this, but i would vote for re-opening the issue as it is a reasonable feature-request.