Advanced Forum Comments not formatted.

Not sure why the comments would be formatted differently, but the only code anywhere in Tapestry related to comments is this definition on line 108 of style.css:

.comment {
	padding: 10px;
	margin: 10px 0;
}

I've successfully used this code to get Tapestry to support Advanced Profile, Advanced Forum, and Node Title:

<?php
function _phptemplate_variables($hook, $vars) {
  $vars = array();
  if (module_exists('advanced_profile')) {
    $vars = advanced_profile_addvars($hook, $vars);
  }
  if (module_exists('advanced_forum')) {
    $vars = advanced_forum_addvars($hook, $vars);
  }
  if ($hook == 'page') {
    if (module_exists('page_title')) {
      $vars['head_title'] = page_title_page_get_title();
    }
  }
  if (theme_get_setting('tapestry_themelogo')) {
    $vars['logo'] = base_path() . path_to_theme() . '/images/' . theme_get_setting('tapestry_style') . '/logo.png';
  }
  return $vars;
}
?>

If this doesn't work for you, I'd be happy to help figure out why.

BTW, I posted this same stuff on your thread in the Adv Forum issue queue.

I'm looking at this! Really! :)

It just took me a while to get a testbed set up. I now see the problem, but I'm not yet sure why it's happening. I'll updated as soon as I know more.

As I said in the other issue, the code looks just fine. Could something be overriding the template file it's using for forum posts?

Michelle

I've spent some time looking at this, and frankly I'm stumped. It appears that the forum listing template files are being called into use at the appropriate time, but that the post template files (advf-forum-post.tpl.php, etc.) are not being applied. I don't know of anything specific to the theme that would prevent all of the template files from being used, but I'm not so brash as to rule out the possibility that the theme is at fault. It's just that, so far, I haven't been able to track down the root cause. I'm not giving up, but I have limited time that I can dedicate to this issue in the immediate future.

I have a project going right now that will make extensive use of the Advanced Forum module in combination with a custom theme that uses the same base of code that LiteJazz 0.17 is based on. I expect that once we reach that stage of the project, I'll encounter this again and I'll need to get to the bottom of it. When that happens, I'll gladly share any solution that I come up with. In the mean time, if someone more knowledgeable than me about adv-forum has a specific idea about what to look at and where to look, I'm very interested.

I'll try downloading the theme and doing some debugging but I won't likely get to it this weekend.

Michelle

You're too swell, Michelle! I'd write a song, but it appears some lads from Liverpool beat me to the punch.

I've got a complete re-write of LiteJazz at the very end of beta testing... I plan to update the d.o project page with this version. I had actually planned to do so by now, but I'd sure like to get a fix for this in first, assuming one is necessary. The latest and greatest version of the theme is available here.

I'm happy to help out in any way I can. But an extra pair of eyes on this would be greatly appreciated.

Thanks,
Bill

So I think I just needed a clean slate and day away from this. I took my latest LiteJazz code base (which included some new features), added the adv forum code and files to a clean copy of the theme, and installed it on the demo site. Everything worked perfectly. Go figure.

You can see the advanced-forum comment theming on this page. Check out the rest of the forum from this page.

I've updated the 5.x theme package available on the download page of the demo site. The 5.x-1.7b3 version is what is running on the site, along with Advanced Forum, Block Theme, and ThemeSettingsAPI. I'll clean it up for the d.o project page once I get some kind of confirmation.

The template.php file in this version of the theme needs no further modification in order to support Advanced Forum. Additionally, the Advanced Forum template files are already included in the theme. It should be plug-and-play. This seems to be the most full-proof method of ensuring the theme supports the module.

Nice, thanks. Sorry I didn't get to looking at this. Can't seem to get back to the modules lately. Too much clamoring for my attention.

Michelle

Version 1.7 (http://drupal.org/node/305936) is pre-configured for use with advanced forum and advanced profile.