Skip to:
Content

BuddyPress.org

Opened 7 months ago

Last modified 4 weeks ago

#8198 assigned enhancement

src\class-buddypress.php coding standards mistakes

Reported by: sharaz Owned by: espellcaste
Milestone: 7.0.0 Priority: normal
Severity: normal Version: 5.0.0
Component: Core Keywords: has-patch
Cc:

Description

Comments mistakes, Object initialization, indentation and more ...

Attachments (1)

8198.patch (21.6 KB) - added by sharaz 7 months ago.

Download all attachments as: .zip

Change History (7)

@sharaz
7 months ago

#1 @sharaz
7 months ago

  • Keywords has-patch added

#2 @imath
7 months ago

  • Keywords 2nd-opinion added
  • Milestone changed from Awaiting Review to 6.0.0

Thanks for your contribution and patch. Let's look at it during 6.0.0.

PS: I think @johnjamesjacoby 's opinion would be nice about code formatting changes.

#3 @imath
5 months ago

  • Milestone changed from 6.0.0 to Up Next

#4 @espellcaste
2 months ago

  • Keywords needs-refresh added; 2nd-opinion removed
  • Milestone changed from Up Next to 7.0.0
  • Owner set to espellcaste
  • Status changed from new to assigned
  • Type changed from defect (bug) to enhancement

Looking at the changes, I don't see possible breaking ones. The only thing I dislike is the alignmnet of the irregular_map classes. But the only reason is because adding new classes forces you to change everything rather than one line.

Last edited 4 weeks ago by espellcaste (previous) (diff)

#5 @espellcaste
4 weeks ago

  • Keywords needs-refresh removed

@imath or @jjj This seems ok to me. Any objections about the patch?

#6 @imath
4 weeks ago

Nothing on my side, but changes reformat a lot the file so I’d like @johnjamesjacoby ‘s opinion about it before moving forward.

Note: See TracTickets for help on using tickets.