[2.1]: Wrong Content-Disposition header being set for attachments served to mobile browsers

[Sesquipedalian]

Basic Information

Attachments are being served to mobile browsers with a "Content-Disposition: inline" header when they should be served with a "Content-Disposition: attachment" header.

https://www.simplemachines.org/community/index.php?msg=4169626

Steps to reproduce

1. Attach an XML file to a post.
2. Attempt to download the attachment using a mobile browser.

Expected result

The XML file is saved as a file.

Actual result

The XML file is opened in a browser window.

Version/Git revision

2.1.4

Database Engine

All

Database Version

No response

PHP Version

No response

Logs

No response

Additional Information

The problem appears to be the !== comparison operators on this line:

SMF/Sources/ShowAttachments.php

Line 336 in 7f22610

if (isset($_REQUEST['image']) || (isBrowser('is_mobile') && (strpos($file['mime_type'], 'audio/') !== 0 || strpos($file['mime_type'], 'video/') !== 0)))

They should actually be === operators.