[ 
http://jira.magnolia.info/browse/MGNLDATA-32?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Philippe Marschall updated MGNLDATA-32:
---------------------------------------

    Attachment: DialogFile.java.patch

Actually the bug is in the module magnolia-gui.

The bug report is a bit unclear. The bug is only visible for something for 
which a preview image can not be shown like a PDF. This is because the class 
DialogFile does not consistently use the mehtod #getFileURI which 
DataFileControl overrides. There is the additional issue that DialogFile does 
not set the file extension for the download URI which makes Magnolia return the 
wrong content type. For something like a PDF this results in the binary data of 
the PDF being displayed.

h2.Steps to Reproduce
-  configure the data-module to have an element containing "data" (controlType 
dataFile, type Binary)
- create a new such data node and store a PDF
- save the changes
- open the node once again
- click on the link with the PDF

h2.Expected Behavior
- a PDF download starts with the correct content type
- a PDF reader like Adobe Reader is started to display the download

h2.Observed Behavior
- a 404 response form the server

Fix attached

DialogFile should be reviewed for more such bugs.

> Invalid link to file in dialog
> ------------------------------
>
>                 Key: MGNLDATA-32
>                 URL: http://jira.magnolia.info/browse/MGNLDATA-32
>             Project: Magnolia Data Module
>          Issue Type: Bug
>    Affects Versions: 1.1
>            Reporter: Wolfgang Habicht
>            Assignee: Philipp Bracher
>            Priority: Minor
>         Attachments: DialogFile.java.patch
>
>
> - configure the data-module to have an element containing "data" (controlType 
> file, type Binary)
> - create a new such data node and store a non-picture file (for example a 
> video)
> - save the changes
> - open the node once again; now there is a symbol with the video and a link 
> to the file
> ==> the url in that like is invalid; /data is missing at the beginning

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.magnolia.info/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

----------------------------------------------------------------
for list details see
http://documentation.magnolia.info/
----------------------------------------------------------------

Reply via email to