Handle missing data better (#3845)

Author: tunetheweb

src/tools/generate/generate_rss.js

@@ -33,7 +33,7 @@ const generate_rss = async (configs, rss_chapters, rss_languages) => {
     const updatedate = get_update_date(`${language}/${year}/chapters/${chapter}.html`);
     const url = convert_file_name(`${language}/${year}/${chapter}`);
     title = `${title} (${year})`;
-    authors = authors.map(author => configs[year].contributors[author].name.replace(/\\/g, '\\\\').replace(/"/g, '\\"'));
+    authors = authors.map(author => configs[year].contributors[author]?.name?.replace(/\\/g, '\\\\')?.replace(/"/g, '\\"'));
     authors = '"' + authors.join('","') + '"';
 
     // Capture the latest updated date as the overall rss updated date

src/tools/generate/shared.js

@@ -24,13 +24,13 @@ simpleConverter.setOption('ghMentions', false);
 
 const convertSimpleMarkdown = (text) => {
   // Showdown replaces & with & so convert those back to avoid escape issues
-  let convertedText = text.replace(/&/g,'&');
+  let convertedText = text?.replace(/&/g,'&');
   convertedText = simpleConverter.makeHtml(convertedText);
   // As we've converted again, need to convert back & again
-  convertedText = convertedText.replace(/&/g,'&');
+  convertedText = convertedText?.replace(/&/g,'&');
   // Strip paragraph tags from beginning and end
-  convertedText = convertedText.replace(/^<p>/,'');
-  convertedText = convertedText.replace(/<\/p>$/,'');
+  convertedText = convertedText?.replace(/^<p>/,'');
+  convertedText = convertedText?.replace(/<\/p>$/,'');
   return convertedText;
 };