Gift Wrapping Contribution

Ah, OK, thanks for setting me straight!

Just to report back that copy/pasting the diffs from this module when compared to 1.5.5a code, into 1.5.6b zencart files works well.
Obviously, copying the core files as they are is a no-no, and I imagine most people have already incorporated some changes, so merging is normal activity.

For the custom templates, I note that tpl_checkout_confirmation_default.php exists in the classic_responsive template (which I use), the other two files need to be copied over template_default template.

The admin/includes/classes/order.php file no longer exists, as we know, so that work simply falls away (store front class order.php does the job).

I did not try to update the HTML code to the standard of 1.5.6b, no doubt some improvement in column alignments or some such might happen, I might have time to tweak the code over the weekend, but this is just to give a heads up that the module absolutely still does work.

Thanks swguy!

I'll try to get the update to Gift Wrap done this week. I have fallen off the wagon with my 156 updates!

Thanks! Any changes I am happy to test them.
I also note that in my demo cart, I found that there are plenty of gift-related attributes, including wrapping, set up for some products.
In that case, there is actually the possibility of choosing wrapping at product selection from the attributes (with their own pricing), and wrapping from this module.
Not sure how to resolve this, maybe just disable the gift wrapping attribute on products if one is intending to use this module....

Gift Wrap has been updated for Zen Cart 1.5.6.

Thanks a stack for the cosmetic changes needed (and one logic check). I could revise my initial copy/paste with those extra changes, looks a bit nicer indeed. Thank you once again!

Hi SW

New Install on 1.5.6c, with Wesminster New, and I can't seem to get this little beauty to work... I've uploaded everything in the right places, have installed the sql patch (got the green light saying it was successful), however it's not showing up in the Orders Total box.

I have some Dbug lines, from before I installed it which are gobbledygook to me.... however I am wondering if they are interferring with this add-on...?

Request URI: /index.php?main_page=product_info&cPath=1_17_66&products_id=182, IP address: 125.236.139.99
#1 require(/home/sentialz/public_html/includes/templates/westminster_new/templates/tpl_modules_main_product_image.php) called at [/home/sentialz/public_html/includes/templates/westminster_new/templates/tpl_product_info_display.php:60]
#2 require(/home/sentialz/public_html/includes/templates/westminster_new/templates/tpl_product_info_display.php) called at [/home/sentialz/public_html/includes/modules/pages/product_info/main_template_vars.php:177]
#3 require(/home/sentialz/public_html/includes/modules/pages/product_info/main_template_vars.php) called at [/home/sentialz/public_html/includes/templates/westminster_new/common/tpl_main_page.php:216]
#4 require(/home/sentialz/public_html/includes/templates/westminster_new/common/tpl_main_page.php) called at [/home/sentialz/public_html/index.php:97]
--> PHP Warning: Use of undefined constant LARGE_IMAGE_WIDTH - assumed 'LARGE_IMAGE_WIDTH' (this will throw an Error in a future version of PHP) in /home/sentialz/public_html/includes/templates/westminster_new/templates/tpl_modules_main_product_image.php on line 25.

There's about 8 sets of this, all very similar except the image width description changes.

Hopefully you might be able to shed some light on it for me....

Tania :-)

Large Image PHP notices fix:
https://www.zen-cart.com/showthread.php?227325-Fixing-LARGE_IMAGE_WIDTH-and-LARGE_IMAGE_HEIGHT-PHP-notices&p=1374072#post1374072

Thanks Scott, picked that up and have run the sql... fingers crossed that stops half of my debug logs :-)