Jump to content
  • Checkout
  • Login
  • Get in touch

osCommerce

The e-commerce.

bugs regarding Bestseller v1.4 contribution


Recommended Posts

i must first say that this contribution is good, useful and easy to install. however, i would like to point out some errors in the contribution.

 

1. there is a typo mistake in the following statement in the readme.

 

Step 5:

Add the following line to /catalog/admin/includes/boxes/catalog.php, right before the line that contains <a href="'. tep_href_link(FILENAME_PRODUCTS_EXPECTED, '', 'NONSSL') . '" class="menuBoxContentLink">' . BOX_CATALOG_PRODUCTS_EXPECTED . '</a>');

ADD:

'<a href="' . tep_href_link(FILENAME_BESTSELLERS2, '', 'NONSSL') . '" class="menuBoxContentLink">' . 'Best Sellers' . '</a><br>' .

 

2. in order for the scrolling pix to be fitting in the bestseller box, the following line must be added to catalog/includes/boxes/bestseller2.php.

 

add: SMALL_IMAGE_WIDTH, SMALL_IMAGE_HEIGHT

 

just after

 

$rows = 0;

$info_box_contents = array();

while ($best_sellers = tep_db_fetch_array($bestseller_products_query)) {

$rows++;

$info_box_contents[] = array('align' => 'center',

'text' => '<a href="' . tep_href_link(FILENAME_PRODUCT_INFO, 'products_id=' . $best_sellers["products_id"], 'NONSSL') . '">' . tep_image(DIR_WS_IMAGES . $best_sellers['products_image'], $best_sellers['products_name'],

 

 

that's all that i would like to add to this contribution.

Link to comment
Share on other sites

  • 3 weeks later...

Perfect! Thanks a lot for your post mate! ;)

Link to comment
Share on other sites

  • 3 months later...

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
×
  • Create New...