In code, it seems contradictory. Exit early so we do not cache the data, nor do we wrap the result in a theme also TODO Please change this theme call as discussed at http://drupal.org/node/224333#theme_page.

I can't think of a compelling reason to actually load the theme, so I've removed it.

CommentFileSizeAuthor
dont-boot-theme.patch866 bytesdstol

Comments

dstol’s picture

dstol
Primary language English
commented
Status: Active » Needs review

Proper status.

instanceofjamie’s picture

instanceofjamie commented
Status: Needs review » Fixed

Applied - works as described.

instanceofjamie’s picture

instanceofjamie commented
Status: Fixed » Closed (won't fix)

I actually reverted this straight after on realising that we a: allow for a custom status page callback, and b: there's some talk of using a proper theme function in http://drupal.org/node/1337954. Think there's a better way to achieve this, and allow for more options.