Project

General

Profile

Actions

Bug #13657

closed

Activating commentpress-modern theme can cause fatal errors if commentpress-core plugin not active

Added by Boone Gorges almost 4 years ago. Updated almost 4 years ago.

Status:
Resolved
Priority name:
Normal
Assignee:
Category name:
-
Target version:
Start date:
2020-12-07
Due date:
% Done:

0%

Estimated time:
Deployment actions:

Description

See #13656. I haven't taken any steps to confirm what's going on here. It looks like we have a separate wp-content/themes/commentpress-modern directory, added in #12573. But the commentpress-modern theme is also shipped inside of the commentpress-core plugin, which uses register_theme_directory() to tell WP about it. The latter solution means that the situation should never arise where the plugin is inactive but the theme is active. But our separate copy of the theme is probably overriding the call to register_theme_directory()

Ray, could you please look at what's happening here? I think we want to remove wp-content/themes/commentpress-modern, but I'm not sure if sites currently using commentpress-modern will properly switch over to the bundled version of the theme, or whether we need some sort of migration tool. (Also, there's the further question of what happens when you are using a plugin-bundled theme, via register_theme_directory(), but the plugin is then deactivated; I assume that this is not what happened in the case of #13656, and I assume it means that WP will just fall back on the default theme, but this could use a check.)


Related issues

Related to CUNY Academic Commons - Bug #13656: site downResolved2020-12-06

Actions
Related to CUNY Academic Commons - Bug #12573: CommentPress Core IssuesNew2020-03-24

Actions
Related to CUNY Academic Commons - Bug #13715: https://ulysses.commons.gc.cuny.edu downResolved2020-12-16

Actions
Actions

Also available in: Atom PDF