Merge pull request #326 from Financial-Times/null-join-fix

Handle asset content_areas coming back as null
Financial-Times · Oct 7, 2021 · 68039c0 · 68039c0
2 parents fd1ed56 + ab2b526
commit 68039c0
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/package.json b/package.json
@@ -1,7 +1,7 @@
 {
   "name": "ft-next-syndication-api",
   "description": "Next Syndication API",
-  "version": "0.38.4",
+  "version": "0.38.5",
   "private": true,
   "dependencies": {
     "@financial-times/n-es-client": "3.0.0",

diff --git a/server/lib/get-contract-by-id.js b/server/lib/get-contract-by-id.js
@@ -51,7 +51,9 @@ function decorateContract(contract, hasGraphics = false) {
 		acc[asset.content_type] = asset;
 
 		asset.assets.forEach(item => {
-			item.content = item.content_set.join('; ');
+			if (Array.isArray(item.content_set)) {
+				item.content = item.content_set.join('; ');
+			}
 
 			if (Array.isArray(item.addendums) && item.addendums.length) {
 				asset.hasAddendums = true;