Opened 13 years ago
Closed 12 years ago
#21789 closed enhancement (fixed)
Twenty Twelve as default theme
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 3.5 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Bundled Theme | Keywords: | has-patch |
Focuses: | Cc: |
Description (last modified by )
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)
Change History (18)
#1
@
13 years ago
- Description modified (diff)
- Summary changed from Twentytwelve as default theme to Twenty Twelve as default theme
#3
@
13 years ago
21789.2.patch also updates default sidebars_widgets
option (see [18643]).
Twenty Twelve only has 3 sidebars.
#4
@
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.
#8
@
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)
- 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.
- 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
@
13 years ago
If we were to go for number 1 above, the scenario's i see are (and the current bevhaviour)
from | to | ideal behaviour | current behaviour |
3.4 | 3.5-beta2 | installs 2012 | installs 2012 |
3.4 (with 2012) | 3.5-beta1 | overwrites 2012 | skips 2012 |
3.5-beta2 | 3.5-beta3 | overwrites 2012 | skips 2012 |
3.5-RC3 | 3.5 | overwrites 2012 | skips 2012 |
3.4 | 3.5 | installs 2012 | installs 2012 |
3.4 (with 2012) | 3.5 | skips 2012 | skips 2012 |
#10
@
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)
#13
@
13 years ago
- Owner changed from dd32 to nacin
Final logic table:
from | to | ideal behaviour | old behaviour | [22226] behaviour | [22227] behaviour |
3.4 | 3.5-beta2 | installs 2012 | installs 2012 | installs 2012 | installs 2012 |
3.4 (with 2012) | 3.5-beta1 | overwrites 2012 | skips 2012 | skips 2012 | overwrites 2012 |
3.5-beta2 | 3.5-beta3 | overwrites 2012 | skips 2012 | overwrites 2012 | overwrites 2012 |
3.5-RC3 | 3.5 | overwrites 2012 | skips 2012 | overwrites 2012 | overwrites 2012 |
3.4 | 3.5 | installs 2012 | installs 2012 | installs 2012 | installs 2012 |
3.4 | 3.5.1 | installs 2012 | installs 2012 | installs 2012 | installs 2012 |
3.4 (with 2012) | 3.5 | skips 2012 | skips 2012 | skips 2012 | skips 2012 |
3.5 (with 2012) | 3.5.1 | skips 2012 | skips 2012 | skips 2012 | skips 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.
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.