Robotail/Corp
Robotail/Corp
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Исправлены ошибки редактирования профиля:

Browse Source 
1. Изменен редирект при ошибках валидации: теперь возврат на страницу профиля вместо несуществующего GET endpoint
2. Добавлена проверка наличия ldap_dn перед обновлением через LDAP: локальные пользователи без ldap_dn теперь обновляются напрямую в БД
This commit is contained in:
admin i 2025-12-02 10:03:47 +03:00
parent 5291a921f4
commit 8edc856636
2 changed files with 33 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

34
backend/api/v1/employees/views.py
View File

@ -1782,8 +1782,18 @@ class EmployeeViewSet(viewsets.ModelViewSet):
logger.info(f"[ME PATCH] Step 9: svc_changes keys: {list(svc_changes.keys())}")
if ldap_enabled and (svc_changes or move_to_department_dn or group_cns is not None):
# Режим с LDAP: обновляем через DirectoryService
# Проверяем, есть ли у пользователя ldap_dn
has_ldap_dn = False
if ldap_enabled:
from employees.models import LdapSyncState
has_ldap_dn = (
LdapSyncState.objects.filter(employee=instance, ldap_dn__isnull=False).exists()
or (hasattr(instance, 'ldap_dn') and instance.ldap_dn)
)
logger.info(f"[ME PATCH] Step 9.5: LDAP enabled, user has ldap_dn: {has_ldap_dn}")
if ldap_enabled and has_ldap_dn and (svc_changes or move_to_department_dn or group_cns is not None):
# Режим с LDAP: обновляем через DirectoryService (только для пользователей с ldap_dn)
logger.info("[ME PATCH] Step 10: LDAP mode - updating via DirectoryService...")
logger.info(f"[ME PATCH] Step 10: DirectoryService params - svc_changes={list(svc_changes.keys())}, move_to_department_dn={move_to_department_dn}, group_cns={group_cns}")
svc = DirectoryService()
@ -1804,9 +1814,9 @@ class EmployeeViewSet(viewsets.ModelViewSet):
{"detail": str(e)},
status=502 if isinstance(e, DirectoryLdapError) else 500,
)
elif not ldap_enabled and svc_changes:
# Режим без LDAP: обновляем напрямую в БД
logger.info(f"[ME PATCH] Step 10: LDAP disabled, updating DB directly with: {list(svc_changes.keys())}")
elif (not ldap_enabled or not has_ldap_dn) and svc_changes:
# Режим без LDAP или пользователь без ldap_dn: обновляем напрямую в БД
logger.info(f"[ME PATCH] Step 10: LDAP disabled or no ldap_dn, updating DB directly with: {list(svc_changes.keys())}")
for k, v in svc_changes.items():
if k != "position" and k != "avatar_bytes":
setattr(instance, k, v)
@ -2054,8 +2064,18 @@ class EmployeeViewSet(viewsets.ModelViewSet):
except Exception as exc:
print(f"[EMP PATCH] Error reading avatar for LDAP: {exc}")
# Проверяем, есть ли у пользователя ldap_dn
has_ldap_dn = False
if ldap_enabled:
# LDAP mode: sync with DirectoryService
from employees.models import LdapSyncState
has_ldap_dn = (
LdapSyncState.objects.filter(employee=emp, ldap_dn__isnull=False).exists()
or (hasattr(emp, 'ldap_dn') and emp.ldap_dn)
)
print(f"[EMP PATCH] LDAP enabled, user has ldap_dn: {has_ldap_dn}")
if ldap_enabled and has_ldap_dn:
# LDAP mode: sync with DirectoryService (только для пользователей с ldap_dn)
svc = DirectoryService()
if svc_changes or move_to_department_dn or group_cns is not None:
try:
@ -2077,7 +2097,7 @@ class EmployeeViewSet(viewsets.ModelViewSet):
),
)
else:
# Non-LDAP mode: direct DB updates
# Non-LDAP mode or user without ldap_dn: direct DB updates
# Restore ldap_changes back to vd for DB-only save
vd.update(ldap_changes)

12
backend/employees/views_front.py
View File

@ -595,9 +595,9 @@ def employee_edit(request, pk: int):
)
messages.error(request, error_msg)
# Возвращаем на страницу редактирования, а не на профиль
edit_url = reverse("employees:employee_edit", args=(pk,))
return redirect(edit_url)
# Возвращаем на страницу профиля сотрудника
profile_url = reverse("employees:employee_detail", args=(pk,))
return redirect(profile_url)
# ---------- EDIT ME ----------
@ -740,9 +740,9 @@ def employee_edit_me(request):
)
messages.error(request, error_msg)
# Возвращаем на страницу редактирования
edit_url = reverse("employees:employee_edit_me")
return redirect(edit_url)
# Возвращаем на страницу профиля (где есть форма редактирования)
profile_url = reverse("employees:profile")
return redirect(profile_url)
# ---------- CREATE (staff) ----------