Bug 1056866

Summary: Error message should be shown when uploading unsupported Open Document Format (e.g. fodt)
Product: [Retired] Zanata Reporter: Damian Jansen <djansen>
Component: Component-LogicAssignee: Damian Jansen <djansen>
Status: CLOSED CURRENTRELEASE QA Contact: Zanata-QA Mailling List <zanata-qa>
Severity: high Docs Contact:
Priority: unspecified    
Version: 3.2CC: dchen, sflaniga, zanata-bugs
Target Milestone: ---   
Target Release: 3.3   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 3.3.0-SNAPSHOT (20140210-0043) Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-03-20 05:47:07 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
Test fodt file
none
Test fodp file none

Description Damian Jansen 2014-01-23 04:53:59 UTC
Created attachment 854183 [details]
Test fodt file

Description of problem:
Cannot upload a .fodt file, responds with
- Invalid document format for test-odt.fodt.
- Transaction failed
There seems to be no indication in the logs to what happened.

Version-Release number of selected component (if applicable):
dev

How reproducible:
Easy always

Steps to Reproduce:
1. Sign in as admin / maintainer, go to a project and project version
2. Click Source Documents, Upload Document
3. Select the attached document and click Upload

Actual results:
- Invalid document format for test-odt.fodt.
- Transaction failed

Expected results:
Upload and save

Additional info:
The same document, saved as an .odt, seems to work fine.

Comment 1 Damian Jansen 2014-01-23 05:09:58 UTC
Created attachment 854189 [details]
Test fodp file

Same goes for .fodp (OpenDoc' Presentation)

Comment 2 Damian Jansen 2014-01-23 05:15:19 UTC
Further testing indicates it applies to all Libreoffice .f*** (Open Document format) files.

Tested using LibreOffice 3.4.5 on RHEL6.4

Comment 3 Sean Flanigan 2014-01-23 05:22:33 UTC
I haven't heard of .fodt before, so I looked it up on wikipedia:

"The OpenDocument format supports the following two ways of document representation:

"As a single XML document - also known as Flat XML or Uncompressed XML Files. Single OpenDocument XML files are not widely used. Filename extensions for a single OpenDocument XML documents are not defined in the OpenDocument technical specification, but commonly used are .xml, .fodt, .fods, ... etc."

https://en.wikipedia.org/wiki/OpenDocument_technical_specification#Document_Representation


So, we're talking about a single XML file instead of the usual "multiple XML files inside a ZIP container".

As if the (compressed) ODT files weren't taking up enough space...

Comment 4 Damian Jansen 2014-01-28 05:04:42 UTC
It appears that it isn't our parsing of the documents that is wrong, but the fact we incorrectly state we support them at all.

Fix: Remove .fod* files from the list of supported filetypes in 
zanata-war/src/main/resources/messages.properties tag jsf.SupportedUploadFormats

Comment 5 Damian Jansen 2014-01-29 05:54:24 UTC
https://github.com/zanata/zanata-server/pull/346

Comment 6 Ding-Yi Chen 2014-02-10 01:02:11 UTC
VERIFIED with Zanata 3.3.0-SNAPSHOT (20140210-0043)

Comment 7 Sean Flanigan 2014-03-20 05:47:07 UTC
Closing VERIFIED bugs for Zanata server 3.3.2.