Gift Wrapping Contribution

[swguy]

I have heard before that Gift Wrap doesn't play well with FEC, which is not surprising; they're both large customizations. You might ask the FEC guy if he has time to debug this.

[Picci]

Can someone provide a link to a store with Gift Wrap 2.3_138 module running so I can see how it is supposed to work?

[heavenlynights]

Hi swguy, great mod!

however I have a bit of a problem, probably user error (me) but not real sure what to look for. (not very good with code)

everything works on the customer side when placing an order, however when viewing history, it doesn't show the table of products gift wrapped within order view as indicated

"Account History Page displaying gift wrapping"

On the admin side, although I placed an order of 3 products and indicated by checkbox only two of the products be gift wrapped, when I go to view the order on the admin view order screen there is no table indicating the products which need to be gift wrapped. the only thing I see is a statement saying "no gift wrapping selected". (even though gift wrapping was selected).

I do use the mod, super orders and am pretty sure that I have added the gift wrap code to the super_orders.php file properly but maybe not??.

Do you have any idea's as to what may of happend.

Your help would be much appreciated.

TIA

[swguy]

Be sure it works in orders (not super orders). If so then recheck your merge with super orders.

[heavenlynights]

Thanks swguy

doesn't work with just orders.php either. have checked the merged code for super_orders and it appears to be as it should be. The only other thing I can think of is that I also use timezone offset mod.

below is a section of code that is also in the orders.php file which relates to the timezone offset mode. Please check to see if anything in this would conflict with the gift wrap code.

$db->Execute("insert into " . TABLE_ORDERS_STATUS_HISTORY . "
                      (orders_id, orders_status_id, date_added, customer_notified, comments)
                      values ('" . (int)$oID . "',
                      '" . zen_db_input($status) . "',
                      date_add(now(), INTERVAL " . TIME_ZONE_OFFSET . " HOUR),
                      '" . zen_db_input($customer_notified) . "',
                      '" . zen_db_input($comments)  . "')");
          $order_updated = true;
        }

This is the only bit of code that is different in the orders.php file

[swguy]

If you don't see it in admin then I suspect you've merged includes/classes/order.php incorrectly; this is where the records get inserted. If you get "no gift wrapping selected" then the query

select orders_products_id
from " . TABLE_ORDERS_GIFTWRAP . "
where orders_id = '" . zen_db_input($oID) . "' and wrap = 1");

returned zero records. Trace this back.

[heavenlynights]

If you don't see it in admin then I suspect you've merged includes/classes/order.php incorrectly; this is where the records get inserted. If you get "no gift wrapping selected" then the query

select orders_products_id
from " . TABLE_ORDERS_GIFTWRAP . "
where orders_id = '" . zen_db_input($oID) . "' and wrap = 1");

returned zero records. Trace this back.

Bingo!! It turns out that the timezone mod had also overwritten includes/classes/order.php. I merged the giftwrap code into the file and wha'la, it now works.

Thanks swguy for your speedy and helpful assistance!!