Fixed Incorrect default media site definition for YouTube


Well-known member
Tested on XenForo 1.5 Beta 2 and Beta 3. In the default media site definition for YouTube, the Match URLs are set to the following:
It's missing a newline between the last two entries. It should be:


Well-known member
Technically an upgrade but the definitions were reinstalled from my add-on using the query from XenForo_Install_Data_MySql::getData()['xf_bb_code_media_site']

The missing newline can be seen in the query:
INSERT INTO xf_bb_code_media_site
(media_site_id, site_title, site_url, match_urls, embed_html, match_is_regex)
('facebook', 'Facebook', '', '*video.php?v={$id:digits}\*photo.php?v={$id:digits}\*/videos/{$id:digits}', '<div class="fb-post" data-href="{$id}" data-width="500"><div class="fb-xfbml-parse-ignore"><a href="{$id}">{$id}</a></div></div>', 0),
('vimeo', 'Vimeo', '', '{$id:digits}\*/videos/{$id:digits}', '<iframe src="{$id}" width="500" height="300" frameborder="0"></iframe>', 0),
('youtube', 'YouTube', '', '{$id}\*&v={$id}\{$id}\{$id}\*/u/*/{$id}{$id}', '<iframe width="500" height="300" src="{$id}?wmode=opaque" frameborder="0" allowfullscreen></iframe>', 0),
('metacafe', 'Metacafe', '', '#metacafe\\.com/watch/(?P<id>[a-z0-9-]+)(/|$)#siU', '<iframe src="{$id}/" width="500" height="300" allowFullScreen frameborder=0></iframe>', 1),
('dailymotion', 'Dailymotion', '', '{$id:alphanum}', '<iframe frameborder="0" width="500" height="300" src="{$id}?width=500&hideInfos=1"></iframe>', 0),
('liveleak', 'Liveleak', '', '{$id}', '<iframe width="500" height="300" src="{$id}" frameborder="0" allowfullscreen></iframe>', 0)


XenForo developer
Staff member
I have fixed this in the installer. It only applied to installs from 1.5 beta 2 (or something else that triggered it), so I won't be changing this after the fact. If you're affected, you should manually add the line break.