Gift Wrapping Contribution

[swguy]

We have been getting a lot of orders for gift wrap. We have to check the comments section to see if they want the b-day gift wrap or the holiday wrap. Sometimes it is hard to tell if they say just "Love Aunt Bo and Uncle Grit".

The "gift wrapping selection" code will provide a space for a note for each wrapped item.

[travellers]

I've obviously done something wrong somewhere along the line - finishing an order as a test customer results in

1054 Unknown column 'orders_id' in 'field list'
in:
[INSERT INTO zen_orders_giftwrap (orders_id, orders_products_id, wrap) VALUES ('4', '11', '1')]

Which leaves me a bit confused, since there's no 'orders_id' table in the sql I ran, only 'zen_orders_id'! I was pretty sure I edited the sql in the way the instructions said to - here's the modified orders_wrap.sql

DROP TABLE IF EXISTS zen_orders_giftwrap;
CREATE TABLE zen_orders_giftwrap(
  zen_orders_giftwrap_id int(11) NOT NULL auto_increment,
  zen_orders_products_id int(11) NOT NULL default '0', 
  zen_orders_id int(11) NOT NULL default '0',
  zen_wrap int(11) NOT NULL default '0',
  PRIMARY KEY  (zen_orders_giftwrap_id),
  KEY zen_idx_orders_id_orders_prod_id_giftwrap (zen_orders_id,zen_orders_products_id)
) TYPE=MyISAM;

All I did was add 'zen_' to each reference I could see to a filename, as my cart was auto-installed with Fantastico which automatically adds that prefix.
Anyone offer any advice? I can't just not use this mod until I work it out because in Order Total Modules it only offers 'Yes' to 'Do you want to offer Gift Wrap - there isn't a 'No' option!

[swguy]

Fantastico adds zen_ to the table name not the field names.

Rerun the orders_wrap.sql through your admin panel under
Admin->Tools->Install SQL Patches.

This takes care of the prefix stuff.

[swguy]

The Beanstream mod is done. Gift wrap fans will be happy to hear I'm prototyping how to offer wrapping paper choices with tasty AJAX goodness. Stay tuned.

[stanbridge]

Help! A customer has just placed an order and I believe the Gift Wrapping module is causing an error to be displayed AND the customer to be over charged.

Background info -
Zen Cart Version: 1.3.8a
Gift Wrap version: Version 2.6_138

Site and ordering, including confirmation screen, was working fine. Shop owner asked me to install GIft Wrapping module.
After installation, everything seemed to work fine. Only I never processed a REAL order (right through to ACTUAL payment) for testing.

NOW, a customer has just placed the following order...

ISSUE part 1

The order was processed with a price that is FAR too high!
(text below copied from order recorded in Admin > Customers > Orders)
Products: 1 x Maileg Vintage style Soldier ToyRattle
Model: [blank]
Tax: 10%
Price (ex): $31.82
Price (inc): $35.00
Total (ex): $31.82
Total (inc): $35.00

Sub-Total: $79.95
Flat Rate (within Australia) (2-3 days): $8.50
Gift Wrap: $4.95
GST (tax): $7.77
Total: $93.90

I checked the merchant facility at the bank's website and the order was processed at $93.90.

NOTE: Not sure if this is related to the gift-wrapping module, but it didn't happen before this module was installed.


Issue part 2

After discovering this problem, I placed an order myself for the item next to it (identical in every way, including stock, but different color). I am in

the same zone as the previous customer, and selected everything the same as she did, including gift wrapping.

Products: 1 x Maileg Vintage style Princess Toy Rattle
Model: [blank]
Tax: 10%
Price (ex): $31.82
Price (inc): $35.00
Total (ex): $31.82
Total (inc): $35.00

Sub-Total: $35.00
Flat Rate (within Australia) (2-3 days): $8.50
Gift Wrap: $4.95
GST (tax): $3.68
Total: $48.95

As you can see, my order calculated to the correct amount (I think, I'm terrible with maths and tax!)


IMPORTANT CLUE no.1

After I placed my order, I did not get the usual confirmation screen. Instead, I got an error message reading as follows:

1146 Table 'sqsq1343_zc1.zen_orders_giftwrap' doesn't exist
in:
[INSERT INTO zen_orders_giftwrap (orders_id, orders_products_id, wrap) VALUES ('7', '7', '1')]

IMPORTANT CLUE no.2

When I go into Admin > Customers > Orders, I see the following error at the top of the page:

1146 Table 'sqsq1343_zc1.zen_orders_giftwrap' doesn't exist
in:
[select orders_products_id from zen_orders_giftwrap where orders_id = '7' and wrap = 1]

If you were entering information, press the BACK button in your browser and re-check the information you had entered to be sure you left no blank fields.


NOTE: In both "important notes" above, the order id "7" is my order. The order id for the customer is "6".



Can anyone tell me what is going on?! I'm extremely panicked and have a store owner most upset!

Any help/solutions would be GREATLY appreciated!

Thanks in advance,


Stanbridge, The Worried

[stanbridge]

Hi All,

I think the calculation bug could be put down to the following post...

http://www.zen-cart.com/forum/showthread.php?t=83928

The errors I'm getting on this screen are most likely unrelated. Please feel free to correct me if you think I'm wrong.

I have looked in phpMyAdmin and can see that the table named "zen_orders_giftwrap" doesn't exist, however, "orders_giftwrap" does.

Can I simply rename this table to fix the problem? Or will I also have to manually key in records up to no 7 (the current highest)?

Also, I forgot to mention last time since receiving these errors both myself and the store owner (each with separate admin accounts) never received email notification of new order. I suspect the missing table error stopped this program from getting this far.

Again, any info is very much appreciated!

[swguy]

From the README file:

3. Only perform this step if you have never installed Gift Wrap 2.0 or
greater:
Run the sql script orders_wrap.sql in phpMyAdmin. If you have used
a prefix in your database, modify orders_wrap.sql to reflect this (e.g.
change "orders_giftwrap" to "zen_orders_giftwrap" if your prefix is
"zen_").

[stanbridge]

Yep, remembered seeing that in the readme file but didn't realise I had anything prefixed with "zen" - mainly because I had never consciously prefixed anything myself.

So now that two orders have come in, to resolve the issue, do I...

1) rename the table (and nothing else)?

2) rename the table AND update a value/s in the table manually?

3) Run the SQL script again?

Thanks for the speedy reply mate!

Cheers

Stanbridge

[swguy]

Clearly you should rename the table.

I would further recommend you set up and maintain a test site so you can stage mod installs before going live. This will allow you to debug the other problem(s) you're having without affecting the client's site. I would also recommend you start running small transactions on the live site after you make changes like this instead of just assuming you haven't broken something. You can always refund them after you're done.

[stanbridge]

Thanks SWGUY! Renamed the table (didn't touch any field values or anything) and all appears to be working brilliantly!

You're quite right about setting up a testing site. I should have known better. Also, I didn't even think about the ability to refund credit card amounts on test purchases. I'm new, can you tell?

I just tested and all works well. The unrelated tax issue appears to be resolved now too.

Once again mate, thanks very much for your help! You're a legend!

Cheers,

Stanbridge