![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
#39539 closed enhancement (invalid)
Combine and simplify strings
| Reported by: | Presskopp | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | Text Changes | Version: | 4.8 |
| Severity: | normal | Keywords: | |
| Cc: | Focuses: |
Description
We made the "unknown post type" invalid, see #39171
We could take this a step further and change
unknown to invalid in:
"Unknown Feed"
https://core.trac.wordpress.org/browser/trunk/src/wp-includes/widgets/class-wp-widget-rss.php?marks=74#L74
and maybe
"Unknown action."
https://core.trac.wordpress.org/browser/trunk/src/wp-admin/comment.php?marks=329#L329
Also the (missing) period could be discussed in those strings.
Attachments (1)
Change History (6)
#1
in reply to: ↑ description
@
10 years ago
Replying to Presskopp:
We made the "unknown post type" invalid, see #39171
The only reason for [39756] was to merge similar strings with the same meaning and remove duplicates. I don't see a benefit in changing other strings just to get rid of "unknown".
Moreover, an empty feed title doesn't make the feed invalid, so the existing string is more accurate.
#2
@
10 years ago
I noticed there are much more instaces of anything 'invalid' and only a few 'unknown' things. But following your input I think the ticket can be closed as wontfix.
Basic patch