Bug 995909

Summary:	Uploading a html file in the TM import feature doesn't return
Product:	[Retired] Zanata	Reporter:	Damian Jansen <djansen>
Component:	Component-Logic	Assignee:	zanata-dev-internal <zanata-dev-internal>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Damian Jansen <djansen>
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	development	CC:	sflaniga, zanata-bugs
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2013-11-27 03:31:32 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:

Description Damian Jansen 2013-08-12 00:27:51 UTC

Description of problem:
Selecting a html as a TM source for import doesn't return control of the UI

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

How reproducible:
Easy always

Steps to Reproduce:
1. Sign in as administrator
2. Go to Administration, Translation Memory
3. Create New
4. Enter in TM details and Save
5. Click Import
6. Click filename
7. Select a html file
8. Press Upload

Actual results:
Process begins, but does not return

Expected results:
Error, not a TMX file / Complete, 0 entries found (preferably the former)

Additional info:
Console says 09:39:44,429 INFO  [org.zanata.tmx.TMXParser] (http-/127.0.0.1:8080-5) parsing started for: testname
No "parsing stopped for" message occurs until the page is refreshed and a new TM import is started.

Comment 1 Damian Jansen 2013-08-12 01:54:57 UTC

Correction - it does return, but it takes a good few minutes, sitting at 100% uploaded.

Comment 2 Sean Flanigan 2013-08-12 04:00:39 UTC

For flexibility, we don't require uploaded files to declare a TMX DTD, but I suppose we should fail fast if the file doesn't at least have a <tmx> element at the root.

Comment 3 Sean Flanigan 2013-08-12 06:08:28 UTC

https://github.com/zanata/zanata-server/pull/118

Comment 4 Damian Jansen 2013-08-13 00:10:34 UTC

Verified at 75ccfffbc0bab9f924b4f1f94ccae46862f5a008

Comment 5 Sean Flanigan 2013-11-27 03:31:32 UTC

Closing VERIFIED bugs which predate Zanata 3.1.2.