@@ -351,6 +357,7 @@ function openCreate(){
$('f-orden').value = '0';
$('f-activo').checked = true;
$('hint-email').textContent = 'El email debe coincidir exactamente con la cuenta de Mailcow que el usuario usa para entrar.';
+ $('btn-delete').style.display = 'none'; // solo en modo edición
$('modal').classList.add('open');
setTimeout(()=>$('f-email').focus(), 50);
}
@@ -367,8 +374,16 @@ function openEdit(id){
$('f-orden').value = u.orden;
$('f-activo').checked = u.activo === 1;
$('hint-email').textContent = (u.email||'').startsWith('placeholder-')
- ? 'Reemplaza este email placeholder por el email real del abogado.'
+ ? 'Reemplaza este email placeholder por el email real del abogado. Si el email real ya existe como otro usuario, mejor borra el placeholder con el botón de abajo.'
: 'Cambiar el email afecta la asociación con Mailcow OAuth.';
+ // Mostrar botón Borrar excepto para uno mismo o para otro socio
+ const canDelete = (u.id !== (window.DONNA && window.DONNA.users.find(x => x.email === MY_EMAIL)?.id))
+ && (u.role !== 'socio');
+ // Más simple: solo ocultar si es uno mismo. La API rechaza al socio.
+ const isSelf = u.email === MY_EMAIL;
+ $('btn-delete').style.display = isSelf ? 'none' : 'inline-block';
+ $('btn-delete').disabled = (u.role === 'socio' && !isSelf);
+ $('btn-delete').title = (u.role === 'socio') ? 'Demuévelo a Abogado o Usuario primero' : 'Eliminar permanentemente';
$('modal').classList.add('open');
setTimeout(()=>$('f-name').focus(), 50);
}
@@ -378,6 +393,30 @@ function closeModal(){
editingId = null;
}
+async function deleteUser(){
+ if (!editingId) return;
+ const u = users.find(x => x.id === editingId);
+ if (!u) return;
+ const msg = (u.email||'').startsWith('placeholder-')
+ ? `¿Eliminar el placeholder "${u.name}" (${u.email})?\n\nEsto libera el "apellido" del dropdown para que el usuario real lo tome.`
+ : `¿Eliminar permanentemente al usuario ${u.name || u.email}?\n\nEsto es irreversible. Los trámites históricos que creó o donde está como abogado/involucrado quedarán como referencia (sus vínculos N:M se borran, los strings de email/nombre quedan como historial).\n\nSi solo quieres bloquearle el acceso, mejor destilda "Activo".`;
+ if (!confirm(msg)) return;
+ const btn = $('btn-delete');
+ btn.disabled = true;
+ try {
+ const result = await apiCall('DELETE', 'delete_user', null, 'id=' + editingId);
+ if (!result) return;
+ users = users.filter(x => x.id !== editingId);
+ closeModal();
+ render();
+ toast('Usuario eliminado');
+ } catch(e) {
+ toast('Error: ' + e.message, 'error');
+ } finally {
+ btn.disabled = false;
+ }
+}
+
async function saveUser(){
const payload = {
email: $('f-email').value.trim().toLowerCase(),
@@ -412,7 +451,10 @@ async function saveUser(){
}
document.addEventListener('keydown', e => {
- if (e.key === 'Escape') closeModal();
+ // Solo cerrar con Esc si el modal está abierto Y el foco no está en un input
+ if (e.key === 'Escape' && $('modal').classList.contains('open')) {
+ closeModal();
+ }
});
['search','f-role','f-activo-filter'].forEach(id => {
$(id).addEventListener('input', render);