Remove unnecessary return statements
These statements were automatically added by CoffeeScript. I'm only removing the obvious cases; there might be more cases where the `return` statement isn't necessary.
This commit is contained in:
@@ -1,21 +1,20 @@
|
||||
// Generated by CoffeeScript 1.12.6
|
||||
(function() {
|
||||
"use strict";
|
||||
App.Documentable = {
|
||||
initialize: function() {
|
||||
$(".js-document-attachment").each(function() {
|
||||
return App.Documentable.initializeDirectUploadInput(this);
|
||||
App.Documentable.initializeDirectUploadInput(this);
|
||||
});
|
||||
$("#nested-documents").on("cocoon:after-remove", function() {
|
||||
return App.Documentable.unlockUploads();
|
||||
App.Documentable.unlockUploads();
|
||||
});
|
||||
return $("#nested-documents").on("cocoon:after-insert", function(e, nested_document) {
|
||||
$("#nested-documents").on("cocoon:after-insert", function(e, nested_document) {
|
||||
var input;
|
||||
input = $(nested_document).find(".js-document-attachment");
|
||||
input["lockUpload"] = $(nested_document).closest("#nested-documents").find(".document:visible").length >= $("#nested-documents").data("max-documents-allowed");
|
||||
App.Documentable.initializeDirectUploadInput(input);
|
||||
if (input["lockUpload"]) {
|
||||
return App.Documentable.lockUploads();
|
||||
App.Documentable.lockUploads();
|
||||
}
|
||||
});
|
||||
},
|
||||
@@ -23,7 +22,7 @@
|
||||
var inputData;
|
||||
inputData = this.buildData([], input);
|
||||
this.initializeRemoveCachedDocumentLink(input, inputData);
|
||||
return $(input).fileupload({
|
||||
$(input).fileupload({
|
||||
paramName: "attachment",
|
||||
formData: null,
|
||||
add: function(e, data) {
|
||||
@@ -31,11 +30,11 @@
|
||||
upload_data = App.Documentable.buildData(data, e.target);
|
||||
App.Documentable.clearProgressBar(upload_data);
|
||||
App.Documentable.setProgressBar(upload_data, "uploading");
|
||||
return upload_data.submit();
|
||||
upload_data.submit();
|
||||
},
|
||||
change: function(e, data) {
|
||||
return data.files.forEach(function(file) {
|
||||
return App.Documentable.setFilename(inputData, file.name);
|
||||
data.files.forEach(function(file) {
|
||||
App.Documentable.setFilename(inputData, file.name);
|
||||
});
|
||||
},
|
||||
fail: function(e, data) {
|
||||
@@ -46,7 +45,7 @@
|
||||
App.Documentable.setInputErrors(data);
|
||||
$(data.destroyAttachmentLinkContainer).find("a.delete:not(.remove-nested)").remove();
|
||||
$(data.addAttachmentLabel).addClass("error");
|
||||
return $(data.addAttachmentLabel).show();
|
||||
$(data.addAttachmentLabel).show();
|
||||
},
|
||||
done: function(e, data) {
|
||||
var destroyAttachmentLink;
|
||||
@@ -63,10 +62,10 @@
|
||||
$(destroyAttachmentLink).on("click", function(e) {
|
||||
e.preventDefault();
|
||||
e.stopPropagation();
|
||||
return App.Documentable.doDeleteCachedAttachmentRequest(this.href, data);
|
||||
App.Documentable.doDeleteCachedAttachmentRequest(this.href, data);
|
||||
});
|
||||
if (input["lockUpload"]) {
|
||||
return App.Documentable.showNotice();
|
||||
App.Documentable.showNotice();
|
||||
}
|
||||
},
|
||||
progress: function(e, data) {
|
||||
@@ -93,43 +92,43 @@
|
||||
},
|
||||
clearFilename: function(data) {
|
||||
$(data.fileNameContainer).text("");
|
||||
return $(data.fileNameContainer).hide();
|
||||
$(data.fileNameContainer).hide();
|
||||
},
|
||||
clearInputErrors: function(data) {
|
||||
return $(data.errorContainer).find("small.error").remove();
|
||||
$(data.errorContainer).find("small.error").remove();
|
||||
},
|
||||
clearProgressBar: function(data) {
|
||||
return $(data.progressBar).find(".loading-bar").removeClass("complete errors uploading").css("width", "0px");
|
||||
$(data.progressBar).find(".loading-bar").removeClass("complete errors uploading").css("width", "0px");
|
||||
},
|
||||
setFilename: function(data, file_name) {
|
||||
$(data.fileNameContainer).text(file_name);
|
||||
return $(data.fileNameContainer).show();
|
||||
$(data.fileNameContainer).show();
|
||||
},
|
||||
setProgressBar: function(data, klass) {
|
||||
return $(data.progressBar).find(".loading-bar").addClass(klass);
|
||||
$(data.progressBar).find(".loading-bar").addClass(klass);
|
||||
},
|
||||
setTitleFromFile: function(data, title) {
|
||||
if ($(data.titleField).val() === "") {
|
||||
return $(data.titleField).val(title);
|
||||
$(data.titleField).val(title);
|
||||
}
|
||||
},
|
||||
setInputErrors: function(data) {
|
||||
var errors;
|
||||
errors = "<small class='error'>" + data.jqXHR.responseJSON.errors + "</small>";
|
||||
return $(data.errorContainer).append(errors);
|
||||
$(data.errorContainer).append(errors);
|
||||
},
|
||||
lockUploads: function() {
|
||||
return $("#new_document_link").addClass("hide");
|
||||
$("#new_document_link").addClass("hide");
|
||||
},
|
||||
unlockUploads: function() {
|
||||
$("#max-documents-notice").addClass("hide");
|
||||
return $("#new_document_link").removeClass("hide");
|
||||
$("#new_document_link").removeClass("hide");
|
||||
},
|
||||
showNotice: function() {
|
||||
return $("#max-documents-notice").removeClass("hide");
|
||||
$("#max-documents-notice").removeClass("hide");
|
||||
},
|
||||
doDeleteCachedAttachmentRequest: function(url, data) {
|
||||
return $.ajax({
|
||||
$.ajax({
|
||||
type: "POST",
|
||||
url: url,
|
||||
dataType: "json",
|
||||
@@ -146,9 +145,9 @@
|
||||
$(data.wrapper).find(".attachment-actions").addClass("small-12").removeClass("small-6 float-right");
|
||||
$(data.wrapper).find(".attachment-actions .action-remove").addClass("small-3").removeClass("small-12");
|
||||
if ($(data.input).data("nested-document") === true) {
|
||||
return $(data.wrapper).remove();
|
||||
$(data.wrapper).remove();
|
||||
} else {
|
||||
return $(data.wrapper).find("a.remove-cached-attachment").remove();
|
||||
$(data.wrapper).find("a.remove-cached-attachment").remove();
|
||||
}
|
||||
}
|
||||
});
|
||||
@@ -157,14 +156,14 @@
|
||||
var remove_document_link, wrapper;
|
||||
wrapper = $(input).closest(".direct-upload");
|
||||
remove_document_link = $(wrapper).find("a.remove-cached-attachment");
|
||||
return $(remove_document_link).on("click", function(e) {
|
||||
$(remove_document_link).on("click", function(e) {
|
||||
e.preventDefault();
|
||||
e.stopPropagation();
|
||||
return App.Documentable.doDeleteCachedAttachmentRequest(this.href, data);
|
||||
App.Documentable.doDeleteCachedAttachmentRequest(this.href, data);
|
||||
});
|
||||
},
|
||||
removeDocument: function(id) {
|
||||
return $("#" + id).remove();
|
||||
$("#" + id).remove();
|
||||
}
|
||||
};
|
||||
|
||||
|
||||
Reference in New Issue
Block a user