Remove jQuery class from the common admin functions (#30137)

- Switched from jQuery class functions to plain JavaScript `classList`
- Tested the new authentication source form and the deletion of system
notices. They work as before

Signed-off-by: Yarden Shoham <git@yardenshoham.com>
Co-authored-by: Giteabot <teabot@gitea.io>
This commit is contained in:
Yarden Shoham 2024-03-27 18:14:18 +02:00 committed by GitHub
parent 1a71dbfb78
commit 1551d73d3f
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

View File

@ -122,7 +122,7 @@ export function initAdminCommon() {
input.removeAttribute('required');
}
$('.binddnrequired').removeClass('required');
document.querySelector('.binddnrequired')?.classList.remove('required');
const authType = this.value;
switch (authType) {
@ -131,7 +131,7 @@ export function initAdminCommon() {
for (const input of document.querySelectorAll('.binddnrequired input, .ldap div.required:not(.dldap) input')) {
input.setAttribute('required', 'required');
}
$('.binddnrequired').addClass('required');
document.querySelector('.binddnrequired')?.classList.add('required');
break;
case '3': // SMTP
showElem('.smtp');
@ -234,16 +234,15 @@ export function initAdminCommon() {
});
document.getElementById('delete-selection')?.addEventListener('click', async function (e) {
e.preventDefault();
const $this = $(this);
$this.addClass('is-loading disabled');
this.classList.add('is-loading', 'disabled');
const data = new FormData();
$checkboxes.each(function () {
if ($(this).checkbox('is checked')) {
data.append('ids[]', $(this).data('id'));
data.append('ids[]', this.getAttribute('data-id'));
}
});
await POST($this.data('link'), {data});
window.location.href = $this.data('redirect');
await POST(this.getAttribute('data-link'), {data});
window.location.href = this.getAttribute('data-redirect');
});
}
}