cvs: pear /PHP_CodeSniffer package.xml /PHP_CodeSniffer/CodeSniffer/Standards/PEAR/Sniffs/NamingConventions ValidClassNameSniff.php /PHP_CodeSniffer/CodeSniffer/Standards/PEAR/Tests/NamingConventions ValidClassNameUnitTest.inc ValidClassNameUnitTest.php

[email protected] ("Greg Sherwood")
Newsgroups php.pear.cvs
Message-ID <cvssquiz1209344548@cvsserver>
squiz		Mon Apr 28 01:02:28 2008 UTC

  Modified files:              
    /pear/PHP_CodeSniffer	package.xml 
    /pear/PHP_CodeSniffer/CodeSniffer/Standards/PEAR/Sniffs/NamingConventions	
                                                                             	ValidClassNameSniff.php 
    /pear/PHP_CodeSniffer/CodeSniffer/Standards/PEAR/Tests/NamingConventions	
                                                                            	ValidClassNameUnitTest.inc 
                                                                            	ValidClassNameUnitTest.php 
  Log:
  Fixed error in PEAR ValidClassNameSniff when checking class names with double underscores
squiz-20080428010228.txt (text/plain, 9.3 KB)
http://cvs.php.net/viewvc.cgi/pear/PHP_CodeSniffer/package.xml?r1=1.261&r2=1.262&diff_format=u
Index: pear/PHP_CodeSniffer/package.xml
diff -u pear/PHP_CodeSniffer/package.xml:1.261 pear/PHP_CodeSniffer/package.xml:1.262
--- pear/PHP_CodeSniffer/package.xml:1.261	Mon Apr 21 01:32:33 2008
+++ pear/PHP_CodeSniffer/package.xml	Mon Apr 28 01:02:28 2008
@@ -23,74 +23,16 @@
  <date>2008-04-21</date>
  <time>10:44:00</time>
  <version>
-  <release>1.1.0a1</release>
-  <api>1.1.0a1</api>
+  <release>1.1.0RC1</release>
+  <api>1.1.0RC1</api>
  </version>
  <stability>
-  <release>alpha</release>
-  <api>alpha</api>
+  <release>beta</release>
+  <api>beta</api>
  </stability>
  <license uri="http://matrix.squiz.net/developer/tools/php_cs/licence">BSD License</license>
  <notes>
