Skip to:
Content

BuddyPress.org

Opened 7 years ago

Closed 7 years ago

#8197 closed defect (bug) (fixed)

src\bp-core\admin\bp-core-admin-functions.php coding standard mistakes

Reported by: sharaz Owned by: slaFFik
Priority: normal Milestone: 6.0.0
Component: Administration Version: 5.0.0
Severity: normal Keywords: has-patch commit
Cc:

Description

According to the WordPress coding standards. I noticed there lot of mistakes like improper use of coma with the array, Do Block comment extra line, printf improper arguments and indentation mistakes.

Attachments (2)

8197.patch​ (21.1 KB ) - added by sharaz 7 years ago.
1.8197.patch​ (20.8 KB ) - added by sharaz 7 years ago.

Download all attachments as: .zip

Change History (5)

@sharaz
7 years ago

@sharaz
7 years ago

#1 @imath
7 years ago

  • Keywords has-patch reporter-feedback added; needs-patch removed
  • Milestone Awaiting Review → 6.0.0

Hi @sharaz

Thanks for your contribution. Just to be sure is 1.8197.patch replacing 8197.patch ?

I believe we should discuss in a master ticket about adding WordPress Coding Standards tool to our build/test tools. And define like a strategy about cleaning "existing" code, just like we did for inline documentation. For example: do 1 component at a time.

For now I'll take a deeper look at your patch before 6.0 release.

#2 @imath
7 years ago

  • Component Core → Administration
  • Keywords commit added; reporter-feedback removed
  • Owner set to slaFFik

It looks good. Let's have it in.

#3 @imath
7 years ago

  • Resolution → fixed
  • Status new → closed

In 12538:

Admin: fix PHP code standards (including Translators comments)

Props sharaz

Fixes #8197

Note: See TracTickets for help on using tickets.