diff options
| author | Julien Castiaux <juc@odoo.com> | 2019-06-11 13:48:23 +0000 |
|---|---|---|
| committer | Julien Castiaux <juc@odoo.com> | 2019-06-17 09:35:37 +0000 |
| commit | cfa1e101e2c98313e6205719bfe34426f39a1b57 (patch) | |
| tree | 526e98577b0725ea426dca099362d0f04239d7ff /addons/mrp/data/mrp_data.xml | |
| parent | 38cc2952356d123a03b1a5e248fc0f3668badc30 (diff) | |
[FIX] website_slides: get video id from youtube url
The regexp used to get the youtube video id from its url is broken in
multiple ways.
If the video ID starts with the letter 'v' in the short (youtu.be) url
then the url is considered invalid. i.e. https://youtu.be/vmhB-pt7EfA
If the query option `v=<video id>` is not the first option of the query
in the regular url (youtube.com/watch) then the url is also considered
invalid. i.e. https://www.youtube.com/watch?feature=youtu.be&v=vmhB-pt7EfA
The problem has been solved by replacing the over-engineered regexp by
the dedicated parsing tools of `urllib.parse`.
opw-2006330
closes odoo/odoo#34036
Signed-off-by: Julien Castiaux <Julien00859@users.noreply.github.com>
Diffstat (limited to 'addons/mrp/data/mrp_data.xml')
0 files changed, 0 insertions, 0 deletions