pass a response callback function to custom target functions. by @bsh…

…elton229. fix #65
NicolasCARPi · Feb 20, 2018 · 9bbf74c · 9bbf74c
1 parent 923415f
commit 9bbf74c
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 7 deletions.
diff --git a/demos/index.html b/demos/index.html
@@ -484,7 +484,7 @@ <h4 class='trigger'><i class='fas fa-eye'></i> Show source code</h4>
                 onedit : function() { console.log('If I return false edition will be canceled'); return true;},
                 before : function() { console.log('Triggered before form appears');},
                 callback : function(result, settings, submitdata) {
-                    console.log('Triggered after submit');
+                    console.log('Callback function: triggered after submit');
                     console.log('Result: ' + result);
                     console.log('Settings.width: ' + settings.width);
                     console.log('Submitdata: ' + submitdata.pwet);

diff --git a/src/jquery.jeditable.js b/src/jquery.jeditable.js
@@ -341,13 +341,22 @@
 
                           /* Check if given target is function */
                           if ($.isFunction(settings.target)) {
-                              var str = settings.target.apply(self, [input.val(), settings, self]);
-                              $(self).html(str);
-                              self.editing = false;
-                              callback.apply(self, [self.innerHTML, settings]);
-                              if (!$.trim($(self).html())) {
-                                  $(self).html(settings.placeholder);
+                             /* Callback function to handle the target reponse */
+                              var responseHandler = function(value) {
+                                  $(self).html(value);
+                                  self.editing = false;
+                                  callback.apply(self, [self.innerHTML, settings]);
+                                  if (!$.trim($(self).html())) {
+                                      $(self).html(settings.placeholder);
+                                  }
+                              };
+                              /* Call the user target function */
+                              var userTarget = settings.target.apply(self, [input.val(), settings, responseHandler]);
+                              /* Handle the target function return for compatibility */
+                              if (false !== userTarget && undefined !== userTarget) {
+                                  responseHandler(userTarget);
                               }
+
                           } else {
                               /* Add edited content and id of edited element to POST. */
                               var submitdata = {};