edfaught 11 Posted January 13 (edited) 30 minutes ago, edfaught said: Does the hotfix also take care of this? Answered: Yes it does. However now I see this: Warning : Use of undefined constant MODULE_PI_GALLERY_KISSIT_SMALL_IMAGE_WIDTH - assumed 'MODULE_PI_GALLERY_KISSIT_SMALL_IMAGE_WIDTH' (this will throw an Error in a future version of PHP) in includes/modules/pi/product_info/pi_gallery_kissit.php on line 84 Because previously and by mistake I didn't have the module installed. Edited January 13 by edfaught Share this post Link to post Share on other sites
♥raiwa 1,419 Posted January 13 (edited) 14 minutes ago, edfaught said: Answered: Yes it does. However now I see this: Warning : Use of undefined constant MODULE_PI_GALLERY_KISSIT_SMALL_IMAGE_WIDTH - assumed 'MODULE_PI_GALLERY_KISSIT_SMALL_IMAGE_WIDTH' (this will throw an Error in a future version of PHP) in includes/modules/pi/product_info/pi_gallery_kissit.php on line 84 Because previously and by mistake I didn't have the module installed. Seems I missed to add these constants during the last update to abstract module. Here the fixed module (needs uninstall/reinstall): pi_gallery_kissit.php Quote (Also, the support link in the download documentation is outdated.) Thanks, I'll also update it. Please confirm and I'll upload the update with these fixes tomorrow. Edited January 13 by raiwa About Me: http://forums.oscommerce.com/user/249059-raiwa/ Need help? How To Get The Help You Need Is your version of osC up to date? You'll find the latest osC community version CE Phoenix here. Public Phoenix Change Log Cheat Set on Google Sheets Share this post Link to post Share on other sites
♥raiwa 1,419 Posted January 14 (edited) On 1/12/2021 at 3:27 PM, Denzel said: Found the list messed up again after searching a product, which is linked to a 2nd folder. Decided to add classes to the <th> tags, so we can exclude the choosen classes: $('table tbody td:has([href*="pID={$products_id}&"])').parent().find('th').not('.pmodel').not('.pname').addClass('pname').before('<th class="pmodel">{$products_model}</th>'); this works for me even with three links 😉 Hi Dennis, Your solution includes other customizations like the product model and I believe you posted the wrong line which adds {$products_model}. It should be {$products_thumb}. Please post generic solutions for a unmodified installation, not custom solutions. What works for me is this: $('table tbody td:has([href*="pID={$products_id}&"])').parent().find('th').not('.pname').addClass('pname').before('<th class="pname">{$products_thumb}</th>'); Thank you 😉 Edited January 14 by raiwa About Me: http://forums.oscommerce.com/user/249059-raiwa/ Need help? How To Get The Help You Need Is your version of osC up to date? You'll find the latest osC community version CE Phoenix here. Public Phoenix Change Log Cheat Set on Google Sheets Share this post Link to post Share on other sites
♥raiwa 1,419 Posted January 14 Uploaded update with the above fixes for Phoenix 1.0.7.13: KissIT image thumbnailer 3.5.0r1 Requires Phoenix 1.0.7.4+ Tested with Phoenix 1.0.7.13 Change log 3.5.0r1: - updated gallery modules for Phoenix 1.0.7.13. - fixed error in admin thumbs category hook. Thanks to @Denzel - updated pi gallery module to abstract_module Change log 3.5.0: - moved and cleaned up thumbs subdirectory creation and removed unnecessary .htaccess. Thanks to @vmn - fixed error in admin thumbs showing duplicated images in product and category listing. Thanks to @Denzel - added support for webp image format - updated pi gallery module to abstract_module - Added optional convert jpeg, png and gif images to webp thumbs (Pro Version) About Me: http://forums.oscommerce.com/user/249059-raiwa/ Need help? How To Get The Help You Need Is your version of osC up to date? You'll find the latest osC community version CE Phoenix here. Public Phoenix Change Log Cheat Set on Google Sheets Share this post Link to post Share on other sites
Denzel 113 Posted January 14 vor 3 Stunden schrieb raiwa: I believe you posted the wrong line That's true, I'm sorry 🙂 Share this post Link to post Share on other sites
cupidare 21 Posted Monday at 07:35 AM Running with 1.0.7.13: just a question: is it intended, that the main-image is not thumbnailed? Because neither an image width or height is specified in pi_gallery_kissit.php Line 68. By adding some numbers there, the main image is cached as well. Share this post Link to post Share on other sites
♥raiwa 1,419 Posted Monday at 08:17 AM 38 minutes ago, cupidare said: Running with 1.0.7.13: just a question: is it intended, that the main-image is not thumbnailed? Because neither an image width or height is specified in pi_gallery_kissit.php Line 68. By adding some numbers there, the main image is cached as well. Yes, it is intended. It's supposed you wish to show the full original image size in the pop up. Only if watermark is used, KissIt will produce a new "thumb", but also in the original image size. There is no "cache" effect, the original image is there and does not need to be "cached" in the thumbs folder. 1 cupidare reacted to this About Me: http://forums.oscommerce.com/user/249059-raiwa/ Need help? How To Get The Help You Need Is your version of osC up to date? You'll find the latest osC community version CE Phoenix here. Public Phoenix Change Log Cheat Set on Google Sheets Share this post Link to post Share on other sites
cupidare 21 Posted Monday at 10:43 AM (edited) Thank you for the answer. I personally store images in full resulution (>4mb) because maybe I will need it in the future ... but for the pi-site it is sufficient to show a much smaller image. So am I right putting the "sufficient"-file-size in this position? Edited Monday at 10:44 AM by cupidare Share this post Link to post Share on other sites
♥raiwa 1,419 Posted Monday at 11:32 AM 48 minutes ago, cupidare said: Thank you for the answer. I personally store images in full resulution (>4mb) because maybe I will need it in the future ... but for the pi-site it is sufficient to show a much smaller image. So am I right putting the "sufficient"-file-size in this position? Yes, if it's ok for your needs. 1 cupidare reacted to this About Me: http://forums.oscommerce.com/user/249059-raiwa/ Need help? How To Get The Help You Need Is your version of osC up to date? You'll find the latest osC community version CE Phoenix here. Public Phoenix Change Log Cheat Set on Google Sheets Share this post Link to post Share on other sites