Problem with notes and file attachments

[jukuo]

Hello!

When I try to download attached file (only in notes section) the filename has changed to index.php. For example view shows xxx.docx but when I try to download, it will change to index.php!
If I open the index.php with word, it will open the correct xxx.docx file.

How to change the filename to correct one?

Sorry about bad english

-Juho

[andopes]

Definitely this is not the regular SugarCRM behavior.
What is the version of your current SugarCRM installation?
Did you install any package (which ones) or customize code by hand?
Did you upgrade your instance from some older one?

Cheers

[nasuk]

Hi,

I'm having the same problem. All was working well, but I upgraded from 6.5.5 to 6.5.9 this morning. Now when I download an attachment (labelled correctly as a PDF), it downloads the correct file, but has named it index.php (and therefore the browser doesn't know how to open it so just downads it).

If I open the index.php file in a text editor I see the "PDF-1.4" in the header, and if I rename the index.php file to index.pdf it will open correctly. I've checked with an xlsx file also and same result.

If I open it using the iPad it actually opens the file but names it index.php - I guess the iPad examines the content of the file to determine its type rather than the extension.

Thanks and best regards

Dominic Watts
SugarCRM CE 6.5.9 -- Ubuntu Linux Server 10.4.3
Apache 2.2.14 -- PHP 5.3.2 -- MySQL 5.1.66

[domwatts]

Hi,

I'm having the same problem. .All was working well, but I upgraded from 6.5.5 to 6.5.9 yesterday morning. .Now when I download an attachment in Notes (file name shown correctly a "Filename.pdf"), it downloads the correct file/data, but has named it index.php (and therefore the browser doesn't know how to open it, and instead just downloads it). .

If I open the index.php file in a text editor I see the "PDF-1.4" in the header, and if I rename the index.php file to index.pdf it will open correctly. .I've checked with an xlsx file also and same result. I have also uploaded a new file and tried downloading it - same result.

Same problem on both IE9 and Chrome, though if I open it using the iPad it actually opens the file correctly but names it index.php in the safari tab title - I guess the iPad examines the content of the file to determine its type rather than the extension.

On my test instance I rolled this back to 6.5.5, and then upgraded in steps, and the code 'broke' between 6.5.7 and 6.5.9

Thanks and best regards,

[apologies if a double post, but don't think previous post as nasuk made it to the moderators due to vbulletin account issue]

Dominic Watts
SugarCRM CE 6.5.9 . -- . Ubuntu Linux Server 10.4.3
Apache 2.2.14 . -- . .PHP 5.3.2 . -- . .MySQL 5.1.66

[cbrown2375]

Hello,
You guys are not the only ones. I am running into this issue now as well. I basically did the same as domwatts. I upgraded from a previous release to 6.5.9 and now we have the issue of attachments coming up with the name of index.php when users try to view/download the file.

[bizna]

Validated here as well, after upgrading to CE 6.5.9. It worked good before that.

It is actually the downloading part that is not working well.

I tried all sorts of 'repairs' from the admin page, with no actual change.

There is a reported bug, that is supposed to be fixed in 6.6.2:
http://www.sugarcrm.com/support/bugs...ch#issue_59963

Is there any way to fix it before the Sugar team attend to what they destroyed during the last update?

[danilaz]

Indeed, there is a bug in 6.5.9 in the file wich manages the download entry point, download.php. You can edit that file as a workaround until new SugarCRM version fixes this problem. You only have to replace notes with Notes in line 135:

135 } elseif($file_type == 'notes') {

135 } elseif($file_type == 'Notes') {

Hope this helps!

[daniloaz]

Indeed, there is a bug in 6.5.9 in the file wich manages the download entry point, download.php. You can edit that file as a workaround until new SugarCRM version fixes this problem. You only have to replace notes with Notes in line 135:

135 } elseif($file_type == 'notes') {

135 } elseif($file_type == 'Notes') {

Hope this helps!

[domwatts]

Thanks daniloaz/danilaz,

This certainly worked for me and should keep me going until it is properly fixed.

I also noted that the following code in download.php changed between 6.5.5 and 6.5.9,

6.5.5
header("Content-Disposition: attachment; filename=\"".$name."\";");

6.5.9
if (!$check_image) {
header("Content-Disposition: attachment; filename=\"".$name."\";");
}

and if I just stop this being an if clause and run the header() regardless as per 6.5.5 it also makes the problem "go away". I guess the issue is to do with whatever $check_image is used for, so I have no idea what I might have broken.

I would love to know what the official fix in 6.2.2 is as I would much rather patch my instance properly - but hey ......

Thanks again,

Dominic Watts
SugarCRM CE 6.5.9 . -- . Ubuntu Linux Server 10.4.3
Apache 2.2.14 . -- . .PHP 5.3.2 . -- . .MySQL 5.1.66

[jmertic]

This will be fixed in 6.5.10 next week. In the meantime, here's the fix...

diff --git a/sugarcrm/download.php b/sugarcrm/download.php
index c9988ef..927d036 100644
--- a/sugarcrm/download.php
+++ b/sugarcrm/download.php
@@ -191,10 +191,8 @@
             {
                 header("Content-type: application/octet-stream");
             }
-
-            if (!$check_image) {
-                header("Content-Disposition: attachment; filename=\"".$name."\";");
-            }
+               header("Content-Disposition: attachment; filename=\"".$name."\";");
+            
 		}
 		// disable content type sniffing in MSIE
 		header("X-Content-Type-Options: nosniff");