add option to use a different pageshell

[cash]

The current page_draw() function has these arguments:

page_draw($title, $body, $sidebar = "")

Sidebar is handled through layouts rather than page_draw so that can be removed. It would also be great if we could specify a different pageshell from the default. For example, I may not want the normal header on a particular page.

[brettp]

heh I have an uncommitted change that does exactly this. Was waiting to commit until I could check if nothing was trying to use $sidebar.

[adenton]

Hi Guys,

I was thinking that the sidebar parameter is far too restrictive. I would change that to the $vars array. I am going to submit a change along the lines of that. There is nothing stopping people passing $sidebar in through the array but it would be much more useful to have additional parameters, especially if people want to vary pageshells.

Asa

[brettp]

A similar change was made in [svn:6602] to allow pageshell, but I'm for removing removing the sidebar option. If we remove the sidebar option, we could go ahead and add $vars as a 4th optional argument:

function page_draw($title, $body, $page_shell = 'page_shells/default', $vars = array())

[cash]

See also #2349

[brettp]

(In [svn:6704]) Fixes #2195. 3rd and 4th params of page_draw() are now $page_shell and $vars.