Changes between Initial Version and Version 1 of Ticket #22252, comment 14
- Timestamp:
- 11/11/2012 09:54:06 PM (11 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #22252, comment 14
initial v1 3 3 The problem is that by this point, only one theme root exists, so template_root and stylesheet_root are left untouched. Deleting them here would fix most of these issues. 4 4 5 The remaining issue is that validate_current_theme() should actually validate that any stored theme roots are accurate, to catch potential edge cases. (Such as when the same bp-default theme is in both roots, then one of the roots disappears.) I'd like to try doing that without causing non-existent option calls on single-site, though. Basically:5 The remaining issue is that validate_current_theme() should actually validate that any stored theme roots are accurate, to catch potential edge cases. (Such as when the same bp-default theme is in both roots, then one of the roots disappears.) I'd like to try doing that without causing non-existent option calls ~~on single-site~~ when there is only one root, though. Basically: 6 6 * If there's one theme root, it should delete any _root options. 7 7 * If there's more than one theme root, it should make sure any _root options are accurate (similar to your code in switch_theme() in 2.patch).