The firs loock is really good , but i missed the "menu" site map wich appears in all the examples, but not in my website, and then it doesn't work in the topics

www.rodats.com/forum

Comments

michelle’s picture

Category: bug » support
Status: Active » Postponed (maintainer needs more info)

I don't know what menu you are talking about. The topics not being themed is a symptom of not correctly adding the function call to _phptemplate_variables().

Michelle

asespat’s picture

This is how it's seen http://www.rodats.com/node/17#comment-2

This is code i have

$style = theme_get_setting('litejazz_style');
if (!$style)
{
$style = 'blue';
}

drupal_add_css(drupal_get_path('theme', 'litejazz') . '/css/' . $style . '.css', 'theme');
drupal_add_css(drupal_get_path('theme', 'litejazz') . '/content.css', 'theme');

if (theme_get_setting('litejazz_iepngfix')) {
drupal_add_js(drupal_get_path('theme', 'litejazz') . '/js/jquery.pngFix.js', 'theme');
}

if (theme_get_setting('litejazz_themelogo'))
{

if (module_exists('advanced_profile')) {
$vars = advanced_profile_addvars($hook, $vars);
}
if (module_exists('advanced_forum')) {
$vars = advanced_forum_addvars($hook, $vars);
}

function _phptemplate_variables($hook, $variables = array()) {
$variables['logo'] = base_path() . path_to_theme() . '/images/' . theme_get_setting('litejazz_style') . '/logo.png';

return $variables;

}
}

michelle’s picture

Status: Postponed (maintainer needs more info) » Fixed

You didn't actually put the calls in the function...

function _phptemplate_variables($hook, $variables = array()) {
  if (module_exists('advanced_profile')) {
    $variables = advanced_profile_addvars($hook, $variables);
  }
 
  if (module_exists('advanced_forum')) {
    $variables = advanced_forum_addvars($hook, $variables);
  }

  $variables['logo'] = base_path() . path_to_theme() . '/images/' . theme_get_setting('litejazz_style') . '/logo.png';

  return $variables;
}

Michelle

asespat’s picture

Now i copy and paste the code and its like this , but still doesn't work fine

<?php
// $Id: template.php,v 1.2 2007/08/23 22:55:46 roopletheme Exp $

if (is_null(theme_get_setting('litejazz_style'))) {
global $theme_key;
// Save default theme settings
$defaults = array(
'litejazz_style' => 0,
'litejazz_width' => 0,
'litejazz_fixedwidth' => '850',
'litejazz_breadcrumb' => 0,
'litejazz_iepngfix' => 0,
'litejazz_themelogo' => 0,
);

variable_set(
str_replace('/', '_', 'theme_'. $theme_key .'_settings'),
array_merge(theme_get_settings($theme_key), $defaults)
);
// Force refresh of Drupal internals
theme_get_setting('', TRUE);
}

function litejazz_regions() {
return array(
'sidebar_left' => t('left sidebar'),
'sidebar_right' => t('right sidebar'),
'content_top' => t('content top'),
'content_bottom' => t('content bottom'),
'header' => t('header'),
'user1' => t('user1'),
'user2' => t('user2'),
'user3' => t('user3'),
'user4' => t('user4'),
'user5' => t('user5'),
'user6' => t('user6'),
'footer' => t('footer')
);
}

$style = theme_get_setting('litejazz_style');
if (!$style)
{
$style = 'blue';
}

drupal_add_css(drupal_get_path('theme', 'litejazz') . '/css/' . $style . '.css', 'theme');
drupal_add_css(drupal_get_path('theme', 'litejazz') . '/content.css', 'theme');

if (theme_get_setting('litejazz_iepngfix')) {
drupal_add_js(drupal_get_path('theme', 'litejazz') . '/js/jquery.pngFix.js', 'theme');
}

if (theme_get_setting('litejazz_themelogo'))
{

if (module_exists('advanced_profile')) {
$vars = advanced_profile_addvars($hook, $vars);
}
if (module_exists('advanced_forum')) {
$vars = advanced_forum_addvars($hook, $vars);
}

function _phptemplate_variables($hook, $variables = array()) {
if (module_exists('advanced_profile')) {
$variables = advanced_profile_addvars($hook, $variables);
}

if (module_exists('advanced_forum')) {
$variables = advanced_forum_addvars($hook, $variables);
}

$variables['logo'] = base_path() . path_to_theme() . '/images/' . theme_get_setting('litejazz_style') . '/logo.png';

return $variables;
}

function _phptemplate_variables($hook, $variables = array()) {
$variables['logo'] = base_path() . path_to_theme() . '/images/' . theme_get_setting('litejazz_style') . '/logo.png';

return $variables;

}
}

michelle’s picture

Take the extra calls to advprofile/forum out and make sure if (theme_get_setting('litejazz_themelogo')) is returning true. Other than that, you'll need to ask in that theme's queue.

Michelle

asespat’s picture

With Garland Works Fine

but i can't with litejazz

if you know where i can inform or maybe is still is not ready for that theme?

and thaks a lot Michelle

I'm very happy with this module , Drupal forum was the only weak point of drupal and now is really nice

michelle’s picture

Project: Advanced Forum » litejazz
Version: 5.x-1.0-alpha4 » 5.x-1.5
Component: Theming » Code
Status: Fixed » Active

Ok, I've explained what changes need to be made to _phptemplate_variables() based on what was pasted here as well as pointing out that the if test wrapping it has to return true. If it's still not working, I have no idea. I don't use this theme. Bumping you over to their queue and maybe they can help.

Michelle

zuzuzzzip’s picture

where do you put the _phptemplate_variables() function??

Just can't seem to find it :/

roopletheme’s picture

If you download the latest version of litejazz from this site, it already has the advanced forum support built into the theme.

roopletheme’s picture

Status: Active » Fixed

Version 1.7 (http://drupal.org/node/305936) has built-in support for the advanced forum and advanced profile modules.

Anonymous’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.