@quintype/seo 1.40.6-embed-url.1 → 1.40.6-embed-url.4

package/.vscode/settings.json

@@ -3,8 +3,8 @@
     "editor.defaultFormatter": "esbenp.prettier-vscode"
   },
   "editor.tabSize": 2,
-  "editor.formatOnSave": true,
   "editor.detectIndentation": false,
+  "editor.formatOnSave": true,
   "files.insertFinalNewline": true,
   "files.trimTrailingWhitespace": true,
   "editor.codeActionsOnSave": {

package/dist/index.cjs.js

@@ -821,12 +821,23 @@ function generateLiveBlogPostingData(structuredData = {}, story = {}, publisherC
 }
 
 function getEmbedUrl(cards) {
-  // get the first story element which has the embed url
-  const storyElement = cards.flatMap(card => card["story-elements"]).find(elem => elem["embed-url"]);
-  if (storyElement) {
-    return storyElement["embed-url"];
-  }
-  return "";
+  let embedUrl = "";
+
+  // not using the return value of top level find
+  // coz we only need the embed url
+  // find is used for early exit
+  cards.find(card => {
+    const storyElements = card["story-elements"];
+    return storyElements.find((elem, index) => {
+      if (elem["embed-url"]) {
+        embedUrl = elem["embed-url"];
+        return true;
+      }
+      return false;
+    });
+  });
+
+  return embedUrl;
 }
 
 function generateVideoArticleData(structuredData = {}, story = {}, publisherConfig = {}, timezone) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@quintype/seo",
-  "version": "1.40.6-embed-url.1",
+  "version": "1.40.6-embed-url.4",
   "description": "SEO Modules for Quintype",
   "main": "dist/index.cjs.js",
   "repository": "https://github.com/quintype/quintype-node-seo",

package/src/structured-data/structured-data-tags.js

@@ -255,14 +255,23 @@ function generateLiveBlogPostingData(structuredData = {}, story = {}, publisherC
 }
 
 function getEmbedUrl(cards) {
-  // get the first story element which has the embed url
-  const storyElement = cards
-    .flatMap((card) => card["story-elements"])
-    .find((elem) => elem["embed-url"]);
-  if (storyElement) {
-    return  storyElement["embed-url"];
-  }
-  return "";
+  let embedUrl = "";
+
+  // not using the return value of top level find
+  // coz we only need the embed url
+  // find is used for early exit
+  cards.find((card) => {
+    const storyElements = card["story-elements"];
+    return storyElements.find((elem, index) => {
+      if (elem["embed-url"]) {
+        embedUrl = elem["embed-url"];
+        return true;
+      }
+      return false;
+    });
+  });
+
+  return embedUrl;
 }