From e63afc14f2151820a5a03e708db9251d9b6d35b0 Mon Sep 17 00:00:00 2001 From: Jack Brinkman Date: Fri, 24 Nov 2023 10:56:34 +1100 Subject: [PATCH 1/2] Made dropZone null by default for fileupload() calls to fix erroneous API calls --- grails-app/assets/javascripts/document.js | 1 + grails-app/assets/javascripts/knockout-custom-bindings.js | 7 ++++--- grails-app/assets/javascripts/outputs.js | 2 ++ 3 files changed, 7 insertions(+), 3 deletions(-) diff --git a/grails-app/assets/javascripts/document.js b/grails-app/assets/javascripts/document.js index c861bf545..8c6bb2659 100644 --- a/grails-app/assets/javascripts/document.js +++ b/grails-app/assets/javascripts/document.js @@ -283,6 +283,7 @@ function attachViewModelToFileUpload(uploadUrl, documentViewModel, uiSelector, p $(uiSelector).fileupload({ url:uploadUrl, pasteZone: null, + dropZone: null, formData:function(form) { return [{name:'document', value:documentViewModel.toJSONString()}] }, diff --git a/grails-app/assets/javascripts/knockout-custom-bindings.js b/grails-app/assets/javascripts/knockout-custom-bindings.js index 79cf95b83..f7d74dc2f 100644 --- a/grails-app/assets/javascripts/knockout-custom-bindings.js +++ b/grails-app/assets/javascripts/knockout-custom-bindings.js @@ -281,6 +281,7 @@ ko.bindingHandlers.stagedImageUpload = { $(element).fileupload({ url: config.url, pasteZone: null, + dropZone: null, autoUpload: true }).on('fileuploadadd', function (e, data) { complete(false); @@ -703,11 +704,11 @@ ko.bindingHandlers.fileUploadNoImage = { var defaults = {autoUpload: true}; var settings = { - pasteZone: null + pasteZone: null, + dropZone: null }; $.extend(settings, defaults, options()); - $(element).fileupload(settings - ).on('fileuploadadd', function (e, data) { + $(element).fileupload(settings).on('fileuploadadd', function (e, data) { window.incrementAsyncCounter && window.incrementAsyncCounter(); }).on('fileuploaddone', function (e, data) { window.decreaseAsyncCounter && window.decreaseAsyncCounter(); diff --git a/grails-app/assets/javascripts/outputs.js b/grails-app/assets/javascripts/outputs.js index 7c091cc87..7cefb6a54 100644 --- a/grails-app/assets/javascripts/outputs.js +++ b/grails-app/assets/javascripts/outputs.js @@ -152,6 +152,7 @@ ko.bindingHandlers.photoPointUpload = { $(element).fileupload({ url:config.url, pasteZone: null, + dropZone: null, autoUpload:true }).on('fileuploadadd', function(e, data) { complete(false); @@ -360,6 +361,7 @@ ko.bindingHandlers.fileUploadWithProgress = { $(element).fileupload({ url: config.url, pasteZone: null, + dropZone: null, autoUpload: true }).on('fileuploadadd', function (e, data) { complete(false); From 4c5eea6384b0bf9a009aa53f160d6ed353d5d929 Mon Sep 17 00:00:00 2001 From: Jack Brinkman Date: Tue, 28 Nov 2023 12:09:03 +1000 Subject: [PATCH 2/2] Fixes #1577 --- .../au/org/ala/biocollect/merit/ActivityService.groovy | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/grails-app/services/au/org/ala/biocollect/merit/ActivityService.groovy b/grails-app/services/au/org/ala/biocollect/merit/ActivityService.groovy index 7288aec23..ac29e5b1f 100644 --- a/grails-app/services/au/org/ala/biocollect/merit/ActivityService.groovy +++ b/grails-app/services/au/org/ala/biocollect/merit/ActivityService.groovy @@ -116,9 +116,9 @@ class ActivityService { webService.doDelete(grailsApplication.config.ecodata.service.url + '/activity/' + id) } - def bulkDelete(List ids, boolean destory) { + def bulkDelete(List ids, boolean destroy = false) { String url = grailsApplication.config.ecodata.service.url + '/activityBulkDelete' - if(destory) + if(destroy) url += '?destroy=true' webService.doPost(url, [ids: ids]) }