Fixed bug #19256 : T_DOC_COMMENT in CSS files breaks ClassDefinitionN…

…ameSpacingSniff
squizlabs · Jan 30, 2012 · ea3ecf3 · ea3ecf3
1 parent 6a7ef1c
commit ea3ecf3
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 0 deletions.
diff --git a/CodeSniffer/Standards/Squiz/Sniffs/CSS/ClassDefinitionNameSpacingSniff.php b/CodeSniffer/Standards/Squiz/Sniffs/CSS/ClassDefinitionNameSpacingSniff.php
@@ -66,6 +66,7 @@ public function process(PHP_CodeSniffer_File $phpcsFile, $stackPtr)
         $endTokens = array(
                       T_CLOSE_CURLY_BRACKET,
                       T_COMMENT,
+                      T_DOC_COMMENT,
                       T_OPEN_TAG,
                      );
 

diff --git a/CodeSniffer/Standards/Squiz/Tests/CSS/ClassDefinitionNameSpacingUnitTest.css b/CodeSniffer/Standards/Squiz/Tests/CSS/ClassDefinitionNameSpacingUnitTest.css
@@ -31,6 +31,10 @@
     width: 1000px;
 }
 
+/**
+ * More styles below here.
+ */
+
 td.TableWidgetType-header.TableWidgetType-header-lastLogin,
 td.TableWidgetType-header.TableWidgetType-header-remove,
 td.TableWidgetType-header.TableWidgetType-header-email,

diff --git a/package.xml b/package.xml
@@ -41,6 +41,8 @@ http://pear.php.net/dtd/package-2.0.xsd">
     -- Second issue was notices caused by bug #19137
   - Fixed bug #19208 : UpperCaseConstantName reports class members
     -- Was also a problem with LowerCaseConstantName as well
+  - Fixed bug #19256 : T_DOC_COMMENT in CSS files breaks ClassDefinitionNameSpacingSniff
+    -- Thanks to Klaus Purer for the patch
  </notes>
  <contents>
   <dir name="/">