Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

CamelCasePropertyName does allow Capitals at the beginning while CamelCaseVariableName does not #276

Closed
ravage84 opened this issue Mar 22, 2015 · 1 comment · Fixed by #833
Closed

CamelCasePropertyName does allow Capitals at the beginning while CamelCaseVariableName does not #276

ravage84 opened this issue Mar 22, 2015 · 1 comment · Fixed by #833
Assignees
@ravage84
Labels
Bug Good first issue If you want to help, this may be a good start
Milestone
2.12.0

Comments

@ravage84
Copy link
Member

The rule CamelCasePropertyName does allow capitals at the beginning while CamelCaseVariableName does not.

https://github.com/phpmd/phpmd/blob/2.2.1/src/main/php/PHPMD/Rule/Controversial/CamelCasePropertyName.php#L70
https://github.com/phpmd/phpmd/blob/2.2.1/src/main/php/PHPMD/Rule/Controversial/CamelCaseVariableName.php#L93

@manuelpichler do you know of a particular reason why this is so?
@ravage84 ravage84 modified the milestone: 2.3.3 Nov 7, 2015
@ravage84 ravage84 modified the milestones: 2.3.3, 2.4.2 Mar 8, 2016
@manuelpichler manuelpichler modified the milestones: 2.4.2, 2.4.3 Mar 10, 2016
@ravage84 ravage84 modified the milestones: 2.4.3, 2.4.4 Apr 25, 2016
@ravage84 ravage84 self-assigned this Jun 27, 2016
@ravage84 ravage84 added the Bug label Jun 3, 2017
@ravage84
Copy link
Member Author

ravage84 commented Jun 3, 2017

Actually, I suspect a bug.

@tvbeek tvbeek added the Good first issue If you want to help, this may be a good start label Sep 24, 2019
@kylekatarnls kylekatarnls modified the milestones: 2.7.1, 2.8.1 Oct 19, 2019
@kylekatarnls kylekatarnls modified the milestones: 2.8.1, 2.8.2 Dec 27, 2019
@kylekatarnls kylekatarnls modified the milestones: 2.8.2, 2.8.3 Feb 25, 2020
@kylekatarnls kylekatarnls modified the milestones: 2.8.3, 2.10.0 May 3, 2020
@martzd martzd mentioned this issue Oct 5, 2020
Merged
@kylekatarnls kylekatarnls modified the milestones: 2.10.0, 2.11.0 Apr 11, 2021
@kylekatarnls kylekatarnls removed this from the 2.11.0 milestone Nov 27, 2021
@kylekatarnls kylekatarnls added this to the 2.12.0 milestone Nov 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ravage84 ravage84

Labels
Bug Good first issue If you want to help, this may be a good start
Milestone
2.12.0
Development

Successfully merging a pull request may close this issue.

Disallow capital letter at the beginning in CamelCasePropertyName rule martzd/phpmd
4 participants
@manuelpichler @ravage84 @tvbeek @kylekatarnls