Errors in beforeDelete are not shown to the user

[multiwebinc]

Winter CMS Build

1.2

PHP Version

8.1

Database engine

SQLite

Plugins installed

No response

Issue description

When saving a model, the following code causes the error message to be displayed to the user:

public function beforeSave() 
{
    throw new \Winter\Storm\Exception\ApplicationException('Unable to save');
}

However when deleting the model, the following code does not display the error to the user:

public function beforeDelete()
{
    if (!empty($this->categories)) {
        throw new \Winter\Storm\Exception\ApplicationException('Unable to delete');
    }
}

The request responses appear to be the same (i.e. both return 500 status code and just the error message string), so I would assume the bug lies somewhere in the client-side code.

Steps to replicate

(see description)

Workaround

No response

[multiwebinc]

The error is because swal is used for both $.wn.alert() as well as $.wn.confirm. It is trying to display an alert before SweetAlert has finished closing.

[multiwebinc]

winter/modules/backend/assets/js/winter.alert.js

Lines 21 to 36 in 68b9a55

	$.wn.alert = function alert(message) {
	swal({
	title: message,
	confirmButtonClass: 'btn-primary'
	})
	}
	$.wn.confirm = function confirm(message, callback) {
	swal({
	title: message,
	showCancelButton: true,
	confirmButtonClass: 'btn-primary'
	}, callback)
	}

[bennothommo]

@multiwebinc thanks for the report. Confirmed the issue - will try and investigate a fix.

[multiwebinc]

@bennothommo Maybe upgrading to SweetAlert 2 would be an option since it is promise-based. They even have an ajax request example in the docs.

[LukeTowers]

@bennothommo any updates?

[bennothommo]

Not at this stage.

[LukeTowers]

Open for anyone who wants to submit a PR to fix it if they want it fixed; otherwise I'll just let the bot close it next time it comes up.