Bug #51728

MediaWizardProvider::process_youtube does not handle all url variants

Added by Sebastian Fischer almost 2 years ago. Updated almost 2 years ago.

Status:Resolved Start date:2013-09-04
Priority:Should have Due date:
Assigned To:Sebastian Fischer % Done:

100%

Category:Frontend Spent time: -
Target version:-
TYPO3 Version:6.2 Is Regression:
PHP Version: Sprint Focus:
Complexity:

Description

Youtube URLs come in a wide spectrum of variation. The wizard only handles those that have v=ID into them. This could be enhanced.


Related issues

related to Core - Bug #58569: Content element "media" does not render Youtube videos Resolved 2014-05-06

Associated revisions

Revision 83b4390d
Added by Steffen Ritter almost 2 years ago

[FEATURE] Enhance youtube id getting

The preg_match to fetch the video id from an url do cover only a few
variation. The enhancement covers a lot more possibilities

Releases: 6.2
Fixes: #51728
Change-Id: Ie9d0b3b5dbfff4e129d37fcf351d1d8c5fcdfc98
Reviewed-on: https://review.typo3.org/23616
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring

History

#1 Updated by Gerrit Code Review almost 2 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23616

#2 Updated by Gerrit Code Review almost 2 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23616

#3 Updated by Gerrit Code Review almost 2 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23616

#4 Updated by Steffen Ritter almost 2 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

Also available in: Atom PDF