Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#16610 closed defect (bug) (fixed)

Twenty Ten 1.2 compat for WP 3.0

Reported by: nacin Owned by:
Priority: normal Milestone: 3.1
Component: Themes Version:
Severity: normal Keywords: has-patch 2nd-opinion
Cc:

Description

The theme directory doesn't have compatibility checks. Additionally, Twenty Ten was 3.0's default theme, so it should remain compat.

Issues:

  1. get_post_format() function_exists() checks, handled in r17474 and r17469.
  1. use_default_gallery_style filter doesn't exist in 3.0.
  1. show_recent_comments_widget_style filter doesn't exist in 3.0.

Neither of these are important. I honestly don't care much if we lose compatibility on those layers. But working on patches.

Attachments (2)

16610.diff (1.1 KB) - added by nacin 2 years ago.
Untested and unconvinced.
16610.2.diff (1.1 KB) - added by nacin 2 years ago.

Download all attachments as: .zip

Change History (8)

Looks like we'll need to do a raw version compare for these. did_action() would work for use_default_gallery_style filter, except that it doesn't work for filters. show_recent_comments_widget_style would be pretty tough to detect otherwise.

nacin2 years ago

Untested and unconvinced.

  • Keywords has-patch 2nd-opinion added

nacin2 years ago

Second patch handles gallery style and ignores recent comments style.

comment:4   ryan2 years ago

Looks good.

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

(In [17477]) Back compat with the gallery_style filter in Twenty Ten. fixes #16610 for the 3.1 branch.

(In [17478]) Back compat with the gallery_style filter in Twenty Ten. fixes #16610 for trunk.

Note: See TracTickets for help on using tickets.