Skip to:
Content

bbPress.org

Opened 4 years ago

Closed 3 years ago

#3366 closed defect (bug) (fixed)

Wrong punctuation in the error message

Reported by: immeet94's profile immeet94 Owned by: johnjamesjacoby's profile johnjamesjacoby
Milestone: 2.6.7 Priority: normal
Severity: normal Version: 2.1
Component: Component - Replies Keywords: has-patch
Cc:

Description

https://plugins.trac.wordpress.org/browser/bbpress/trunk/includes/replies/functions.php#L1215

'<strong>ERROR</strong>: A reply ID is required'

Miss the dot at the end of error message :)

Attachments (1)

3366.diff (676 bytes) - added by immeet94 4 years ago.

Download all attachments as: .zip

Change History (7)

@immeet94
4 years ago

This ticket was mentioned in Slack in #bbpress by immeet94. View the logs.


4 years ago

#2 @mercime
4 years ago

Also consider what's been brought up in core's Trac: Removing all caps for ERROR messages (I would including removing <strong> tags as well) as was done in this commit; and then possibly Remove 'Error:' prefixes from error notices

#3 @johnjamesjacoby
3 years ago

  • Component changed from General to Component - Replies
  • Milestone changed from Awaiting Review to 2.6.7
  • Owner set to johnjamesjacoby
  • Status changed from new to assigned
  • Version changed from trunk to 2.1

#4 @johnjamesjacoby
3 years ago

In 7177:

Replies: add missing period at the end of error string in bbp_move_reply_handler().

In trunk for 2.7.0. See #3366.

Props immeet94.

#5 @johnjamesjacoby
3 years ago

In 7178:

Replies: add missing period at the end of error string in bbp_move_reply_handler().

In branches/2.6 for 2.6.7. See #3366.

Props immeet94.

#6 @johnjamesjacoby
3 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed
Note: See TracTickets for help on using tickets.