From 423bfe47d8c5c103ce9a39a808f4b258f63fa3d5 Mon Sep 17 00:00:00 2001
From: Lynn Smeria <ae@cephea.de>
Date: Sun, 11 Nov 2018 11:35:06 +0100
Subject: [PATCH] Try extra hard to extract dates

---
 package.json    |  2 +-
 src/generate.js | 35 ++++++++++++++++++++++++++---------
 2 files changed, 27 insertions(+), 10 deletions(-)

diff --git a/package.json b/package.json
index 0962659..9e0d898 100644
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "publikator",
-  "version": "0.17.3",
+  "version": "0.17.5",
   "main": "index.js",
   "repository": "https://github.com/aengl/publikator.git",
   "author": "Lynn Smeria <ae@cephea.de>",
diff --git a/src/generate.js b/src/generate.js
index 4cb4dfe..00114d7 100644
--- a/src/generate.js
+++ b/src/generate.js
@@ -17,18 +17,30 @@ const collect = (tracks, callback) => {
   return values.length === 1 ? values[0] : values;
 };
 
+/**
+ * Returns the first defined metadata at the specified path.
+ */
+const find = (tracks, path) =>
+  _.get(tracks.find(track => !_.isNil(_.get(track, path))), path);
+
 /**
  * Creates release information for a single album.
  */
 const getAlbumInfo = (root, tracks) => ({
   layout: 'album',
   slug: path.basename(root),
-  name: tracks[0].common.album || '',
+  name: find(tracks, 'common.album'),
   artists: collect(tracks, t => t.common.artists || t.common.artist),
   bitrate: collect(tracks, t => t.format.bitrate),
   trackCount: tracks.length,
-  cover: tracks[0].cover || null,
-  date: tracks[0].common.date || null,
+  cover: find(tracks, 'cover'),
+  date:
+    find(tracks, 'common.date') ||
+    find(tracks, 'common.originaldate') ||
+    find(tracks, 'all.ORIGINALDATE') ||
+    find(tracks, 'common.year') ||
+    find(tracks, 'common.originalyear') ||
+    find(tracks, 'all.ORIGINALYEAR'),
   tracks: _.sortBy(tracks, 'common.track.no'),
 });
 
@@ -47,6 +59,14 @@ const getTrackInfo = (albumInfo, trackIndex) => {
   };
 };
 
+/**
+ * Converts an object to YAML.
+ */
+const toYaml = obj =>
+  yaml.safeDump(obj, {
+    skipInvalid: true,
+  });
+
 module.exports = {
   /**
    * Generates Jekyll-compatible release data
@@ -68,7 +88,7 @@ module.exports = {
           } track(s)`
         );
         const albumInfo = getAlbumInfo(albumRoot, tracks);
-        const releaseInfo = `---\n${yaml.safeDump(albumInfo)}---\n`;
+        const releaseInfo = `---\n${toYaml(albumInfo)}---\n`;
         await fs.writeFile(
           path.resolve(albumCollectionRoot, `${baseName}.md`),
           releaseInfo
@@ -84,10 +104,7 @@ module.exports = {
               `${track.slug}.md`
             );
             await fs.ensureFile(trackInfoPath);
-            await fs.writeFile(
-              trackInfoPath,
-              `---\n${yaml.safeDump(track)}---\n`
-            );
+            await fs.writeFile(trackInfoPath, `---\n${toYaml(track)}---\n`);
           })
         );
 
@@ -99,6 +116,6 @@ module.exports = {
     debug(`generating data for ${albumsInfo.length} album(s)`);
     const albumsInfoPath = path.resolve(root, '_data', 'albums.yml');
     await fs.ensureFile(albumsInfoPath);
-    await fs.writeFile(albumsInfoPath, yaml.safeDump(albumsInfo));
+    await fs.writeFile(albumsInfoPath, toYaml(albumsInfo));
   },
 };