From: Aleksander Łukasz <aleksander.lukasz@nowoczesnapolska.org.pl>
Date: Tue, 23 Apr 2013 11:44:43 +0000 (+0200)
Subject: Fix: Editing nodes with XNodeAttributes might lead to data corruption
X-Git-Url: https://git.mdrn.pl/redakcja.git/commitdiff_plain/fb8befbb76d37a32317ef82324cecc5dd5208c08?ds=inline

Fix: Editing nodes with XNodeAttributes might lead to data corruption

If node was saved via the save button, the blur event handler was not
unbound and was called on the next blur event.

If this blur event resulted from clicking the edit
button on another node, the XNodeAttributes from that
node were set to the old one handled by unbound event handler,
because those attributes were always taken from the currently
edited node.
---

diff --git a/redakcja/static/js/wiki/view_editor_wysiwyg.js b/redakcja/static/js/wiki/view_editor_wysiwyg.js
index cb7c40f8..c08945ce 100644
--- a/redakcja/static/js/wiki/view_editor_wysiwyg.js
+++ b/redakcja/static/js/wiki/view_editor_wysiwyg.js
@@ -413,13 +413,14 @@
 	      });
 	$overlay.append($attributes);
 
+    var overlayParent = $box[0].offsetParent || $box.parent();
 	$overlay.css({
             position: 'absolute',
             height: h,
             left: x,
             top: y,
             width: w
-        }).appendTo($box[0].offsetParent || $box.parent()).show();
+        }).appendTo(overlayParent).show();
         
 
         if ($origin.is('.motyw')) {
@@ -498,7 +499,7 @@
                             }
 			    /* Set attributres back to the node */
 			    var attrs = {};
-			    $(".html-editarea-attributes [name]")
+			    $(".html-editarea-attributes [name]", overlayParent)
 				.each(function(i, textField) {
 				    attrs[textField.name] = textField.value;
 				});
@@ -564,6 +565,7 @@
 
                 $('.accept-button', $overlay).click(function(){
                     save();
+                    $(document).unbind('click.blur-overlay');
                 });
 
                 $(document).bind('click.blur-overlay', function(event){