Not a bug XenForo Redirection Scripts for vBulletin

Rasmus Vind

Well-known member
Affected version
Latest version
In the file XF301VB/Pub/Controller/Redirect there is the method:
PHP:
    public function actionAttachment(ParameterBag $params)
    {
        if ($params->filedataid)
        {
            return $this->redirectContent($params, 'attachment_filedata', 'XF:Attachment', 'attachments',
                $params->filedata_id);
        }
        else
        {
            return $this->redirectContent($params, 'attachment', 'XF:Attachment', 'attachments');
        }
    }

The code checks if $params->filedataid is set and then uses $params->filedata_id. From what I can tell it would never go into the then-clause.
 
Top Bottom