Bug 880879 - Undo button causing repeated save failures and other weirdness
Summary: Undo button causing repeated save failures and other weirdness
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Zanata
Classification: Retired
Component: Component-UI
Version: 2.0
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: ---
: 2.1
Assignee: Patrick Huang
QA Contact: Ding-Yi Chen
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-11-28 02:19 UTC by Hedda Peters
Modified: 2013-02-26 04:06 UTC (History)
3 users (show)

Fixed In Version: 2.1-SNAPSHOT (20121206-1113)
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2013-02-26 04:06:15 UTC
Embargoed:


Attachments (Terms of Use)

Description Hedda Peters 2012-11-28 02:19:14 UTC
Description of problem:
I encountered a failed save operation when trying out the undo-button on a message that I translated just before. After that failed save, I seem to be unable to save that entry no matter what I do (bar reloading the whole page, which I haven't tried yet)


How reproducible:
not sure


Additional info:

https://translate.engineering.redhat.com/webtrans/translate?project=rhn-satellite-user-guide&iteration=5.5&localeId=de&locale=de#view:doc;doc:OpenSCAP;projectsearch:compliance 

	
[12:09:07] Save FAILED: id 182189
[12:09:25] Save FAILED: id 182189
[12:09:39] Save FAILED: id 182189
... (some succesful operations on other strings)
[12:14:38] Save FAILED: id 182189

Comment 1 Hedda Peters 2012-11-28 03:07:22 UTC
I reloaded the browser's page now, and the troublesome entry appears translated/approved.

Comment 2 Patrick Huang 2012-11-28 03:20:45 UTC
It maybe related to bug 873525

Comment 3 Patrick Huang 2012-11-28 03:24:23 UTC
(In reply to comment #1)
> I reloaded the browser's page now, and the troublesome entry appears
> translated/approved.

Does this mean it's saved even though the notification says failed? I can see a bunch of translation history entries within minutes

Comment 4 Hedda Peters 2012-11-28 03:34:25 UTC
(In reply to comment #3)
> (In reply to comment #1)
> > I reloaded the browser's page now, and the troublesome entry appears
> > translated/approved.
> 
> Does this mean it's saved even though the notification says failed? I can
> see a bunch of translation history entries within minutes

Possibly. What I did was somthing like this:

1) Entry was untranslated, I translated as "ABC" and saved successfully.
2) I moved on to different entry and translated something else
3) Came back to original entry and changed translation to "ABC test" - I *think* this save operation still succeeded, although not entirely sure now.
4) I then changed the translation back to "ABC", but this save operation produced the error message of failed save.
5) Tried again multiple times to save the translation "ABC", it kept failing. The entry was highlighted in blue indicating unsaved changes

After the page reload the correct translation "ABC" was shown, but I can't tell from which save-operation that translation comes exactly.

Comment 5 Sean Flanigan 2012-11-28 04:01:30 UTC
Where in those steps did you use Undo?  And by that did you mean the Cancel Changes button?  Or was it the history button?

Comment 6 Hedda Peters 2012-11-28 04:33:26 UTC
(In reply to comment #5)
> Where in those steps did you use Undo?  And by that did you mean the Cancel
> Changes button?  Or was it the history button?

No, you're right - I missed that. I used the "Undo" button at step 4. (Which now confirms that the save in step 3 must have worked). So I wanted to use the "Undo" button to change the translation back from "ABC test" to "ABC", which failed.

Sorry for the confusion, I didn't pay too much attention at the time, not expecting to having to try to reproduce.

Comment 7 Sean Flanigan 2012-11-28 05:39:58 UTC
Thanks Hedda, but I'm still not clear on what button you pressed.  What do you mean by the Undo button in this case?

Comment 9 Sean Flanigan 2012-11-28 06:45:54 UTC
Oh, that Undo button!  Duh, sorry!  I forget we'd finally implemented that.

If I use Undo for step 4, it undoes 'ABCtest' and changes it back to 'ABC' (in the database, if you refresh the page or look at the TM results you can see it), but it leaves the field on the screen empty.  Then if you type something, the 'ABC' suddenly appears.  It still saves okay for me, so I can't reproduce this bug, but we certainly need to look at Undo.

Comment 10 Patrick Huang 2012-11-30 03:00:26 UTC
cannot reproduce it. But I did fix the blank field symptom (sign for CodeMirror refresh) after undo. 
Fix commit in : https://github.com/zanata/zanata/commit/50d04fc713711b5e62db0d26aab529028b3ce49b

Comment 11 Ding-Yi Chen 2012-12-06 02:29:05 UTC
VERIFIED with Zanata version 2.1-SNAPSHOT (20121206-1113)


Note You need to log in before you can comment on or make changes to this bug.