The drupal_alter() function can only take 3 arguments, but the last one can be an array of additional contexts. See the API docs at http://api.drupal.org/api/drupal/includes!module.inc/function/drupal_alt...

This means currently it's not possible to access $order in hook_commerce_google_analytics_item_alter().

Patch to fix this attached.

CommentFileSizeAuthor
#2 commerce_google_analytics_drupal_alter.patch790 bytesstella
commerce_google_analytics_drupal_alter.patch966 bytesstella
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Cyberschorsch’s picture

Cyberschorsch
Primary language German
CreditAttribution: Cyberschorsch commented
Status: Needs review » Needs work

This patch is not working since the paths are not correct. The path needs to be set to the module path instead of the drupal root path.

stella’s picture

stella CreditAttribution: stella commented
Status: Needs work » Needs review
FileSize
commerce_google_analytics_drupal_alter.patch790 bytes

Updated patch attached.

Cyberschorsch’s picture

Cyberschorsch
Primary language German
CreditAttribution: Cyberschorsch commented
Status: Needs review » Reviewed & tested by the community

Works good. Marking as RTBC.

Cyberschorsch’s picture

Cyberschorsch
Primary language German
CreditAttribution: Cyberschorsch commented
Status: Reviewed & tested by the community » Closed (fixed)