Merge pull request #6 from OpenframeProject/repush

jmwohl · jmwohl · commit 41001a4a9aec · 2016-04-08T14:04:13.000+02:00
Compare current and incoming artwork JSON, not just ID
diff --git a/src/controller.js b/src/controller.js
@@ -291,7 +291,7 @@ fc.changeArtwork = function() {
 
     return new Promise(function(resolve, reject) {
         // old artwork is new artwork, don't update.
-        if (old_artwork && old_artwork.id === new_artwork.id) {
+        if (old_artwork && JSON.stringify(old_artwork) === JSON.stringify(new_artwork)) {
             debug('new artwork same as current', old_artwork.id, new_artwork.id);
             return reject();
         }

Original file line number	Diff line number	Diff line change
`@@ -291,7 +291,7 @@ fc.changeArtwork = function() {`
`291`	`291`
`292`	`292`	`return new Promise(function(resolve, reject) {`
`293`	`293`	`// old artwork is new artwork, don't update.`
`294`		`- if (old_artwork && old_artwork.id === new_artwork.id) {`
	`294`	`+ if (old_artwork && JSON.stringify(old_artwork) === JSON.stringify(new_artwork)) {`
`295`	`295`	`debug('new artwork same as current', old_artwork.id, new_artwork.id);`
`296`	`296`	`return reject();`
`297`	`297`	`}`