From 983696bc58f50366b139bff42f931d40ba5f4a8b Mon Sep 17 00:00:00 2001
From: Tobias Theobald <tobitheo@gmail.com>
Date: Sun, 27 Oct 2024 23:32:06 +0100
Subject: [PATCH] Fix broken nfo generation due to missed API change when
 switching xml builder libs - fixes #216 Also makes the workflow a bit more
 robust so that less confusing retry messages are logged and artwork is
 downloaded even if nfo creation fails

---
 src/lib/Video.ts | 19 ++++++++++++++++---
 1 file changed, 16 insertions(+), 3 deletions(-)

diff --git a/src/lib/Video.ts b/src/lib/Video.ts
index edbc047..af755fe 100644
--- a/src/lib/Video.ts
+++ b/src/lib/Video.ts
@@ -140,11 +140,23 @@ export class Video extends Attachment {
 						logger.log("Download complete!");
 						if (settings.extras.saveNfo) {
 							logger.log("Saving .nfo");
-							await this.saveNfo();
+							try {
+								await this.saveNfo();
+							} catch (error) {
+								// non-critical error
+								const message = this.parseErrorMessage(error);
+								logger.error(`Failed to save .nfo file! ${message} - Skipping`);
+							}
 						}
 						if (settings.extras.downloadArtwork) {
 							logger.log("Saving artwork");
-							await this.downloadArtwork();
+							try {
+								await this.downloadArtwork();
+							} catch (error) {
+								// non-critical error
+								const message = this.parseErrorMessage(error);
+								logger.error(`Failed to save artwork! ${message} - Skipping`);
+							}
 						}
 					}
 					// eslint-disable-next-line no-fallthrough
@@ -227,7 +239,8 @@ export class Video extends Attachment {
 			episode = match[2];
 		}
 		const nfo = builder
-			.create("episodedetails")
+			.create()
+			.ele("episodedetails")
 			.ele("title")
 			.txt(this.videoTitle)
 			.up()