From 573ed669224ef511f8b7c6e78c77fdcdbfdc47ad Mon Sep 17 00:00:00 2001
From: vieron <javiersanchezmarin@gmail.com>
Date: Tue, 14 Aug 2012 00:06:03 +0200
Subject: [PATCH] fixes in drag/stop callbacks. Related to #23.

- break ondrag callback execution if stop event already was fired.
- unset this.$player after the execution of draggable.stop callback.
---
 src/jquery.gridster.js | 16 ++++++++++------
 1 file changed, 10 insertions(+), 6 deletions(-)

diff --git a/src/jquery.gridster.js b/src/jquery.gridster.js
index 3d10810600..56eabc6faa 100644
--- a/src/jquery.gridster.js
+++ b/src/jquery.gridster.js
@@ -474,6 +474,11 @@
     * @param {Object} A prepared ui object.
     */
     fn.on_drag = function(event, ui) {
+        //break if dragstop has been fired
+        if (this.$player === null) {
+            return false;
+        };
+
         var abs_offset = {
             left: ui.position.left + this.baseX,
             top: ui.position.top + this.baseY
@@ -548,15 +553,14 @@
         this.$player.coords().grid.row = this.placeholder_grid_data.row;
         this.$player.coords().grid.col = this.placeholder_grid_data.col;
 
-        this.$player = null;
-
-        this.$preview_holder.remove();
-
-        this.set_dom_grid_height();
-
         if (this.options.draggable.stop) {
           this.options.draggable.stop.call(this, event, ui);
         }
+
+        this.$preview_holder.remove();
+        this.$player = null;
+
+        this.set_dom_grid_height();
     };