[mojo-dev] [jira] (MJS-50) sonar-javascript: function naming convention check throws error for classes

[Greg Jopa (JIRA)]

Title: Message Title

Greg Jopa created an issue   Mojo's _javascript_ Tools / MJS-50 sonar-_javascript_: function naming convention check throws error for classes Issue Type: Bug Affects Versions: 2.0 Assignee: Christopher Hunt Created: 14/Feb/14 1:00 PM Environment: Sonar _javascript_ Plugin 1.5 (http://docs.codehaus.org/display/SONAR/_javascript_+Plugin) Priority: Major Reporter: Greg Jopa In _javascript_, it's common to define a Class with a capital letter. For example: function Person() { } var person1 = new Person(); var person2 = new Person(); (source: https://developer.mozilla.org/en-US/docs/Web/_javascript_/Introduction_to_Object-Oriented_JavaScript) Note that the class name starts with a capital letter which is considered best practice in _javascript_. The function name check does not allow for a function name to start with a capital letter (https://github.com/SonarCommunity/sonar-_javascript_/blob/master/_javascript_-checks/src/main/java/org/sonar/_javascript_/checks/FunctionNameCheck.java). I think this rule should be changed to allow for functions to start with a capital letter. Add Comment   This message was sent by Atlassian JIRA (v6.1.6#6162-sha1:7af547c) --------------------------------------------------------------------- To unsubscribe from this list, please visit: http://xircles.codehaus.org/manage_email