From 604857648b705a3e2acb26fe44935b9d48753cfb Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ga=C3=ABl=20M=C3=A9tais?= Date: Mon, 4 May 2015 17:03:59 +0200 Subject: [PATCH] Easier to understand css offenders --- lib/metadata/policies.js | 15 ++++++++++++--- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/lib/metadata/policies.js b/lib/metadata/policies.js index 9e63f4f..8e55155 100644 --- a/lib/metadata/policies.js +++ b/lib/metadata/policies.js @@ -390,11 +390,20 @@ var policies = { return { count: offenders.length, list: offenders.map(function(offender) { + if (offender === '[inline CSS] (Empty CSS was provided)') { + return { + error: 'Empty style tag', + file: null, + line: null, + column: null + }; + } + var parts = /^(?:(?:<([^ \(]*)>|\[inline CSS\]) ?)?(?:\((((?! @ ).)*)(?: @ (\d+):(\d+))?\))?$/.exec(offender); if (parts) { return { - error: parts[2], + error: parts[2] || 'Unknown parsing error', file: parts[1] || null, line: (parts[4] && parts[5]) ? parseInt(parts[4], 10) : null, column: (parts[4] && parts[5]) ? parseInt(parts[5], 10) : null @@ -406,11 +415,11 @@ var policies = { if (parts) { return { - error: parts[1], + error: parts[1] || 'Unknown parsing error', file: parts[2] || null, line: parseInt(parts[3], 10), column: parseInt(parts[4], 10) - } + }; }