geertjanknapen1
commented
2 weeks ago I did try using the Files fieldtypes, and this does not throw an exception, but as we handle mailing ourselves, the proper files are not added as an attachment as the documentation states.
Additional information regarding how you end up in the custom FormController. Routing
Route::controller(FormController::class)
->middleware(['ajax']) # Returns 403 error if !$request->ajax(), otherwise continues.
->group(function () {
Route::post('/ajaxforms/{formHandle}', 'submitStatamicForm')->name('submit-statamic-form');
});AJAX form submission
'use strict'
// Define the 'forms' namespace
if (!('forms' in window)) window['forms'] = [];
$(function() {
$('body').on('click', '.form-ajax-submit', function (e) {
e.preventDefault();
forms.disableSubmit(e); // Disables the submit button after user clicked it, not relevant for issue.
forms.ajaxSubmit(e);
});
});
/*******************************
* Methods ('forms' namespace) *
*******************************/
forms.ajaxSubmit = function (e) {
let form = $(e.target).closest('form');
let formId = form.attr('id');
let formName = form.attr('name');
let formErrorSpan = form.find('span[data-failure]');
let failureMessage = formErrorSpan.data('failure');
// clear potential previously set validation errors
// forms.clearValidationErrors(form); // For this issue you can assume no validation errors happen.
// show spinning loader
$('#envelope-icon').addClass('d-none');
$('#spinning-loader').removeClass('d-none');
let formDataForm = $('form#' + formId)[0];
let formData = new FormData(formDataForm);
// If the form contains the resume fields (file) we want to append both the resume and motivation files to the FormData
if ($('input[name="resume"]').length > 0) {
formData.append('resume', $('input[name="resume"]')[0].files[0]);
formData.append('motivation', $('input[name="motivation"]')[0].files[0]);
}
$.ajax('/ajaxforms/' + formName, {
type: 'POST',
processData: false,
contentType: false,
data: formData,
})
.done(function (response) {
let formContainer = form.closest('div');
$(formContainer).html(response.renderedSuccessView);
forms.enableSubmit(e);
})
.fail(function (response) {
if (response.status === 404) {
console.log(response.responseJSON.message);
formErrorSpan.html(failureMessage);
formErrorSpan.show();
} else {
$.each(response.responseJSON.error, function (formFieldName, validationErrorText) {
let formField = form.find('[name=' + formFieldName + ']');
let formFieldValidationSpan = form.find('span.' + formFieldName);
formFieldValidationSpan.html(validationErrorText);
formFieldValidationSpan.show();
formField.addClass('is-invalid');
});
}
forms.enableSubmit(e); // Enables the submit button again, not relevant for issue
});Below is the code for the listener, it uses Enums to check the form handle which should not be relevant to the issue.
The handleFormSubmission is part of a Trait and just sends the Mailables to the correct location
public function handle(FormSubmitted $event)
{
if ($event->submission->form->handle() === Form::APPLICATION_FORM->value) {
// Handle form submission (send mail and store submitted data)
$this->handleFormSubmission($event, ApplicationFormInternalMailable::class, ApplicationFormExternalMailable::class);
// return false so Statamic does not send the e-mail (this also stops propagation of this event to other listeners!!).
return false;
}
// return nothing so that propagation is not stopped unless another listener picks up and handles this event
}
duncanmcclean
Bug description
Important, I have seen and looked at #6431 and #7294 which mention the same error message, they do not seem related to this issue.
So, we have custom form handling for our frontend forms. A long story short, it works as follows;
parent::submit().So, explaining out of the way, I use Assets fields in the form, since Statamic does not handle the emailing so I need to store the uploaded files on disk, as per the documentation.
This throws the aforementioned exception in the FormController, but I can't seem to figure out why or what I'm doing wrong, so any nudges in the correct direction would be greatly appreciated.
How to reproduce
I don't think it's easily reproducible, we have a heavily customised setup.
But the jist would be, prevent Statamic from sending the emails by listening for the FormSubmitted event and returning false at the end of it. Create a form with one or more Assets fields. Try to submit that through AJAX with a custom FormController doing as explained in the Bug description.
Logs
Environment
Installation
Existing Laravel app
Additional details
Form blueprint:
FormController (custom):