Make WordPress Core

Opened 7 years ago

Closed 5 years ago

#42735 closed enhancement (fixed)

Add missing periods to some (error) messages

Reported by: presskopp's profile Presskopp Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 5.4 Priority: normal
Severity: normal Version:
Component: Text Changes Keywords: has-patch needs-refresh
Focuses: Cc:

Description

All those 'Invalid xyz'-like messages do end with a period, but not all :)

Attachments (1)

42735.diff (2.7 KB) - added by Presskopp 7 years ago.

Download all attachments as: .zip

Change History (8)

@Presskopp
7 years ago

#1 @Presskopp
7 years ago

  • Keywords has-patch added

#2 @Presskopp
6 years ago

  • Keywords needs-refresh added

Another missing period:

An error occurred during the activation

https://core.trac.wordpress.org/browser/branches/5.0/src/wp-activate.php?marks=124#L124

EDIT: Ah, it's a H2 and probably won't be changed, not sure about the
"Unknown Feed"

Last edited 6 years ago by Presskopp (previous) (diff)

#3 @Presskopp
6 years ago

and another:

%s comment permanently deleted

#4 @Presskopp
6 years ago

  • Summary changed from Add missing periods to some error messages to Add missing periods to some (error) messages

#5 @SergeyBiryukov
5 years ago

  • Milestone changed from Awaiting Review to 5.4
  • Owner set to SergeyBiryukov
  • Status changed from new to reviewing

#6 @SergeyBiryukov
5 years ago

In 47152:

Text Changes: Add missing periods to comment status update notices, for consistency.

Props Presskopp.
See #42735.

#7 @SergeyBiryukov
5 years ago

  • Resolution set to fixed
  • Status changed from reviewing to closed

In 47153:

Text Changes: Add missing periods to some error messages, for consistency.

Props Presskopp.
Fixes #42735.

Note: See TracTickets for help on using tickets.