Hi,

When going to the td_redirect callback without URL parameters, there are couple PHP notices:

Notice: Undefined index: tduid in commerce_tradedoubler_callback() (line 54 of /var/www/builds/build_2014-02-18-134533/sites/all/modules/contrib/commerce_tradedoubler/commerce_tradedoubler.module).
Notice: Undefined variable: url in commerce_tradedoubler_callback() (line 62 of /var/www/builds/build_2014-02-18-134533/sites/all/modules/contrib/commerce_tradedoubler/commerce_tradedoubler.module).

Patch follows.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

iSampo’s picture

Status: Active » Needs review
FileSize
1.99 KB

Here's patch for the undefined index errors. Also fixed a typo on the watchdog message and reformatted the watchdog message so the URL parameters are printed separately instead of after each other without spaces.

E: It seems my editor also stripped some trailing white spaces there. Oh well. :)

marcofernandes’s picture

Status: Needs review » Closed (fixed)

Patched and on 7.x-1.2 release.

Thank you for your contribution!