feat: Support parsing video element with inner source element

plone · Dec 20, 2021 · b1baa76 · b1baa76
1 parent 533207e
commit b1baa76
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 1 deletion.
diff --git a/src/converters/blocks.js b/src/converters/blocks.js
@@ -60,7 +60,14 @@ const iframeBlock = (elem) => {
 };
 
 const videoBlock = (elem) => {
-  const src = elem.src;
+  let src = elem.src;
+  if (src === '') {
+    // If src is empty search for the first source element
+    const child = elem.firstElementChild;
+    if (child.tagName === 'SOURCE') {
+      src = child.src;
+    }
+  }
   const youtubeId = getYTVideoId(src);
   const block = {
     '@type': 'video',

diff --git a/src/converters/blocks.test.js b/src/converters/blocks.test.js
@@ -28,6 +28,15 @@ describe('videoBlock processing', () => {
     expect(result['@type']).toBe('video');
     expect(result['url']).toBe('https://youtu.be/jn4zGVJq9m0');
   });
+
+  test('External video (Youtube) from source element', () => {
+    const elem = elementFromString(
+      '<video width="170" height="85"><source src="https://youtu.be/47BC9R2vD2w"></video>',
+    );
+    const result = videoBlock(elem);
+    expect(result['@type']).toBe('video');
+    expect(result['url']).toBe('https://youtu.be/47BC9R2vD2w');
+  });
 });
 
 describe('imageBlock processing', () => {