diff --git a/themes/engines/phptemplate/phptemplate.engine b/themes/engines/phptemplate/phptemplate.engine index 92046945802dabf683f797f6c5bc60bae7586354..e64b554ea6ef3ed565e4c7896aaffde4f35c7d81 100644 --- a/themes/engines/phptemplate/phptemplate.engine +++ b/themes/engines/phptemplate/phptemplate.engine @@ -92,16 +92,17 @@ function _phptemplate_default_variables($hook, $variables) { function phptemplate_features() { return array( - 'logo', - 'toggle_favicon', - 'toggle_name', - 'toggle_search', - 'toggle_slogan', - 'toggle_mission', - 'toggle_primary_links', - 'toggle_secondary_links', - 'toggle_node_user_picture', - 'toggle_comment_user_picture'); + 'logo', + 'toggle_comment_user_picture', + 'toggle_favicon', + 'toggle_mission', + 'toggle_name', + 'toggle_node_user_picture', + 'toggle_primary_links', + 'toggle_search', + 'toggle_secondary_links', + 'toggle_slogan' + ); } /** @@ -154,33 +155,32 @@ function phptemplate_page($content) { } $variables = array( + 'breadcrumb' => theme('breadcrumb', drupal_get_breadcrumb()), + 'closure' => theme('closure'), + 'content' => '<!-- begin content -->' . $content . '<!-- end content -->', + 'footer_message' => variable_get('site_footer', FALSE), + 'head' => drupal_get_html_head(), 'head_title' => implode(' | ', $head_title), + 'help' => theme('help'), 'language' => $GLOBALS['locale'], - 'site' => variable_get('site_name', 'drupal'), - 'head' => drupal_get_html_head(), - 'onload_attributes' => theme('onload_attribute'), + 'layout' => $layout, 'logo' => theme_get_setting('logo'), + 'messages' => theme('status_messages'), + 'mission' => $mission, + 'onload_attributes' => theme('onload_attribute'), + 'primary_links' => theme_get_setting('primary_links'), 'site_name' => (theme_get_setting('toggle_name') ? variable_get('site_name', 'Drupal') : ''), 'site_slogan' => (theme_get_setting('toggle_slogan') ? variable_get('site_slogan', '') : ''), 'search_box' => theme_get_setting('toggle_search'), - 'search_url' => url('search'), 'search_button_text' => t('search'), 'search_description' => t('Enter the terms you wish to search for.'), - 'title' => drupal_get_title(), - 'primary_links' => theme_get_setting('primary_links'), + 'search_url' => url('search'), 'secondary_links' => theme_get_setting('secondary_links'), - 'breadcrumb' => theme('breadcrumb', drupal_get_breadcrumb()), - 'tabs' => theme('menu_local_tasks'), - 'messages' => theme('status_messages'), - 'layout' => $layout, - 'help' => theme('help'), - 'styles' => theme_get_styles(), - 'mission' => $mission, 'sidebar_left' => $sidebar_left, - 'content' => '<!-- begin content -->' . $content . '<!-- end content -->', 'sidebar_right' => $sidebar_right, - 'footer_message' => variable_get('site_footer', FALSE), - 'closure' => theme('closure') + 'styles' => theme_get_styles(), + 'tabs' => theme('menu_local_tasks'), + 'title' => drupal_get_title() ); if ((arg(0) == 'node') && is_int(arg(1))) { $variables['node'] = node_load(arg(1)); @@ -202,18 +202,18 @@ function phptemplate_node($node, $main = 0, $page = 0) { } $variables = array( - 'title' => check_plain($node->title), - 'node_url' => url('node/'. $node->nid), - 'terms' => theme('links', $taxonomy), - 'name' => format_name($node), - 'date' => format_date($node->created), - 'content' => ($main && $node->teaser) ? $node->teaser : $node->body, - 'links' => $node->links ? theme('links', $node->links) : '', - 'taxonomy' => $taxonomy, - 'main' => $main, - 'page' => $page, - 'node' => $node // we pass the actual node to allow more customization - ); + 'content' => ($main && $node->teaser) ? $node->teaser : $node->body, + 'date' => format_date($node->created), + 'links' => $node->links ? theme('links', $node->links) : '', + 'main' => $main, + 'name' => format_name($node), + 'node' => $node, // we pass the actual node to allow more customization + 'node_url' => url('node/'. $node->nid), + 'page' => $page, + 'taxonomy' => $taxonomy, + 'terms' => theme('links', $taxonomy), + 'title' => check_plain($node->title) + ); // Flatten the node object's member fields. $variables = array_merge(object2array($node), $variables); @@ -233,18 +233,17 @@ function phptemplate_node($node, $main = 0, $page = 0) { */ function phptemplate_comment($comment, $links = 0) { return _phptemplate_callback('comment', array( - 'new' => $comment->new ? t('new') : '', + 'author' => format_name($comment), 'comment' => $comment, - 'submitted' => t('Submitted by %a on %b.', array('%a' => format_name($comment), '%b' => format_date($comment->timestamp))), - 'title' => l($comment->subject, $_GET['q'], NULL, NULL, "comment-$comment->cid"), - - 'picture' => theme_get_setting('toggle_comment_user_picture') ? theme('user_picture', $comment) : '', - 'links' => $links, 'content' => $comment->comment, - 'author' => format_name($comment), - 'date' => format_date($comment->timestamp) + 'date' => format_date($comment->timestamp), + 'links' => $links, + 'new' => $comment->new ? t('new') : '', + 'picture' => theme_get_setting('toggle_comment_user_picture') ? theme('user_picture', $comment) : '', + 'submitted' => t('Submitted by %a on %b.', + 'title' => l($comment->subject, $_GET['q'], NULL, NULL, "comment-$comment->cid") )); } @@ -262,9 +261,9 @@ function phptemplate_block($block) { */ function phptemplate_box($title, $content, $region = 'main') { return _phptemplate_callback('box', array( - 'title' => $title, 'content' => $content, - 'region' => $region + 'region' => $region, + 'title' => $title )); } @@ -304,7 +303,7 @@ function _phptemplate_default($hook, $variables, $file = null) { } if ($file) { - extract($variables); // Extract the vars to local namespace + extract($variables); // Extract the vars to local namespace ob_start(); // Start output buffering include($file); // Include the file $contents = ob_get_contents(); // Get the contents of the buffer