#24213 closed defect (bug) (fixed)
Revisions: wp_list_post_revisions( format => 'form-table' ) is broken
Reported by: | ocean90 | Owned by: | ocean90 |
---|---|---|---|
Milestone: | 3.6 | Priority: | high |
Severity: | normal | Version: | 3.6 |
Component: | Revisions | Keywords: | has-patch |
Focuses: | Cc: |
Description
Current state:
Since [23769].
The doc block includes "@todo split into two functions (list, form-table) ?"
Is the form table still needed and if yes should the new UI respond to the request?
Attachments (3)
Change History (11)
#2
in reply to:
↑ 1
@
11 years ago
Replying to SergeyBiryukov:
[23769] removed a big portion of previous UI in
form-table
mode (seewp-includes/post-template.php
, lines 1405 to 1432).
I guess the remaining part should be removed as well. The
format
argument can be deprecated, as it's no longer used anywhere.
yes, this should be able to be safely removed
#3
follow-up:
↓ 4
@
11 years ago
- Cc a.hoereth added
- Keywords has-patch needs-testing added; needs-patch removed
#4
in reply to:
↑ 3
@
11 years ago
Replying to a.hoereth:
thanks! post-template.php.patch looks good: tested and list works as expected.
#5
@
11 years ago
24213.patch adds also left, right and parent to the deprecated check and updates inline docs.
Nacin proposed to remove the wp_parse_args()
/extract()
and check $args['parent']
etc directly.
#6
@
11 years ago
List of plugins which use the function: https://gist.github.com/markjaquith/a862bc3deac10d19de35
[23769] removed a big portion of previous UI in
form-table
mode (seewp-includes/post-template.php
, lines 1405 to 1432).I guess the remaining part should be removed as well. The
format
argument can be deprecated, as it's no longer used anywhere.