Make WordPress Core

Opened 13 years ago

Closed 12 years ago

#21789 closed enhancement (fixed)

Twenty Twelve as default theme

Reported by: markoheijnen's profile markoheijnen Owned by: nacin's profile nacin
Milestone: 3.5 Priority: normal
Severity: normal Version:
Component: Bundled Theme Keywords: has-patch
Focuses: Cc:

Description (last modified by SergeyBiryukov)

Since Twenty Twelve will be included in 3.5 lets also make it a default for the current users that installes a WordPress installation from trunk/nightly build.

Attachments (4)

21789.patch (1.2 KB) - added by markoheijnen 13 years ago.
dashboard_filter.patch (2.0 KB) - added by markoheijnen 13 years ago.
21789.2.patch (2.5 KB) - added by SergeyBiryukov 13 years ago.
21789.diff (5.6 KB) - added by dd32 13 years ago.

Download all attachments as: .zip

Change History (18)

@markoheijnen
13 years ago

#1 @SergeyBiryukov
13 years ago

  • Description modified (diff)
  • Summary changed from Twentytwelve as default theme to Twenty Twelve as default theme

#2 @markoheijnen
13 years ago

Added a patch to remove the reference of Twenty Eleven in the Welcome panel and adds a filter to be able the same from inside the theme.

#3 @SergeyBiryukov
13 years ago

21789.2.patch also updates default sidebars_widgets option (see [18643]).

Twenty Twelve only has 3 sidebars.

#4 @nacin
13 years ago

I don't think the beta tester plugin is currently installing or updating Twenty Twelve. We should make sure it can with these changes.

Also, Twenty Twelve is going to be 1.0 before 3.5, and available for 3.4 installs. We should make sure we do install it with 3.5 if it doesn't exist, but don't upgrade it if it does.

This might require our wp-content/bundled code to get a bit more flexible. cc dd32.

#5 @nacin
13 years ago

In [22050]:

Twenty Twelve 1.0, barring any last-minute tweaks. see #21789.

#6 @nacin
13 years ago

In [22054]:

Make Twenty Twelve the default theme for new installs. props markoheijnen, SergeyBiryukov. see #21789.

#7 @nacin
13 years ago

  • Owner set to dd32
  • Status changed from new to assigned

#8 @dd32
13 years ago

Also, Twenty Twelve is going to be 1.0 before 3.5, and available for 3.4 installs. We should make sure we do install it with 3.5 if it doesn't exist, but don't upgrade it if it does.

This should be the case already. We have a check for it if ( $wp_filesystem->is_dir($dest . $filename) ) continue; here.

I don't think the beta tester plugin is currently installing or updating Twenty Twelve. We should make sure it can with these changes.

Upgrading using the builtin upgrader will NOT be upgrading the bundled theme or plugins, thoughts: (for before beta 2)

  1. Core should detect that either old, or new version is an non-stable release (nacin suggests existence of a dash in version number), and in the event of that, force overwrites of bundled items.
  2. Add a filter around the skip, defaulting to the current existance checks, to allow for plugins (beta tester) to force overwrites - although point 1 would probably cover this.. there's no harm in adding the filter?

#9 @dd32
13 years ago

If we were to go for number 1 above, the scenario's i see are (and the current bevhaviour)

fromtoideal behaviourcurrent behaviour
3.43.5-beta2installs 2012installs 2012
3.4 (with 2012) 3.5-beta1overwrites 2012skips 2012
3.5-beta2 3.5-beta3overwrites 2012skips 2012
3.5-RC3 3.5overwrites 2012skips 2012
3.4 3.5installs 2012installs 2012
3.4 (with 2012)3.5skips 2012skips 2012

#10 @dd32
13 years ago

In attempting this I've run into #11639 again, and the simplest option is to just fix it.

I'm attaching a patch here which covers both this AND #11639 - without the #11639 component of the patch, the 3.4->3.5-beta1 branch in the above table would fail (but the others would work).

@nacin: What do you think, Fix #11639 or leave scenario 2 from the above table to "fail"? (while fixing scenario 3 and 4)

@dd32
13 years ago

#11 @dd32
13 years ago

In [22226]:

When upgrading a site to/from a Beta, alpha, or non-stable release, overwrite any existing bundled themes & plugins with the version from the release zip. See #21789

#12 @dd32
13 years ago

In [22227]:

Replace the hard coded version and PHP/MySQL requirements in update-core.php by pulling in the new version.php file during installation. Fixes #22015 #11639. See #21789

#13 @dd32
13 years ago

  • Owner changed from dd32 to nacin

Final logic table:

fromtoideal behaviourold behaviour[22226] behaviour[22227] behaviour
3.43.5-beta2installs 2012installs 2012installs 2012installs 2012
3.4 (with 2012) 3.5-beta1overwrites 2012skips 2012skips 2012overwrites 2012
3.5-beta2 3.5-beta3overwrites 2012skips 2012overwrites 2012overwrites 2012
3.5-RC3 3.5overwrites 2012skips 2012overwrites 2012overwrites 2012
3.4 3.5installs 2012installs 2012installs 2012installs 2012
3.4 3.5.1installs 2012installs 2012installs 2012installs 2012
3.4 (with 2012)3.5skips 2012skips 2012skips 2012skips 2012
3.5 (with 2012)3.5.1skips 2012skips 2012skips 2012skips 2012

In the case where it skips installing/updating it does NOT update the language files for the theme. Theme updates via the Theme Upgrader will continue to miss translations unless they're added to the .org theme zip.

I think that's all needed for the ticket.

#14 @dd32
12 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed
Note: See TracTickets for help on using tickets.