If "Save file" is activated, the file will not be sent as an attachment.

[Ainschy]

By shortening the absolute file path, the file can be sent as an attachment.

[aschempp]

Makes sense to me, thanks!

However:

• Use TL_ROOT instead of \Environment::get('documentRoot')
• Add a check if the target file actually exists in TL_ROOT

[Toflar]

And why do you do this str_replace('\\', '/',$file['tmp_name'] )? Do you run it on windows?

[Ainschy]

Yes, my development System is Windows.

[Ainschy]

Die zusätzliche Abfrage habe ich nun vereinfacht.

[Toflar]

Now I understand what this should be doing, however I think this should rather look like this:

public static function getFileUploadPathForToken(array $file)
{
    // Check if it has been saved by Contao and thus moved to it's final destination already
    if (isset($file['uploaded']) && $file['uploaded'] === true) {
        $basePath = TL_ROOT . '/';
        if (preg_match('/^' . preg_quote($basePath, '/') . '/', $file['tmp_name'])
            && file_exists($file['tmp_name'])
        ) {

            return str_replace($basePath, '', $file['tmp_name']);
        }

        return null;
    }

    if (!is_uploaded_file($file['tmp_name'])) {

        return null;
    }

    $tmpDir   = 'system/tmp';
    $filePath = $tmpDir . '/' . $file['name'];
    \Files::getInstance()->move_uploaded_file($file['tmp_name'], $filePath);
    \Files::getInstance()->chmod($filePath, $GLOBALS['TL_CONFIG']['defaultFileChmod']);

    return $filePath;
}

Can you check that?