Manufacturers (v1.5.8a)

[earp]

Sorry if I'm missing something, but when I try to create manufacturers, there is no provision or ability to do so on the Admin - Catalog - Manufacturers page. Same issue on two separate clean installations of version 1.5.8a with only the Clone Template mod installed. Thanks.

[simon1066]

Sorry if I'm missing something, but when I try to create manufacturers, there is no provision or ability to do so on the Admin - Catalog - Manufacturers page. Same issue on two separate clean installations of version 1.5.8a with only the Clone Template mod installed. Thanks.

I think you're correct, I get the same on ZC v1.5.8a but not on ZC v1.5.8,

Until the issue is resolved you can navigate to the Insert Manufacturers screen with this URL:

YOUR_ADMIN/index.php?cmd=manufacturers&mID=&action=new

Edit: It might be to do with having no existing manufacturers in ZC v1.5.8a (I don't remember deleting any) as once one manufacturer has been added the insert button is then successfully displayed.

[swguy]

Confirmed as a bug. Please reply to this thread if you'll take it.

[earp]

I'm not programmer enough to fix the bug, but Compare It! says the problem is on line 329 in admin/manufacturers.php. I replaced that file with the one from 1.5.8 and all is good.

[simon1066]

I'm not programmer enough to fix the bug, but Compare It! says the problem is on line 329 in admin/manufacturers.php. I replaced that file with the one from 1.5.8 and all is good.

I think doing that, while it works, will produce a PHP Warning in the logs folder on PHP 8 (Undefined variable $mInfo) - which is possibly why the change to line 329 was made. I've had a look at the code but am also not knowledgeable enough.

[swguy]

You can do it, Simon!

[OJ_SIMON]

The error free fix is:
go to line 329 in admin/manufacturers.php
Find:

      <?php if (empty($action) && !empty($mInfo)) { ?>
        <div class="col-sm-12 text-right">
          <a href="<?php echo zen_href_link(FILENAME_MANUFACTURERS, ($currentPage != 0 ? 'page=' . $currentPage . '&' : '') . 'mID=' . $mInfo->manufacturers_id . '&action=new'); ?>" class="btn btn-primary" role="button"><?php echo IMAGE_INSERT; ?></a>
        </div>
      <?php } ?>

Replace the above with:

      <?php if (empty($action) && !empty($mInfo)) { ?>
        <div class="col-sm-12 text-right">
    <a href="<?php echo zen_href_link(FILENAME_MANUFACTURERS, ($currentPage != 0 ? 'page=' . $currentPage . '&' : '') . 'mID=' . $mInfo->manufacturers_id . '&action=new'); ?>" class="btn btn-primary" role="button"><?php echo IMAGE_INSERT; ?></a>
        </div>
      <?php }else{ ?>
      <div class="col-sm-12 text-right">
      <a href="<?php echo zen_href_link(FILENAME_MANUFACTURERS, ($currentPage != 0 ? 'page=' . $currentPage . '&' : '') . 'mID=&action=new'); ?>" class="btn btn-primary" role="button"><?php echo IMAGE_INSERT; ?></a>
        </div>
      <?php } ?>

Hope this helps.