Swith administration theme then back gives warnings [#158927]

On fresh HEAD install, switch administration theme to any option, then back to system default and the following warnings come up:

    * warning: array_map() [function.array-map]: Argument #2 should be an array in D:\websites\drupal-HEAD\modules\system\system.module on line 1233.
    * warning: array_key_exists() [function.array-key-exists]: The second argument should be either an array or an object in D:\websites\drupal-HEAD\modules\system\system.module on line 1271.
    * warning: array_keys() [function.array-keys]: The first argument should be an array in D:\websites\drupal-HEAD\modules\system\system.module on line 1248.
    * warning: array_key_exists() [function.array-key-exists]: The second argument should be either an array or an object in D:\websites\drupal-HEAD\modules\system\system.module on line 1271.
    * warning: array_keys() [function.array-keys]: The first argument should be an array in D:\websites\drupal-HEAD\modules\system\system.module on line 1248.

Cannot reproduce afer the first time unless you start again from scratch with a new database. On a new install though, can reproduce every time.

Comment	File	Size	Author
#2	admin_theme_0.patch	1.05 KB	John Morahan
#2

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

chx CreditAttribution: chx commented 26 September 2007 at 19:38

I confirm this behaviour. Indeed, after install the first but only the first switch back gives a warning. I have never seen any bug like this.

Comment #2

John Morahan CreditAttribution: John Morahan commented 26 September 2007 at 20:41

Status:

Active

» Needs review

File	Size
admin_theme_0.patch	1.05 KB

It's trying to initialize the blocks for the new admin "theme" which is actually 0 for the system default.

Comment #3

webernet CreditAttribution: webernet commented 26 September 2007 at 21:04

Status:

Needs review

» Reviewed & tested by the community

Tested OK, and the code looks fine.

Comment #4

Gábor Hojtsy

he/him

Hungarian

Hungary

CreditAttribution: Gábor Hojtsy commented 27 September 2007 at 13:13

Status:

Reviewed & tested by the community

» Fixed

It is kind of interesting to have 0 the default admin theme, but I understand it is done to default to the site theme unless specifically selected. Anyway, the patch looked good, so committed.