Rasmus Vind
Well-known member
- Affected version
 - Latest version
 
In the file XF301VB/Pub/Controller/Redirect there is the method:
	
	
	
		
The code checks if
				
			
		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.