Skip to:
Content

BuddyPress.org

Opened 13 years ago

Closed 13 years ago

#5316 closed defect (bug) (fixed)

One semicolon too many at the messages template

Reported by: MacPresss Owned by: boonebgorges
Priority: normal Milestone: 2.0
Component: Messages Version: 1.9.1
Severity: minor Keywords: has-patch
Cc:

Description

One semicolon too mant at the messages.php template.

Line 57

Old:

<?php bp_get_template_part( 'members/single/messages/notices-loop' );; ?>

New:

<?php bp_get_template_part( 'members/single/messages/notices-loop' ); ?>

Attachments (1)

messages.php​ (1.3 KB ) - added by MacPresss 13 years ago.
Patch ?

Download all attachments as: .zip

Change History (3)

@MacPresss
13 years ago

Patch ?

#1 @boonebgorges
13 years ago

  • Milestone Awaiting Review → 2.0
  • Type enhancement → defect (bug)

Good catch!

(You've submitted a full PHP file rather than a patch against a SVN checkout. Here's a pretty straightforward guide to getting a SVN dev environment setup and using TortoiseSVN to generate patch files: ​http://www.doitwithwp.com/submit-patch-wordpress-core-beginners/ It's for WP, but you can apply the same ideas to BP. Our SVN trunk address is ​http://buddypress.svn.wordpress.org/trunk.)

#2 @boonebgorges
13 years ago

  • Owner set to boonebgorges
  • Resolution → fixed
  • Status new → closed

In 7717:

Remove duplicate semicolon in messages template.

Fixes #5316

Props MacPresss

Note: See TracTickets for help on using tickets.