-  - Added support for multiple tokenizers so PHP_CodeSniffer can check more than just PHP files
-    -- PHP_CodeSniffer now has a JS tokenizer for checking JavaScript files
-    -- Sniffs need to be updated to work with additional tokenizers, or new sniffs written for them
-  - phpcs now exits with status 2 if the tokenier extension has been disabled (feature request #13269)
-  - Added scripts/phpcs-svn-pre-commit that can be used as an SVN pre-commit hook
-    - Also reworked the way the phpcs script works to make it easier to wrap it with other functionality
-    - Thanks to Jack Bates for the contribution
-  - Fixed error in phpcs error message when a supplied file does not exist
-  - Fixed a cosmetic error in AbstractPatternSniff where the "found" string was missing some content
-  - Added sniffs that implement part of the PMD rule catalog to the Generic standard
-    -- Thanks to Manuel Pichler for the contribution of all these sniffs.
-  - Squiz FunctionCommentThrowTagSniff no longer throws errors for function that only throw variables
-  - Generic ScopeIndentSniff now has private member to enforce exact indent matching
-  - Replaced Squiz DisallowCountInLoopsSniff with Squiz DisallowSizeFunctionsInLoopsSniff
-    -- Thanks to Jan Miczaika for the sniff
-  - Squiz BlockCommentSniff now checks inline doc block comments
-  - Squiz InlineCommentSniff now checks inline doc block comments
-  - Squiz BlockCommentSniff now checks for no blank line before first comment in a function
-  - Squiz DocCommentAlignmentSniff now ignores inline doc block comments
-  - Squiz ControlStructureSpacingSniff now ensures no blank lines at the start of control structures
-  - Squiz ControlStructureSpacingSniff now ensures no blank lines between control structure closing braces
-  - Squiz IncrementDecrementUsageSniff now ensures inc/dec ops are bracketed in string concats
-  - Squiz IncrementDecrementUsageSniff now ensures inc/dec ops are not used in arithmetic operations
-  - Squiz FunctionCommentSniff no longer throws errors if return value is mixed but function returns void somewhere
-  - Squiz OperatorBracketSniff no allows function call brackets to count as operator brackets
-  - Squiz DoubleQuoteUsageSniff now supports \x \f and \v (feature request #13365)
-  - Squiz ComparisonOperatorUsageSniff now supports JS files
-  - Squiz ControlSignatureSniff now supports JS files
-  - Squiz ForLoopDeclarationSniff now supports JS files
-  - Squiz OperatorBracketSniff now supports JS files
-  - Squiz InlineControlStructureSniff now supports JS files
-  - Generic LowerCaseConstantSniff now supports JS files
-  - Generic DisallowTabIndentSniff now supports JS files
-  - Generic MultipleStatementAlignmentSniff now supports JS files
-  - Added Squiz ObjectMemberCommaSniff to ensure the last member of a JS object is not followed by a comma
-  - Added Squiz ConstantCaseSniff to ensure the PHP constants are uppercase and JS lowercase
-  - Added Squiz JavaScriptLintSniff to check JS files with JSL
-    -- Set path using phpcs --config-set jsl_path /path/to/jsl
-  - Added MySource FirebugConsoleSniff to ban the use of "console" for JS variable and function names
-  - Added MySource JoinStringsSniff to enforce the use of join() to concatenate JS strings
-  - Added MySource AssignThisSniff to ensure this is only assigned to a var called self
-  - Added MySource DisallowNewWidgetSniff to ban manual creation of widget objects
-  - Removed warning shown in Zend CodeAnalyzerSniff when the ZCA path is not set
-  - Fixed error in Squiz ValidVariableNameSniff when checking vars in the form $obj->$var
-  - Fixed error in Squiz DisallowMultipleAssignmentsSniff when checking vars in the form $obj->$var
-  - Fixed error in Squiz InlineCommentSniff where comments for class constants were seen as inline
-  - Fixed error in Squiz BlockCommentSniff where comments for class constants were not ignored
-  - Fixed error in Squiz OperatorBracketSniff where negative numbers were ignored during comparisons
-  - Fixed error in Squiz FunctionSpacingSniff where functions after member vars reported incorrect spacing
-  - Fixed bug #13062 : Interface comments aren't handled in PEAR standard
-    -- Thanks to Manuel Pichler for the path
-  - Fixed bug #13119 : php minimum requirement need to be fix
-  - Fixed bug #13156 : Bug in Squiz_Sniffs_PHP_NonExecutableCodeSniff
-  - Fixed bug #13158 : Strange behaviour in AbstractPatternSniff
-  - Fixed bug #13169 : Undefined variables
-  - Fixed bug #13178 : Catch exception in File.php
-  - Fixed bug #13254 : Notices output in checkstyle report causes XML issues
-  - Fixed bug #13446 : crash with src of phpMyAdmin
-    -- Thanks to Manuel Pichler for the path
+  - Fixed error in PEAR ValidClassNameSniff when checking class names with double underscores
  </notes>
  <contents>
   <dir name="/">
http://cvs.php.net/viewvc.cgi/pear/PHP_CodeSniffer/CodeSniffer/Standards/PEAR/Sniffs/NamingConventions/ValidClassNameSniff.php?r1=1.9&r2=1.10&diff_format=u
Index: pear/PHP_CodeSniffer/CodeSniffer/Standards/PEAR/Sniffs/NamingConventions/ValidClassNameSniff.php
diff -u pear/PHP_CodeSniffer/CodeSniffer/Standards/PEAR/Sniffs/NamingConventions/ValidClassNameSniff.php:1.9 pear/PHP_CodeSniffer/CodeSniffer/Standards/PEAR/Sniffs/NamingConventions/ValidClassNameSniff.php:1.10
--- pear/PHP_CodeSniffer/CodeSniffer/Standards/PEAR/Sniffs/NamingConventions/ValidClassNameSniff.php:1.9	Thu Feb 28 22:01:17 2008
+++ pear/PHP_CodeSniffer/CodeSniffer/Standards/PEAR/Sniffs/NamingConventions/ValidClassNameSniff.php	Mon Apr 28 01:02:28 2008
@@ -10,7 +10,7 @@
  * @author    Marc McIntyre <[email protected]>
  * @copyright 2006 Squiz Pty Ltd (ABN 77 084 670 600)
  * @license   http://matrix.squiz.net/developer/tools/php_cs/licence BSD Licence
- * @version   CVS: $Id: ValidClassNameSniff.php,v 1.9 2008/02/28 22:01:17 squiz Exp $
+ * @version   CVS: $Id: ValidClassNameSniff.php,v 1.10 2008/04/28 01:02:28 squiz Exp $
  * @link      http://pear.php.net/package/PHP_CodeSniffer
  */
 
@@ -92,7 +92,9 @@
             } else {
                 $newName = strtoupper($firstBit{0}).substr($firstBit, 1).'_';
                 foreach ($nameBits as $bit) {
-                    $newName .= strtoupper($bit{0}).substr($bit, 1).'_';
+                    if ($bit !== '') {
+                        $newName .= strtoupper($bit{0}).substr($bit, 1).'_';
+                    }
                 }
 
                 $newName = rtrim($newName, '_');
http://cvs.php.net/viewvc.cgi/pear/PHP_CodeSniffer/CodeSniffer/Standards/PEAR/Tests/NamingConventions/ValidClassNameUnitTest.inc?r1=1.2&r2=1.3&diff_format=u
Index: pear/PHP_CodeSniffer/CodeSniffer/Standards/PEAR/Tests/NamingConventions/ValidClassNameUnitTest.inc
diff -u pear/PHP_CodeSniffer/CodeSniffer/Standards/PEAR/Tests/NamingConventions/ValidClassNameUnitTest.inc:1.2 pear/PHP_CodeSniffer/CodeSniffer/Standards/PEAR/Tests/NamingConventions/ValidClassNameUnitTest.inc:1.3
--- pear/PHP_CodeSniffer/CodeSniffer/Standards/PEAR/Tests/NamingConventions/ValidClassNameUnitTest.inc:1.2	Thu Feb 28 22:01:17 2008
+++ pear/PHP_CodeSniffer/CodeSniffer/Standards/PEAR/Tests/NamingConventions/ValidClassNameUnitTest.inc	Mon Apr 28 01:02:28 2008
@@ -41,4 +41,8 @@
 
 interface ___ {}
 
+class Invalid__Name {}
+
+interface Invalid__Name {}
+
 ?>
http://cvs.php.net/viewvc.cgi/pear/PHP_CodeSniffer/CodeSniffer/Standards/PEAR/Tests/NamingConventions/ValidClassNameUnitTest.php?r1=1.4&r2=1.5&diff_format=u
Index: pear/PHP_CodeSniffer/CodeSniffer/Standards/PEAR/Tests/NamingConventions/ValidClassNameUnitTest.php
diff -u pear/PHP_CodeSniffer/CodeSniffer/Standards/PEAR/Tests/NamingConventions/ValidClassNameUnitTest.php:1.4 pear/PHP_CodeSniffer/CodeSniffer/Standards/PEAR/Tests/NamingConventions/ValidClassNameUnitTest.php:1.5
--- pear/PHP_CodeSniffer/CodeSniffer/Standards/PEAR/Tests/NamingConventions/ValidClassNameUnitTest.php:1.4	Thu Feb 28 22:01:17 2008
+++ pear/PHP_CodeSniffer/CodeSniffer/Standards/PEAR/Tests/NamingConventions/ValidClassNameUnitTest.php	Mon Apr 28 01:02:28 2008
@@ -10,7 +10,7 @@
  * @author    Marc McIntyre <[email protected]>
  * @copyright 2006 Squiz Pty Ltd (ABN 77 084 670 600)
  * @license   http://matrix.squiz.net/developer/tools/php_cs/licence BSD Licence
- * @version   CVS: $Id: ValidClassNameUnitTest.php,v 1.4 2008/02/28 22:01:17 squiz Exp $
+ * @version   CVS: $Id: ValidClassNameUnitTest.php,v 1.5 2008/04/28 01:02:28 squiz Exp $
  * @link      http://pear.php.net/package/PHP_CodeSniffer
  */
 
@@ -54,6 +54,8 @@
                 38 => 1,
                 40 => 2,
                 42 => 2,
+                44 => 1,
+                46 => 1,
                );
 
     }//end getErrorList()
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.