Sometimes shorturls are used for campaign codes ?utm_source=google&utm_medium=cost-per-click but when passed through the shorturl_init() the query strings are dropped. Here's a preliminary patch.

CommentFileSizeAuthor
#1 shorturl.module-987304-1.patch1.47 KBdstol
shorturl.module.patch1.79 KBBrockBoland

Comments

dstol’s picture

dstol
Primary language English
commented
Priority: Normal » Major
Status: Active » Needs review
StatusFileSize
new shorturl.module-987304-1.patch1.47 KB

This one works a little better and removes the nested if foreach control structure

irakli’s picture

irakli commented

Thanks for the patch. It misses one use case, though: when original_url also has query parameters this code won't work.

Will fix that and commit.

Thanks

irakli’s picture

irakli commented
Status: Needs review » Fixed

Comitted: http://drupal.org/cvs?commit=461458

irakli’s picture

irakli commented

For the final fix, had to commit this one too: http://drupal.org/cvs?commit=461470

Status: Fixed » Closed (fixed)

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