1473 replies to this topic

[burt]

Quote

as you can see here
http://www.teksigns.com/shop/product_info....products_id=544

Thanks for that, now I can visualise what it looks like, I know it's not what I'm after.  Nice contribution though - well done to the contributor.

[thetrecker]

We got it working by not taking any notice of your updates, but changing the define table name additional_images to product_images

Also your SQL statement does not work!!!!!!

Otherwise, excellent script, just needs sorting out.......

[MasterBeta]

Excellent catch erisen!!!!!  

Now at least im getting a real error instead of a 404 so the page has been called up properly. The error in the popup window is as follows:

require('includes/application_top.php'); $navigation->remove_current_page(); $products_query = tep_db_query("SELECT images_description, popup_images FROM " . TABLE_ADDITIONAL_IMAGES . " WHERE additional_images_id = '" . $HTTP_GET_VARS['imagesID'] . "'"); $products_values = tep_db_fetch_array($products_query); ?>
Fatal error: Call to undefined function: tep_image() in /catalog/popup_add_image.php on line 37


Line 37 looks like this:

echo tep_image(DIR_WS_IMAGES . $products_values['popup_images'], $products_values['images_description'], POPUP_IMAGE_WIDTH, POPUP_IMAGE_HEIGHT); ?>

Any ideas?

Thanks,
Ash

[Birger]

I have the same problem  

[Parikesit]

There are some different configuration in OLD 2.2 CVS and the later CVS like MS1.

Example:
In the header of the popup_images.php CVS#20020620 we find:

  

  require('includes/application_top.php');

  

But in the MS1 there is a line "$navigation->remove_current_page();"
  

  require('includes/application_top.php');



  $navigation->remove_current_page();

  

For save configuration, take a look to popup_images.php from your CVS version. Than replace the some changing lines and save as "popup_add_image.php"


Regards
Parikesit

[Parikesit]

Find the similliar lines in "popup_images.php" and replace. Than save as "popup_add_image.php":

  $products_query = tep_db_query("SELECT images_description, popup_images FROM " . TABLE_ADDITIONAL_IMAGES . " WHERE additional_images_id = '" . $HTTP_GET_VARS['imagesID'] . "'");

  $products_values = tep_db_fetch_array($products_query);

and

echo tep_image(DIR_WS_IMAGES . $products_values['popup_images'], $products_values['images_description'], POPUP_IMAGE_WIDTH, POPUP_IMAGE_HEIGHT);

Enjoy

[nomove]

Hello,

i had the same prob, but i found the answer very easily. The only
thing you have to do is to write a forgotten "<?" above the "require...."

After that it works fine!

Greets

nomove

[Birger]

Quote

Hello,

a forgotten "<?" above the "require...."


nomove

Thanks!
You are the bigest!!!!  

(forgoted "<?php", but only in /catalog/admin/popup_ad_image.php and /catalog/popup_ad_image.php If you changes also in /catalog/admin/popup_image.php and /catalog/popup_image.php this produces an error in 'primary' popup imageproduct).

[MasterBeta]

Thank you nomove!!!!
I can't believe i missed that!

Thanks to Parikesit for the great addition, and thank you to those that helped get it running!

Cheers
-Ash

[senate]

could someone please post instructions "start to finish" on installing this contrib please.  the best thing would be to post a "final version" instead of patches :idea:

[chfields]

I would like to know if this contrib will allow me to choose my thumbnail images for the new products box on the start page?

[BrandonTiesman]

Is there gonna be a package released of this mod as 1 package. The additional patches are ok, but the code is spread out. If you could make a new release, all working code in one download, that would be great. If there is already a package like this available, Im sorry, I must have missed it . Thank You

[Parikesit]

Quote

could someone please post instructions "start to finish" on installing this contrib please.  the best thing would be to post a "final version" instead of patches :idea:

Hi,  :?

The instruction is inside the package contribution.

You need to download the full package and the last patch. [1] Extract the full package to destination folder. [2] Ectract the patch to same directory; replace the same file!


Enjoy
Parikesit

[BrandonTiesman]

but why keep it all cut up.. its a great mod!

[BrandonTiesman]

[code]Fatal error: Cannot instantiate non-existent class: imagesbox in /home/elkivory/public_html/catalog/includes/modules/additional_images.php on line 49[code][/code]

[Parikesit]

Additional Images Module | UPDATE

I have added a new FULL package to the contribution.
Please download the latest one here: http://www.oscommerce.com/community/contributions,1032

New File:
I add example of popup image handling for both old CVS and MS1.


Regards
zaenal

[BrandonTiesman]

Quote

[code]Fatal error: Cannot instantiate non-existent class: imagesbox in /home/elkivory/public_html/catalog/includes/modules/additional_images.php on line 49[code][/code]

Admin seems to work fine, I ad an image and refresh product page and this error occurs. Ive went over this package 3 times. Not sure whats going on, need some help if anyone has the time.

[Parikesit]

Quote

Admin seems to work fine, I ad an image and refresh product page and this error occurs. Ive went over this package 3 times. Not sure whats going on, need some help if anyone has the time.

Please download the new version!

Extract the file. Than go to folder catalog>includes>classes> of this extracted package contribution. Here you will find a file boxes_changes.

Add this content to your current boxes.php.


Happy nice try!
zaenal

[sefu]

Im using Snapshop 2.2 from June 2002
Just following the instructions but i cant find the following javascript in the product_info.php

<script language="javascript"><!--

  document.write('<?php echo '<a href="javascript:popupWindow('' . tep_href_link(FILENAME_POPUP_IMAGE, 'pID=' . $product_info['products_id']) . '')">' . tep_image(DIR_WS_IMAGES . $product_info['products_image'], addslashes($product_info['products_name']), SMALL_IMAGE_WIDTH, SMALL_IMAGE_HEIGHT, 'hspace="5" vspace="5"') . '<br>' . TEXT_CLICK_TO_ENLARGE . '</a>'; ?>');

  //--></script>

  <noscript>

  <?php echo '<a href="' . tep_href_link(DIR_WS_IMAGES . $product_info['products_image']) . '">' . tep_image(DIR_WS_IMAGES . $product_info['products_image'], $product_info['products_name'], SMALL_IMAGE_WIDTH, SMALL_IMAGE_HEIGHT, 'hspace="5" vspace="5"') . '<br>' . TEXT_CLICK_TO_ENLARGE . '</a>'; ?>

  </noscript>

Is there anyway to get around this?

[Parikesit]

Please send the file to my email to add the script, because i don't have CVS# June 2002


Cheers,