Closed (won't fix)
Project:
Drupal core
Version:
6.9
Component:
base system
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
21 May 2008 at 13:53 UTC
Updated:
19 Jun 2013 at 10:58 UTC
function system_init() {
// Use the administrative theme if the user is looking at a page in the admin/* path.
if (arg(0) == 'admin' || (variable_get('node_admin_theme', '0') && arg(0) == 'node' && (arg(1) == 'add' || arg(2) == 'edit'))) {
global $custom_theme;
...
should be
if (arg(0) == 'admin' || (variable_get('node_admin_theme', '0') && arg(0) == 'node' && (arg(1) == 'add' || arg(2) == 'edit' || arg(2) == 'delete'))) {
Comments
Comment #1
mjtitus commentedShould really be:
if (arg(0) == 'admin' || (variable_get('node_admin_theme', '0') && arg(0) == 'node' && (arg(1) == 'add' || arg(2) == 'edit' || arg(2) == 'delete' || arg(2) == 'revisions'))) {
Comment #2
jlarky commentedComment #3
thedavidmeister commentedI agree with the sentiment here, unfortunately changes to the API like this won't make it into D6. You may be interested in theme key https://drupal.org/project/themekey
The admin theme switching works completely differently in D7+ so this issue is not relevant there.
Needs review means that there is a patch in the queue, there is no patch here. Please refer to https://drupal.org/node/156119
Comment #4
thedavidmeister commentedwhile I agree with the sentiment here, we won't be changing the API in D6, checkout https://drupal.org/project/themekey for a potential solution.
Please don't set to "needs review" without submitting a working patch - https://drupal.org/node/156119
admin themes work completely differently in D7+ so this issue doesn't apply there.
Comment #5
thedavidmeister commentedugh, double post >.<