$extra_params .= isset($HTTP_POST_VARS[$extra_param]) ? $extra_param.'='.$HTTP_POST_VARS[$extra_param].'&' : isset($HTTP_GET_VARS[$extra_param]) ? $extra_param.'='.$HTTP_GET_VARS[$extra_param].'&' : $extra_param.'=&';
Would something like that do the trick? Probably better to seperate into a seperate utility sub.
function GetGPLVar($param) {
if isset($HTTP_POST_VARS[$param]) {
return $HTTP_POST_VARS[$param]
} elseif isset($HTTP_GET_VARS[$param]) {
return $HTTP_POST_VARS[$param]
} else {
return isset(${$param}) ? ${$param} : ''
}
...
$extra_params .= $extra_param.'='.GetGPLVar($extra_param).'&'
...
I know there's a more appropriate place for something like this, but I'm not familiar enough with XOOPS core yet to even guess. Also please pardon my hack|buggy PHP coding, I don't have time to properly test the code now or look up the XOOPS coding standards.
Thanks for all the great work.