Browse Source

Fix unused unicode ranges metric

Gaël Métais 4 years ago
parent
commit
201bf75bf7
2 changed files with 3 additions and 4 deletions
  1. 1 1
      lib/runner.js
  2. 2 3
      lib/tools/redownload/redownload.js

+ 1 - 1
lib/runner.js

@@ -79,7 +79,7 @@ var Runner = function(params) {
         });
 
         // Fix: don't display Unicode ranges if the module is not present in Phantomas
-        if (!data.toolsResults.phantomas.metrics.charactersCount) {
+        if (!data.toolsResults.phantomas.metrics.differentCharacters) {
             delete data.toolsResults.redownload.metrics.unusedUnicodeRanges;
             delete data.toolsResults.redownload.offenders.unusedUnicodeRanges;
         }

+ 2 - 3
lib/tools/redownload/redownload.js

@@ -63,10 +63,9 @@ var Redownload = function() {
             }
         }
 
-        // Prevent a bug with the font analyzer on empty pages
         var differentCharacters = '';
-        if (data.toolsResults.phantomas.offenders.charactersCount && data.toolsResults.phantomas.offenders.charactersCount.length > 0) {
-            differentCharacters = data.toolsResults.phantomas.offenders.charactersCount[0];
+        if (data.toolsResults.phantomas.offenders.differentCharacters && data.toolsResults.phantomas.offenders.differentCharacters.length > 0) {
+            differentCharacters = data.toolsResults.phantomas.offenders.differentCharacters[0];
         }
 
         // Transform every request into a download function with a callback when done