소스 검색

fix false 'downloaded' status being returned when null hash is 'checked'

jomo 10 년 전
부모
커밋
04c167f5aa
1개의 변경된 파일4개의 추가작업 그리고 3개의 파일을 삭제
  1. 4 3
      modules/helpers.js

+ 4 - 3
modules/helpers.js

@@ -111,10 +111,11 @@ exp.get_image_hash = function(uuid, callback) {
           if (err) {
           if (err) {
             callback(err, -1, details && details.hash);
             callback(err, -1, details && details.hash);
           } else {
           } else {
-            var oldhash = details && details.hash || "none";
-            logging.debug(uuid + " old hash: " + oldhash);
+            // skin is only checked (3) when uuid known AND hash didn't change
+            // in all other cases the skin is downloaded (2)
+            var status = details && (details.hash == hash) ? 3 : 2;
+            logging.debug(uuid + " old hash: " + (details && details.hash));
             logging.log(uuid + " hash: " + hash);
             logging.log(uuid + " hash: " + hash);
-            var status = hash == oldhash ? 3 : 2;
             callback(null, status, hash);
             callback(null, status, hash);
           }
           }
         });
         });