Opened 3 years ago
Closed 3 years ago
#8913 closed defect (bug) (maybelater)
Check PHPcs Coding standard
| Reported by: | viralsampat | Owned by: | espellcaste |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | Build/Test Tools | Version: | |
| Severity: | normal | Keywords: | has-patch |
| Cc: |
Description
Hello Team,
I have reviewed the code and found PHPcs errors & warnings in some of the files:
Here, I mentioned all those files:
/src/bp-xprofile/bp-xprofile-filters.php
/src/bp-xprofile/bp-xprofile-cssjs.php
/src/bp-xprofile/bp-xprofile-caps.php
/src/bp-xprofile/classes/class-bp-xprofile-field.php
/src/bp-xprofile/classes/class-bp-xprofile-field-type-checkbox.php
Thanks,
Attachments (7)
Change History (12)
#1
@
3 years ago
- Component Core → Extended Profile
- Keywords has-patch added; needs-patch reporter-feedback dev-feedback removed
- Milestone Awaiting Review → Up Next
Thanks for your ticket & patch. As we're doing a lot of changes in 12.0, I prefer to delay things about WP code standards to next milestone.
@
3 years ago
I have checked above mentioned issue and founds few files. Here, I have added its patch.
@
3 years ago
I have checked above mentioned issue and founds few more files. I have applied its patch.
@
3 years ago
I have checked above mentioned issue and founds few more files. I have applied its patch.
@
3 years ago
I have checked above mentioned issue and founds few more files. I have applied its patch.
@
3 years ago
I have checked above mentioned issue and founds few more files. I have applied its patch.
@
3 years ago
I have checked above mentioned issue and founds few more files. I have applied its patch.
#5
@
3 years ago
- Milestone 14.0.0
- Resolution → maybelater
- Status accepted → closed
So, I have plans to update this in a better way. Making it easier to code review via github. So, I'm going to resolve this and handle those updates in the next milestone.
I also tried to apply those patches but patch -p failed. Not sure why.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
I have checked above mentioned issue and resolved few errors and warning. Here, I have added its patch.