There's an extra ellipsis in the description. I've attached the patch that corrects it.
How small :-P

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lut4rp’s picture

Assigned: Unassigned » lut4rp
keith.smith’s picture

Component: system.module » user interface text
Status: Active » Needs work

Yes, your patch does correct a needless ellipsis.

But, in the new string:

Settings for logging and alerts modules. Various modules can route Drupal's system events to different destination, such as syslog, database, email etc.

There are a few issues. "destination" should be plural, and there should be a comma between "email" and "etc".

But, if it were me, I'd just end the string after the first sentence. That second sentence is not needed, IMO.

lut4rp’s picture

I checked Wikipedia and here's what it recommends...

Generally, however, a comma is not used before and, or or etc. in a list: "John, Warren and Peter came to dinner." "Fruit, vegetables or cereals may be substituted." "Why not hire your skis, boots, overpants etc.?"

So, the statement should be...

Settings for logging and alert modules. Various modules can route Drupal's system events to different destinations, such as syslog, database, email etc.

I think alerts should be changed to alert. Sounds more correct?

lut4rp’s picture

FileSize
1.15 KB
lut4rp’s picture

FileSize
967 bytes

Oops, the last patch had an extra space.

keith.smith’s picture

No reason to have dueling wikipedia entries, but the entry at http://en.wikipedia.org/wiki/Et_cetera seems to contradict your post above.

Typically, the abbreviated versions should always be followed by a full stop (period), and it is customary—even in British English where the serial comma is typically not used—that "etc." always be preceded by a comma. Thus:

A, B, C, etc.

not:

A, B, C etc

Unfortunately, I'm not near my Chicago Manual of Style nor any other style guide at the moment for a more definitive reference.

lut4rp’s picture

FileSize
1022 bytes

I just spend some time researching this on the web and in ##english. keith.smith, you're right, there should be a comma before the "etc." as well.
Re-rolling and attaching the new patch.

lut4rp’s picture

Status: Needs work » Needs review
Issue tags: +this-is-easy
catch’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: -this-is-easy +Quick fix
webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed, thanks! :)

Not sure if it's worth back-porting this to 6.x, as it would break translations.

Status: Fixed » Closed (fixed)
Issue tags: -Quick fix

Automatically closed -- issue fixed for 2 weeks with no activity.