Why do I have to click a second button to view products under a manufacturer?

[betterlead]

What the heck am I doing wrong? Second button shows up again after upgrading the template (flexiblewebdesign's LightMart from 1.0 to 1.1)

The code in the file is still the same. Or is it not? Could someone dear help to check it for me?

Cheers
/Betterlead

Code Select Expand

<div class="vmmanufacturer<?php echo $params->get('moduleclass_sfx'); ?>">
<?php foreach ($manufacturers as $manufacturer) {
$link = JROUTE::_('index.php?option=com_virtuemart&view=category&virtuemart_manufacturer_id=' . $manufacturer->virtuemart_manufacturer_id);

?>
<div style="float:left;">
<a href="<?php echo $link; ?>">
<?php
if ($manufacturer->images && ($show == 'image' or $show == 'all' )) { ?>
<?php echo $manufacturer->images[0]->displayMediaThumb('',false);?>
<?php
}
if ($show == 'text' or $show == 'all' ) { ?>
<div><?php echo $manufacturer->mf_name; ?></div>
<?php
} ?>
</a>
</div>
<?php
if ($col == $manufacturers_per_row) {
echo "</div><div style='clear:both;'>";
$col= 1 ;
} else {
$col++;
}
} ?>
</div>
<br style='clear:both;' />

<?php
} else {
$last = count($manufacturers)-1;
?>

<ul class="vmmanufacturer<?php echo $params->get('moduleclass_sfx'); ?>">
<?php
foreach ($manufacturers as $manufacturer) {
$link = JROUTE::_('index.php?option=com_virtuemart&view=category&virtuemart_manufacturer_id=' . $manufacturer->virtuemart_manufacturer_id);
?>