Opened 12 years ago
Closed 12 years ago
#23783 closed defect (bug) (fixed)
Update strings to reflect the changes in wp_post_revision_title()
Reported by: | SergeyBiryukov | Owned by: | |
---|---|---|---|
Milestone: | 3.6 | Priority: | normal |
Severity: | normal | Version: | 3.6 |
Component: | Revisions | Keywords: | |
Focuses: | Cc: |
Description
Background: #23497
wp_post_revision_title()
got a backward-incompatible change in [23506].
Previously, it used to return the revision date and time:
http://core.trac.wordpress.org/browser/tags/3.5.1/wp-includes/post-template.php#L1294
Now it returns the revision author name first:
http://core.trac.wordpress.org/browser/trunk/wp-includes/post-template.php?rev=23653#L1319
This creates three issues:
- The strings in
edit-form-advanced.php
still expect date and time first:
http://core.trac.wordpress.org/browser/trunk/wp-admin/edit-form-advanced.php?rev=23683#L39 Changing "from" to "by" and updating the translator comment would probably be enough. However, that still might be incompatible with custom strings (see "Customizing the messages" example):
http://codex.wordpress.org/Function_Reference/register_post_type#Example. - The "by (author name)" part in the list of revisions is redundant (see the screenshot).
wp_post_revision_title()
inline description should be updated.
Attachments (1)
Change History (7)
#2
follow-up:
↓ 3
@
12 years ago
The instance in revision.php
is unused since [23506]:
http://core.trac.wordpress.org/browser/trunk/wp-admin/revision.php?rev=23639#L61
#3
in reply to:
↑ 2
@
12 years ago
Replying to SergeyBiryukov:
The instance in
revision.php
is unused since [23506]:
http://core.trac.wordpress.org/browser/trunk/wp-admin/revision.php?rev=23639#L61
removed in next patch
#4
@
12 years ago
i believe i addressed the main issue in my latest patch by breaking out the code as a new function. the new function still needs updated phpdoc and translator instructions, but at least it won't interfere with other code using wp_post_revision_title() like the one you found or a plugin.
Another option might be to revert the changes to
wp_post_revision_title()
and introduce a new function.