From: Leif Åstrand <leif@vaadin.com>
Date: Tue, 25 Mar 2014 07:03:52 +0000 (+0200)
Subject: Trigger a checkstyle error if importing the wrong Element class (#13287)
X-Git-Tag: 7.2.0.beta1~84
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=dc89e28f4d8251d0b721a2b9e143770a5fcda195;p=vaadin-framework.git

Trigger a checkstyle error if importing the wrong Element class (#13287)

Change-Id: Ief15e88c9c6fe5fd88dbf9f12783ac2f5ae617c6
---

diff --git a/checkstyle/import-control.xml b/checkstyle/import-control.xml
new file mode 100644
index 0000000000..076dbd8a22
--- /dev/null
+++ b/checkstyle/import-control.xml
@@ -0,0 +1,9 @@
+<?xml version="1.0"?>
+<!DOCTYPE import-control PUBLIC
+    "-//Puppy Crawl//DTD Import Control 1.1//EN"
+    "http://www.puppycrawl.com/dtds/import_control_1_1.dtd">
+
+<import-control pkg="com.vaadin">
+    <allow pkg=".*" regex="true" />
+    <disallow class="com.google.gwt.user.client.Element" />
+</import-control>
\ No newline at end of file
diff --git a/checkstyle/vaadin-checkstyle.xml b/checkstyle/vaadin-checkstyle.xml
index bcbbaf9171..ca572fe1a5 100644
--- a/checkstyle/vaadin-checkstyle.xml
+++ b/checkstyle/vaadin-checkstyle.xml
@@ -142,6 +142,10 @@
         <module name="IllegalImport" /> <!-- defaults to sun.* packages -->
         <module name="RedundantImport" />
         <module name="UnusedImports" />
+        <module name="ImportControl">
+            <property name="file" value="${checkstyle.importControl.file}" />
+            <property name="severity" value="error" />
+        </module>
 
 
         <!-- Checks for Size Violations. -->
diff --git a/common.xml b/common.xml
index cef335035d..59e32136b4 100644
--- a/common.xml
+++ b/common.xml
@@ -422,6 +422,7 @@
     <property name="cs.dir" location="${vaadin.basedir}/checkstyle" />
     <property name="cs.xml" location="${cs.dir}/vaadin-checkstyle.xml" />
     <property name="cs.header-file" location="${cs.dir}/header" />
+    <property name="cs.import-control-file" location="${cs.dir}/import-control.xml" />
     <taskdef resource="checkstyletask.properties" uri="antlib:com.puppycrawl.tools.checkstyle" classpathref="taskdefs.classpath" />
 
     <target name="checkstyle">
@@ -438,6 +439,7 @@
             </fileset>
             <formatter type="xml" toFile="${result.dir}/checkstyle-errors.xml" />
             <property key="checkstyle.header.file" file="${cs.header-file}" />
+            <property key="checkstyle.importControl.file" file="${cs.import-control-file}" />
         </cs:checkstyle>
     </target>
 </project>