Running this module on on PHP 5.3 generates a deprecated function error. The split function should be replaced with explode().

Deprecated function: Function split() is deprecated in GA_Parse->ParseCookies() (line 80 of ***/ga_tokenizer/class.gaparse.php).

CommentFileSizeAuthor
#1 split-deprecated.1198504.patch658 bytesnrambeck
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

nrambeck’s picture

nrambeck CreditAttribution: nrambeck commented
FileSize
split-deprecated.1198504.patch658 bytes

See attached patch for resolution.

MaxWesten’s picture

MaxWesten CreditAttribution: MaxWesten commented
Status: Active » Reviewed & tested by the community

#1 Works like a charm Thx.

pkiraly’s picture

pkiraly CreditAttribution: pkiraly commented

It works for the 6.x-1.2 version as well. Thanks!

bancarddata’s picture

bancarddata CreditAttribution: bancarddata commented

Also works great for me - Drupal 7.12, ga_tokenizer 7.x-1.3. Recommend committing. Thanks all!

fenstrat’s picture

fenstrat
he/him
Primary language English
Location Australia
CreditAttribution: fenstrat commented

I can also confirm this works on ga_tokenizer-7.x-1.3

estepix’s picture

estepix CreditAttribution: estepix commented

Is this already included in the latest release or dev version?
Thanks

fenstrat’s picture

fenstrat
he/him
Primary language English
Location Australia
CreditAttribution: fenstrat commented

@estepix no, this is marked as RTBC so the patch in #1 works but it still hasn't been committed. See http://drupal.org/patch/apply on how to apply it.

pdolphin’s picture

pdolphin CreditAttribution: pdolphin commented

Patch worked for me with ga_tokenizer-7.x-1.3. Thanks!!

valery86’s picture

valery86 CreditAttribution: valery86 commented

It worked for me too.

Thanks a lot

Brian294’s picture

Brian294 CreditAttribution: Brian294 commented
Status: Reviewed & tested by the community » Closed (fixed)

Thank you. Committed to 7.x branch. Will create a 7.4 version after the next git repo refresh.

Peace,
Brian