MGSteve

I've got a few contributions to make for this contribution! - How do I submit them. I've added routines in to delete the images from the server if the images are removed from the product - but only if they are not used in other products. Also fixed one omission - when products are deleted, the additional images are not (from the DB or file). So, the contribution would: Delete additional photos if the additional photos were deleted against a product Delete additional photos if the product was deleted Delete default photo if the default photo was changed (currently doesn't delete it) All of these will not delete an image that is used on other products as either the default image or additional images. About an hour or twos work there :D Where'd I send it then?

Just to add, in reference to point 1) it was because in popup_image.php the var in the line you added was called $products_values not $products. Changed it to $products and it works fine :D (Running MS2 by the way)

Nice mod, just thought I'd bring up one or two issues... 1) Popup image for the default image (top right of product info page) no longer works, just get a white page - I'm looking into this though. 2) Typo in the distribution ZIP file for the contribution. In admin\categories_changes file, at the bottom of the file... ? ? for ($i=0; $i<[B]si[/B]($additional_images_id); $i++) { ? ? ? ? ? ? tep_db_query("[B]te[/B] from " . TABLE_ADDITIONAL_IMAGES . " where additional_images_id = '" . tep_db_input($additional_images_id[$i]) . "'"); ? ? ? ? ? } ? ? ? ? ? ? ? ? ? ? ? } ? ? ? ? ? ? ? ? ? ? ? ? //tep_db_query("[B]de[/B] from " . TABLE_ADDITIONAL_IMAGES . " where additional_images_id = '" . $HTTP_POST_VARS['additional_images_id'] . "'"); ? ? ? ? tep_redirect([B]tep_hlink[/B](FILENAME_CATEGORIES, 'cPath=' . $cPath . '&pID=' . $products_id)); bold bits are invalid... a search / replace gone wrong perhaps? The code is correct in the Categories.php file though :D edit... ok, the bolding didn't work, but the bits between the bold brackets are the bits that are wrong!