From 58053cff320a5cc9b4da0aba2637efa5da6ece6f Mon Sep 17 00:00:00 2001
From: Marcelo Junior <marceloadsj1@gmail.com>
Date: Sun, 25 Aug 2019 00:48:48 +0200
Subject: [PATCH] Warning if a class name is used multiple times

---
 src/compiler/compile/nodes/Element.ts | 43 +++++++++++++++++++++++----
 1 file changed, 37 insertions(+), 6 deletions(-)

diff --git a/src/compiler/compile/nodes/Element.ts b/src/compiler/compile/nodes/Element.ts
index 593f303a2f..a8bc6e539a 100644
--- a/src/compiler/compile/nodes/Element.ts
+++ b/src/compiler/compile/nodes/Element.ts
@@ -181,12 +181,12 @@ export default class Element extends Node {
 					break;
 
 				case 'Transition':
-				{
-					const transition = new Transition(component, this, scope, node);
-					if (node.intro) this.intro = transition;
-					if (node.outro) this.outro = transition;
-					break;
-				}
+					{
+						const transition = new Transition(component, this, scope, node);
+						if (node.intro) this.intro = transition;
+						if (node.outro) this.outro = transition;
+						break;
+					}
 
 				case 'Animation':
 					this.animation = new Animation(component, this, scope, node);
@@ -268,6 +268,7 @@ export default class Element extends Node {
 		}
 
 		this.validate_attributes();
+		this.validate_classes();
 		this.validate_bindings();
 		this.validate_content();
 		this.validate_event_handlers();
@@ -470,6 +471,36 @@ export default class Element extends Node {
 		}
 	}
 
+	validate_classes() {
+		const classAttribute = this.attributes.find(
+			attribute => !attribute.is_spread && attribute.name.toLowerCase() === "class"
+		);
+
+		let value: string | true = '';
+
+		if (classAttribute) {
+			value = classAttribute.get_static_value()
+		}
+
+		const classNames = String(value).split(" ");
+
+		this.classes.forEach(class_directive => {
+			const { name } = class_directive;
+
+			name.split(",").forEach(className => {
+				if (classNames.includes(className)) {
+					this.component.warn(this, {
+						code: `class-name-multiple-attrs`,
+						message: `Class: avoid using class name '${className}' in more than one class attribute`
+					});
+
+				} else {
+					classNames.push(className);
+				}
+			})
+		})
+	}
+
 	validate_bindings() {
 		const { component } = this;