NOTE:: Before reporting an issue, make sure you are running the latest version, currently 3.3.1

View Issue Details Jump to Notes ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0004406phplist applicationStatisticspublic14-10-05 19:4418-02-08 14:16
PlatformOSOS Version
Product Version2.10.1 
Target Version2.10.7Fixed in Version2.10.3 
Summary0004406: Click tracking breaks mixed case URLs
Description;-1;-1;-1&sku=401395 [^]

by producing this for the redirect:;-1;-1;-1&sku=401395 [^]

Why is the text being forced to lowercase? It shouldn't be modified at all.
And is there a quick patch for this? I'm hunting around in files, but only just figured out the issue.
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]
related to 0003721closed phplist 2.10.x 
has duplicate 0006001resolvedmichiel Text Mails changing all HTML links to lowercase 
has duplicate 0006515resolvedmichiel clicktracking error with uppercase url 
related to 0007162resolvedmichiel URLs forced to lower case in text emails 

-  Notes
michiel (manager)
14-10-05 19:54

ah, that's a good point, never thought about case in URLs.

in sendemaillib.php

look for (several times)
$link = strtolower(cleanUrl($links[1][$i]));

and take out the strtolower
su (reporter)
14-10-05 20:11

Thanks, Michiel. Hopefully, I manged to fix this before the client noticed *grin*
mchristo2 (reporter)
07-04-06 00:11

I removed the strtolower() from the sendemaillib.php code as instructed, but the hyperlinks placed on text still change to lower case. There are several other uses of strtolower() in sendemaillib.php; should I remove them, too? As in:

if (strtolower(trim($linktext)) == strtolower(trim($linkurl)) ||
      'http://'.strtolower [^](trim($linktext)) == strtolower(trim($linkurl))) {
        $linkreplace = $linkurl;
    } else {
      $linkreplace = $linktext.' <'.strtolower($linkurl).'>';

Copyright © 2000 - 2017 MantisBT Team
Powered by Mantis Bugtracker