Parcourir la source

Fix #215: Set default delete method REST compliant

Kartik Visweswaran il y a 10 ans
Parent
commit
26a9f846b0
3 fichiers modifiés avec 3 ajouts et 2 suppressions
  1. 1 0
      CHANGE.md
  2. 2 2
      js/fileinput.js
  3. 0 0
      js/fileinput.min.js

+ 1 - 0
CHANGE.md

@@ -46,6 +46,7 @@ version 4.1.8
 23. (enh #211): Add ability to show detailed server error stack via `showAjaxErrorDetails`.
 24. (enh #212): Revamp preview to use a new preview caching object.
 25. (enh #213): Code cleanup, eliminate change event on clear and properly reset preview cache after ajax deletes.
+26. (enh #215): Set default delete method REST compliant.
 
 version 4.1.7
 =============

+ 2 - 2
js/fileinput.js

@@ -907,7 +907,7 @@
                 }
                 settings = $.extend({
                     url: vUrl,
-                    type: 'POST',
+                    type: 'DELETE',
                     dataType: 'json',
                     data: $.extend({key: vKey}, extraData),
                     beforeSend: function (jqXHR) {
@@ -919,7 +919,7 @@
                         index = parseInt($frame.data('fileindex').replace('init_', ''));
                         config = isEmpty(cache.config) && isEmpty(cache.config[index]) ? null : cache.config[index];
                         extraData = isEmpty(config) || isEmpty(config.extra) ? deleteExtraData : config.extra;
-                        if (data.error === undefined) {
+                        if (data === undefined || data.error === undefined) {
                             previewCache.unset(self.id, index);
                             self.raise('filedeleted', [vKey, jqXHR, extraData]);
                         } else {

Fichier diff supprimé car celui-ci est trop grand
+ 0 - 0
js/fileinput.min.js


Certains fichiers n'ont pas été affichés car il y a eu trop de fichiers modifiés dans ce diff