I am using this module as well but i needed to modify the code a lot.
I added and changed code and included code/suggestions in the issue queue(e.g. http://drupal.org/node/926772).
Added/updated:
-(Better) Support for status changes from ogone
-Free order support (when total price is zero or below)
-More languages added
-Added more meaningful watchdog entries
-readme file
-I had an issue with the PAY_ID. drupal_get_schema didn't find this column in the database. To solve this, i a added the lenth definition in the .install file
-Displaying more meaningful payment info in the order (payment method instead of type)

I included a patch on the Git branch and a zip file including all the patched files.

CommentFileSizeAuthor
#2 uc_ogone_patched.zip10.03 KBrobbertnl
#2 uc_ogone_update_v2.patch25.92 KBrobbertnl
uc_ogone_update.zip10.09 KBrobbertnl
uc_ogone_update.patch19.78 KBrobbertnl

Comments

robbertnl’s picture

robbertnl commented

Language detection is also working now

robbertnl’s picture

robbertnl commented
StatusFileSize
new uc_ogone_update_v2.patch25.92 KB
new uc_ogone_patched.zip10.03 KB

New version:
-fixed issues with production settings (Ogone uses untrimmed values in production, and allows trimmed values in test environment)
-added Ogone payment methods preselect in your store admin panel.

bartezz’s picture

bartezz commented

Hi RobertNL,

Thanx for publishing this. The maintainer of this module should abandon it so more active users like yourself could take over. Or maybe you could possibly release a complete new module?

As I've mentioned in other issues in this issue queue I've patched the living daylights out of uc_ogone. I'm using some patches by PieterDC. Have you included some or all of these as well?

The biggest issues I'm having currently is that when users close the ogone.com window before they are redirected to the shop the payment is not registered in ubercart therefor not triggering the corresponding triggers to email users, admins, export orders, etc etc. I have emailed PieterDC about this and he told me to check the settings in Ogone, which I believed to be correct but I'll have to quadrupal :) check.

Cheers

robbertnl’s picture

robbertnl commented

Hi Bartezz,

Thank you for your message.
I included some patches, not all I think. Mostly based on this conversation: http://drupal.org/node/926772
My changed version should address your problem. Read the attached changed readme file;
In Ogone set:
-Post payment URL: absolute path to http://www.yourdomain.xyz/cart/uc_ogone_update_status
(This path applies only for my changed version.)
Be sure that the url is accessible for Ogone and not blocked in your firewall. The IP addresses Ogone uses is listed in their documentation.

bartezz’s picture

bartezz commented

Thanx for the fast reply! Did you test this in a multilingual setup? http://www.yourdomain.xyz/cart/uc_ogone_update_status would be redirected to http://www.yourdomain.xyz/en/cart/uc_ogone_update_status for instance possibly losing return values? Problem is that the site I need this for is live an receiving many orders so I can't test :( If you don't know for certain I could hopefully setup a test environment...

Cheers

BTW. Don't you mean http://www.yourdomain.xyz/cart/ogone_return_ok instead of http://www.yourdomain.xyz/cart/uc_ogone_update_status (cart/uc_ogone_update_status is not a MENU_CALLBACK

robbertnl’s picture

robbertnl commented

I am using i18n, but disabled all other languages at the moment, so I didnt test that. But i dont expect problems. I did test the payment with other languages enabled, but didnt check if the postponed call from ogone to cart/uc_ogone_update_status fires. But again i dont see why this should be a problem.
The url shouldn't contain any language information. So i dont understand why you should have a language prefix in the URI? Its just a service url, which is called a few moments later. The acting user wont see that page. cart/uc_ogone_update_status is a MENU_CALLBACK, so what do you mean it isn't?

I would suggest to test it first. At least by other people (because i am the only one who tested it with my own setup).

bartezz’s picture

bartezz commented

*blush* I have had problems with iZARC zip extractor on Win7 before and should have checked. I tried overwriting a directory with your ZIP's contents but it failed I guess so I was looking at the old uc_ogone_payment_menu(), sorry!

Is there a particular reason you changed the path from the original anyway? I see you are not using cart/ogone_return_ok_callback anymore? And if I'm not mistaken cart/ogone_false has also been dropped.

I will see if I can get a test setup running ASAP this is too important to leave it at this! Too many issues with the original module. Would you be willing to make an official release of this module?

Cheers

robbertnl’s picture

robbertnl commented

I dropped the cart/ogone_return_ok_callback bacause i think the name of this function is confusing. Because the callback doesnt handle only OK returns anymore but I also included handling for some other state changes.

I can't make an official release as i am not the maintainer of this module. Besides i think my code needs some reviews.
Making a new release should be simple. Because i already included a patch on the latest dev version.

bartezz’s picture

bartezz commented

Hi Robert,

As I have tried both your and PieterDC's updated version I would like to ask/suggest a cooperation to get this module working correctly and fully functional. Would like to ask you to check http://drupal.org/node/1427078

Maybe if we start working on getting your changes and those by PieterDC in a complete version we can close all support/feature requests and point to a new and better contrib module!

Cheers

robbertnl’s picture

robbertnl commented

Fixed in the 6.x-2.x branch.
See http://drupal.org/node/1519734

robbertnl’s picture

robbertnl commented
Version: 6.x-1.x-dev » 6.x-2.x-dev
Status: Needs review » Closed (fixed)