Error when click on Review Order [#1108392]

Hi,

I have installed ubercart from ubercart 7.x-3.0-beta2.zip and running on drupal 7.0. I am able to add products to shopping cart. In checkout form, after filling all the information and click on "Review Order" button, getting the following error.

Notice: Undefined property: UcOrder::$payment_details in uc_payment_method_cod() (line 258 of C:\wamp\www\drupal-7.0\sites\all\modules\ubercart\payment\uc_payment_pack\uc_payment_pack.module).
Notice: Undefined property: UcOrder::$payment_details in uc_payment_method_cod() (line 259 of C:\wamp\www\drupal-7.0\sites\all\modules\ubercart\payment\uc_payment_pack\uc_payment_pack.module).
Notice: Undefined property: UcOrder::$payment_details in uc_payment_method_cod() (line 260 of C:\wamp\www\drupal-7.0\sites\all\modules\ubercart\payment\uc_payment_pack\uc_payment_pack.module).
PDOException: SQLSTATE[23000]: Integrity constraint violation: 1048 Column 'delivery_month' cannot be null: INSERT INTO {uc_payment_cod} (order_id, delivery_month, delivery_day, delivery_year) VALUES (:db_insert_placeholder_0, :db_insert_placeholder_1, :db_insert_placeholder_2, :db_insert_placeholder_3); Array ( [:db_insert_placeholder_0] => 30 [:db_insert_placeholder_1] => [:db_insert_placeholder_2] => [:db_insert_placeholder_3] => ) in uc_payment_method_cod() (line 262 of C:\wamp\www\drupal-7.0\sites\all\modules\ubercart\payment\uc_payment_pack\uc_payment_pack.module).

Please help me!!

Comment	File	Size	Author
#17	AJAX.gif	225.58 KB	MPeli
#17	AJAX-shipping-quote.gif	223.62 KB	MPeli
#17	review-order.gif	101.55 KB	MPeli
#4	1107402.patch	6.95 KB	TR
#4
#1	1107402.patch	6.95 KB	TR
#1
	ubercart_checkout_error.jpg	121.56 KB	vallabh

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

TR CreditAttribution: TR commented 30 March 2011 at 23:04

Status:

Active

» Needs review

File	Size
1107402.patch	6.95 KB

I think this patch will solve the issues with the COD payment method. Please try it and let me know if it works.

Comment #2

longwave

he/him

English

CreditAttribution: longwave commented 30 March 2011 at 23:11

@TR: there are several misspellings of "recieve" in the comments in that patch.

Comment #3

30 March 2011 at 23:21

Status:

Needs review

» Needs work

The last submitted patch, 1107402.patch, failed testing.

Comment #4

TR CreditAttribution: TR commented 30 March 2011 at 23:38

Status:

Needs review

» Needs work

File	Size
1107402.patch	6.95 KB

Whoops. Very embarrassing because I know better, then cut and paste replicated the error...

Here's the new patch. testbot is still having problems with fixed point releases, so ignore that.

Comment #5

TR CreditAttribution: TR commented 30 March 2011 at 23:38

Status:

Needs work

» Needs review

Comment #6

30 March 2011 at 23:51

The last submitted patch, 1107402.patch, failed testing.

Comment #7

TR CreditAttribution: TR commented 6 April 2011 at 18:19

Status:

Needs work

» Fixed

Committed to 7.x-3.x then backported and committed to 6.x-2.x.

Comment #8

vallabh CreditAttribution: vallabh commented 7 April 2011 at 19:17

Anybody till now was able to install ubercart in drupal 7.0?

Comment #9

TR CreditAttribution: TR commented 7 April 2011 at 20:01

@vallabh: I don't fully understand your question. The bug you reported only affects the COD payment method. COD is not used by very many people, which is why the bug was not noticed before. All the other payment methods work fine.

Comment #10

enrikito CreditAttribution: enrikito commented 21 April 2011 at 08:43

Version:

7.x-3.0-beta2

» 7.x-3.x-dev

7.x-3.x-dev 2011-Apr-19

Comment #11

TR CreditAttribution: TR commented 21 April 2011 at 14:24

Status:

Fixed

» Postponed (maintainer needs more info)

@enrikito: What errors are you seeing?

Comment #12

enrikito CreditAttribution: enrikito commented 22 April 2011 at 08:22

in COD payment method

Comment #13

TR CreditAttribution: TR commented 22 April 2011 at 08:31

Priority:

Critical

» Normal

@enrikito: The problems reported by the original poster have been fixed. If you think there is an additional problem you need to provide some details.

Comment #14

enrikito CreditAttribution: enrikito commented 22 April 2011 at 09:43

sorry...

If clic in COD payment method the error in this report occurs when click on "Review Order" button

Comment #15

bwynants CreditAttribution: bwynants commented 26 April 2011 at 20:25

I'm seeing an error too. Dev download from April 26:

Please notice 'Let customers enter a desired delivery date' is OFF in the settings

26-04-2011 22:21:34 drupal[59874] http://xxx|1303849294|php|127.0.0.1|http://xxx/cart/checkout|http://xxx/cart/checkout|1||PDOException: SQLSTATE[23000]: Integrity constraint violation: 1048 Column 'delivery_month' cannot be null: INSERT INTO {uc_payment_cod} (order_id, delivery_month, delivery_day, delivery_year) VALUES (:db_insert_placeholder_0, :db_insert_placeholder_1, :db_insert_placeholder_2, :db_insert_placeholder_3); Array
(
[:db_insert_placeholder_0] => 10
[:db_insert_placeholder_1] =>
[:db_insert_placeholder_2] =>
[:db_insert_placeholder_3] =>
)
in uc_payment_method_cod() (line 272 of /xxx/sites/all/modules/ubercart/payment/uc_payment_pack/uc_payment_pack.module).

Comment #16

michalmajzlik CreditAttribution: michalmajzlik commented 28 April 2011 at 08:05

When you check checkbox "Let customers enter a desired delivery date." for COD, error wont display. And if you dont want display date at checkout page, hide with css.

Not best, but temporary while update is not release, is it ok.

Comment #17

MPeli CreditAttribution: MPeli commented 21 May 2011 at 07:18

File	Size
review-order.gif	101.55 KB
AJAX-shipping-quote.gif	223.62 KB
AJAX.gif	225.58 KB

I am getting the same errors while using COD. It is not possible to finish an order. See the attached pictures.

Comment #18

ali_b CreditAttribution: ali_b commented 4 June 2011 at 16:11

Comment #19

TR CreditAttribution: TR commented 4 June 2011 at 18:03

Status:

Postponed (maintainer needs more info)

» Needs work

Hmm, I don't know what was going on with that patch - it's named after an entirely different issue and it looks like it's just a code cleanup/review done while I was examining the code to see where the bug was. I don't see anything in the patch that would have fixed the problem. I'll try again.