phplist

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
0008092phplist applicationClick Trackingpublic30-09-06 16:0718-02-08 14:08
Reportervide 
PrioritynormalSeverityminorReproducibilityalways
StatusresolvedResolutionfixed 
PlatformOSOS Version
Product Version2.10.2 
Target Version2.10.7Fixed in Version2.10.3 
Summary0008092: Single quote hrefs not translated into tracked urls
DescriptionI'm using PHPList 2.10.2 and I've found that activating click tracking on links, links that are like href='mysite' doesn't get translated. You have to have double quotes as in href="mysite" to get them translated.
 
 I've found the "culpable" regexp in admin/sendemaillib.php line 361
 
 preg_match_all('/<a>]*)>(.*)</a>/Umis',$htmlmessage,$links);
 
 and I've changed it to:
 
 preg_match_all('/<a(.*)href=["\'](.*)["\']([^>]*)>(.*)</a>/Umis',$htmlmessage,$links);
 
 to match both single (') and double (") quotes.
TagsNo tags attached.
Attached Files

- Relationships
related to 0003721closed phplist 2.10.x 

-  Notes
(0019439)
vide (reporter)
02-10-06 10:56

There is and identical bug in the stripHTML() function while converting the links in text mode:

(old)
#$text = preg_replace("/<a href=\"(.*?)\"[^>]*>(.*?)<\/a>/is","[URLTEXT]\\2[/URLTEXT][LINK]\\1[/LINK]",$text,100);

(new)
  $text = preg_replace("/<a href=[\"\'](.*?)[\"\'][^>]*>(.*?)<\/a>/is","[URLTEXT]\\2[/URLTEXT][LINK]\\1[/LINK]",$text,100);

always in sendemaillib.php line 818


Copyright © 2000 - 2017 MantisBT Team
Powered by Mantis Bugtracker