Skip to:
Content

BuddyPress.org

Opened 11 months ago

Last modified 7 weeks ago

#8198 assigned enhancement

src\class-buddypress.php coding standards mistakes

Reported by: sharaz Owned by: espellcaste
Milestone: Up Next 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 11 months ago.

Download all attachments as: .zip

Change History (9)

@sharaz
11 months ago

#1 @sharaz
11 months ago

  • Keywords has-patch added

#2 @imath
11 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
9 months ago

  • Milestone changed from 6.0.0 to Up Next

#4 @espellcaste
6 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 5 months ago by espellcaste (previous) (diff)

#5 @espellcaste
5 months ago

  • Keywords needs-refresh removed

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

#6 @imath
5 months ago

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

#7 @espellcaste
2 months ago

@johnjamesjacoby Were you able to take a look?

#8 @imath
7 weeks ago

  • Milestone changed from 7.0.0 to Up Next

Let's try to work on this during next development cycle

Note: See TracTickets for help on using tickets.