search

aik099 / CodingStandard

The PHP_CodeSniffer coding standard I'm using on all of my projects
BSD 3-Clause "New" or "Revised" License
5 stars 2 forks source link

Exclusions for camelCase method/property naming #82

Closed aik099 closed 9 years ago

aik099 commented 9 years ago

There is a sniff, that enforces class method/property name to be in camelCase, e.g. methodName versus MethodName. Unfortunately for BC reasons public methods/properties can't be renamed.

Instead of just ignoring any public method/property from name validation I propose to:

  1. run name validation sniff only across all code base to detect what methods/properties do not comply
  2. create massive exclusion list for those classes and their sub-classes (maybe)
aik099 commented 9 years ago

Done in 54fdf028f79d8ce6ef532dedace0a89253b909bd. 249 errors solved on whole In-Portal code base.