Hi All,
we have put on simplenews a external link with more then one GET Parameter (also i.e. google.ch/?test=hallo&test2=test).

on your function _simplenews_statistics_decode (line 264) you use the php function parse_str($pars, $stat);
they split the URL string to much...

i have do a little workaround (not nice) but they works so...
$arr_check = explode('&', $pars);
if(count($arr_check) > 3){
for($x=3; $x < count($arr_check); $x++){
$replace = str_replace('amp;', '###-###', $arr_check[$x]);
$str .= $replace;
unset($arr_check[$x]);
}
$arr_check[2] = $arr_check[2].$str;

$pars = implode("&", $arr_check);
}
parse_str($pars, $stat);

CommentFileSizeAuthor
#1 simplenews_statistics.patch1.36 KByingtho

Comments

yingtho’s picture

StatusFileSize
new1.36 KB

The patch above didn't work properly. I have changed the code to extract url separately, see enclosed patch.

yingtho’s picture

Status: Active » Needs review
yingtho’s picture

Assigned: cola » Unassigned
-Mania-’s picture

I'm having problems with URLs as well however the patch #1 did not work for me.

weseze’s picture

Status: Needs review » Postponed (maintainer needs more info)

Can you provide more details on how to reproduce this bug?

weseze’s picture

Status: Postponed (maintainer needs more info) » Closed (cannot reproduce)