I updated my ubercart but i checked if ideal would work with the ideal-similator.nl but my ideal stays pending iDEAL pending

Perhaps something with the simulator or the new ubercart?

CommentFileSizeAuthor
#23 workaround.patch907 bytespraseodym

Comments

janton’s picture

or maybe this is a problem with the simulator?

Does somebody else have the same problem?

janton’s picture

If i check the order i see:
Order comments:
Date User Notified Status Comment
12/07/2011
4:10:50 PM - Payment received Payment of €7.95 submitted through iDEAL.
Admin comments:
Date User Comment
12/07/2011
4:10:50 PM - iDEAL reported payment of €7.95 from 108429563 (PHP Solutions, Meppel).
12/07/2011
4:10:50 PM - Order created through website.

It says Payment received, but the order status stays iDEAL Pending.
Could this be because the action changes in the ubercart 6x - 2.7

Or is this again something strainge that only my machine has...

janton’s picture

i think problem is new action changes from ubercart?

Update order status on full payment

>If the order state is in checkout.
>If the order state is post checkout.

Or could this this not cause the problems?

praseodym’s picture

Yes, this is probably the result of changes in Ubercart. I haven't yet have time to investigate further, so you are welcome to send in a patch.

praseodym’s picture

Not sure what causes this error. The code 'flow' of this module is the same as the uc_paypal module, so unless it has the same problem it must be a problem in Ubercart code or configuration. In my installation, there is a CA 'workflow' that sets status to 'payment received' when the outstanding balance is zero. If this rule fires as expected, payment status should be updated.

janton’s picture

They changed the rules because of Paypal (WPS) payment that send IPN after transaction is complete. (that resulted in 2 times sending email checkout complete)

In my CA i see: (the new Ubercart Rules there is this new line called "Conditions group")
Update order status on full payment
Operator:
X AND. If all of these conditions are TRUE.

--Condition: Check the order balance
Title: If the balance is less than or equal to $0.00.
Balance is less than or equal to €0,00.

--Conditions group
Operator:
X OR. If any of these conditions are TRUE.

-Condition: Check the order state
Title: If the order state is in checkout.
Arguments: Order
Order state: In checkout

-Condition: Check the order state
Title: If the order state is post checkout.
Arguments: Order

(*So there is this new group: Conditions group that also need to retun true for one of the conditions.)
I'm not a well developer, but i guess the iDEAL payment doesn't have this order state?
OR there is something wrong with my CA because of the update or other causes...

praseodym’s picture

The module has its own order state, ideal_pending, just like the PayPal module. I don't see why the rule shouldn't fire in this case.

Another option would be setting the status to payment_received in the module code instead, but as I said earlier the PayPal module never does this either.

janton’s picture

praseodym, i will try to install everything on a fresh drupal install tomorrow. to check if this happends for everyone.
Or do you also have this problem with the new ubercart version?

eduarddrenth’s picture

We as well upgraded to ubercart 6.2.7 and also see "ideal pending" never being changed into "Voltooid" (which is dutch for completed)....

Good luck fixing this!

praseodym’s picture

I'm having the same problem with an installation that was upgraded from earlier versions of Ubercart and using the ING production gateway. It'd be really helpful if you could check if the same problem appears in a clean installation. If it does then I'll go for the 'hardcoded' option and just set it to 'Payment received'.

@eduarddrenth: it shouldn't actually go to 'Voltooid' unless the order consists only of non-shippable items. Otherwise, the right order is iDEAL Pending->Payment received->(Have someone ship the order and manually set to) Completed

eduarddrenth’s picture

Thanks for clarification,

My orders consist of only non-shippable items. Until the upgrade "Voltooid" would be the final status.

In our case this issue doesn't break functionality though

praseodym’s picture

Well, then it is clearly an issue with Ubercart's Conditional Actions (CA) system. Looks like there is no easy way to debug it, though (no logs or anything).

eduarddrenth’s picture

In cases like this I always fall back on xdebug from within netbeans, works like a charm

eduarddrenth’s picture

In my case maybe I can adapt this: admin/store/ca/uc_checkout_complete_paid/edit/conditions

Change bestelstatus (order status) from "betaling ontvangen" (payment received) into "ideal pending"

Or perhaps better change here admin/store/settings/orders/edit/workflow from "ideal pending" to "betaling ontvangen"

promes’s picture

We also have the same problem after upgrading from ubercart 6x-2.4 to 6x-2.7.

praseodym’s picture

Ah: since 'iDEAL Pending' is actually of type 'Payment received' the rule probably does not fire. I copied this behaviour from the PayPal module, so I'll check again why it does work there. If I'm certain I'll update the module to fix this.

janton’s picture

@praseodym:
did you already find a fix?

dkarso’s picture

I just notice this bug too and consequently the error report "iDEAL payment status request error: AP2600: 'Transaction does not exist - Field generating error: Transaction does not exist"

Does this module check for payment received on the iDeal server via a cron task?

janton’s picture

"iDEAL payment status request error: AP2600: 'Transaction does not exist - Field generating error: Transaction does not exist"
I don't have that error..

neorg’s picture

Just subscribing

praseodym’s picture

The Google Checkout module also uses the same code flow as this module, and depends on the Ubercart 'core' trigger to set status to 'Payment received'. I can't test if it works there, but since I couldn't find any bugs I think it does. So, no clue why it won't work for us.

Anyway, I've implemented the workaround on a live site and will have to see if it works there.

janton’s picture

What work around are you revering to?
I hope you can fix it, because it would be nice to see all payments go to complete.. because some other modules in my small webshop depend on it.

praseodym’s picture

Status: Active » Needs review
StatusFileSize
new907 bytes

Give the attached patch a try and let me know if it works.

janton’s picture

ok i try this patch on a live site.. i will tell how it goes!

janton’s picture

My iDEAL payment now say:
PayPal pending

that doesn't seem right?

janton’s picture

i know there is a panel somewhere that you can say the order from payments.. like pending or completed etc. I'm already looking for a few minutes but i really can't find this.

> Found it:
/admin/store/settings/orders/edit/workflow
But well.. it doesn't make any difference because all look correct there, i will include a picture!

janton’s picture

Check here my settings, perhaps something is wrong or different then yours?

http://imageshack.us/photo/my-images/837/1432012221255.png/

*i now changed Payment received Default order status to Payment received
... will check how that goes ;-)

praseodym’s picture

It should indeed be 'Payment received', not sure why it was set to 'PayPal pending'. Changing it should solve the issue.

janton’s picture

Yes i can say the "workaround" works well!

janton’s picture

Ok after 3 weeks, this patch is still working perfect!

praseodym’s picture

Good to hear, I'll apply the patch and release a new version soon.

artbussy’s picture

Has the new version been released? I still have this problem (should learn how to patch, I guess).

praseodym’s picture

No, it is not yet in any release. However, the patch is so simple that you could actually apply it manually by just adding this one line in the correct place.

praseodym’s picture

Version: 6.x-2.3 » 6.x-2.4
Assigned: Unassigned » praseodym
Status: Needs review » Fixed

Fixed in 6.x-2.4.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.