Quick question about messagStack

[jrhame]

I have this in the tpl_header file.

<?php
  // Display all header alerts via messageStack:
  if ($messageStack->size('header') > 0) {
    echo $messageStack->output('header');
  }
  if (isset($_GET['error_message']) && zen_not_null($_GET['error_message'])) {
  echo htmlspecialchars(urldecode($_GET['error_message']));
  }
  if (isset($_GET['info_message']) && zen_not_null($_GET['info_message'])) {
   echo htmlspecialchars($_GET['info_message']);
} else {

}
?>

The only time I notice this is getting called is when I successfully tell a friend about a product.

All the other errors/warnings/success's all display in the center column where I want them to. This one when called however, is pushing the whole layout down, thus making the site look like poo. I moved this code into the tpl_product_info_display.php inside the center column under my heading, where I want it.

It's working. But was I just wondering if I am going to run into problems later. I can't seem to find any though.

[DrByte]

If you want to move it, I might suggest that the better place to relocate it would be to your custom tpl_main_page.php file, next to the other messageStack output for 'upload'.
That will force all those messages to appear in the center area, which may or may not be appropriate for errors. It's up to you -- it's your site.

[jrhame]

If you want to move it, I might suggest that the better place to relocate it would be to your custom tpl_main_page.php file, next to the other messageStack output for 'upload'.
That will force all those messages to appear in the center area, which may or may not be appropriate for errors. It's up to you -- it's your site.

I'll try that, thanks

edit: It works. It appears over the title heading, but thats ok. I can live with that. I'm just not sure where else I might need that in the main page. And only having it in the product info page made me leary.

Thanks

Josh