Skip to:
Content

BuddyPress.org

Opened 12 years ago

Closed 12 years ago

Last modified 5 years ago

#5878 closed defect (bug) (fixed)

Error message punctuation, try again.

Reported by: netweb Owned by: boonebgorges
Priority: normal Milestone: 2.2
Component: I18N Version:
Severity: minor Keywords: has-patch
Cc:

Description

Various error messages throughout BuddyPress end with try again.

A dozen of these use try again and do not include the full stop period at the end of the sentence.

For consistency these should all be the same.

Attachments (2)

try-again.diff (6.4 KB ) - added by netweb 12 years ago.
try-again.2.diff (29.8 KB ) - added by tharsheblows 12 years ago.

Download all attachments as: .zip

Change History (12)

@netweb
12 years ago

#1 @boonebgorges
12 years ago

  • Keywords needs-refresh added
  • Milestone Awaiting Review2.2

Thanks. If we're going to make string changes, let's actually make the text good in all cases. Executive decision: "Please try again" should always be a separate sentence - not separated with a semi-colon, and definitely not with a comma.

#2 @netweb
12 years ago

Ha! I saw the semi-colons but didn't notice a single comma in any of those strings ;)

This belongs here http://theoatmeal.com/comics/semicolon

#3 @DJPaul
12 years ago

  • Keywords good-first-bug added; has-patch removed

#4 @DJPaul
12 years ago

  • Component CoreText Changes

#5 @tharsheblows
12 years ago

  • Keywords has-patch added; needs-refresh good-first-bug removed

I've tested and it works, have searched for more to do but can't find any... looks ready to commit.

#6 @netweb
12 years ago

All I did in my original patch was add a . (period) to the end of the sentences and per Boone's comment above there are numerous instances of inconsistencies with ; (semi-colons) and , (commas) used throughout that code, both in my patch and other cases I missed.

e.g.

  • There was an error when promoting that user, please try again. <- Comma
  • There was an error when demoting that user; please try again.' <- Semi colon

Thus all strings throughout should look something like:

  • This is the first statement. Please try again.

#7 @tharsheblows
12 years ago

Oh ha, I looked the wrong way and searched for

  • Please try again,
  • Please try again;

I will take the advice given multiple times and try again!

#8 @tharsheblows
12 years ago

Ok, third (second?) time's the charm. Maybe.

#9 @boonebgorges
12 years ago

  • Owner set to boonebgorges
  • Resolutionfixed
  • Status newclosed

In 9190:

Remove some run-on sentences, also some comma splices, they were kind of annoying, they made BuddyPress more annoying to use, this is much better, thanks for using BuddyPress.

Props tharsheblows.
Fixes #5878.

#10 @DJPaul
10 years ago

  • Component Locale - Text ChangesI18N
Note: See TracTickets for help on using tickets.