Special case images to accomodate for legacy non-dir images (#747)

* Case for legacy non-dir images * Change from sourceFile to fileName to keep it consistent with the property being passed into function
AudiusProject · Aug 11, 2020 · 5fc3a6a · 5fc3a6a
1 parent 48e29e0
commit 5fc3a6a
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/creator-node/src/routes/nodeSync.js b/creator-node/src/routes/nodeSync.js
@@ -308,9 +308,12 @@ async function _nodesync (req, walletPublicKeys, creatorNodeEndpoint) {
               // The files inside the directory are synced separately
               if (nonTrackFile.type !== 'dir') {
                 // if it's an image file, we need to pass in the actual filename because the gateway request is /ipfs/Qm123/<filename>
-                if (nonTrackFile.type === 'image') {
+                // need to also check fileName is not null to make sure it's a dir-style image. non-dir images won't have a 'fileName' db column
+                if (nonTrackFile.type === 'image' && nonTrackFile.fileName !== null) {
                   return saveFileForMultihash(req, nonTrackFile.multihash, nonTrackFile.storagePath, userReplicaSet, nonTrackFile.fileName)
-                } else return saveFileForMultihash(req, nonTrackFile.multihash, nonTrackFile.storagePath, userReplicaSet)
+                } else {
+                  return saveFileForMultihash(req, nonTrackFile.multihash, nonTrackFile.storagePath, userReplicaSet)
+                }
               }
             }
           ))