Skip to content

[IMP] base_ux: default user tz to creator's when missing#415

Closed
les-adhoc wants to merge 1 commit into
ingadhoc:19.0from
adhoc-dev:19.0-t-69983-les
Closed

[IMP] base_ux: default user tz to creator's when missing#415
les-adhoc wants to merge 1 commit into
ingadhoc:19.0from
adhoc-dev:19.0-t-69983-les

Conversation

@les-adhoc

Copy link
Copy Markdown
Contributor

Cuando hr está instalado, la vista de Preferencias / cambio de contraseña marca tz como requerido. Un usuario creado por script/provisioning nace con tz=False (el partner solo defaultea tz desde context['tz'], la cookie del navegador en el login web) y queda trabado al abrir Preferencias.

Este cambio overridea res.users.create para que, si tz no viene en los vals ni en el context, haga fallback al tz del usuario creador. Así el campo nunca nace vacío sin depender de la cookie ni del método de creación, y sin tocar el required que hr impone a propósito.

Verificación

Probado vía odoo shell sobre base con base_ux + hr instalados:

Caso Resultado
Alta por script (sin tz en context) hereda el tz del creador
Login web (tz en context) gana el context
tz explícito en vals se respeta
Batch create (mezcla con/sin tz) cada uno correcto
Caveat: creador sin tz queda False (documentado)

Task #69983

res.partner only defaults tz from context['tz'] (the browser cookie set
on web login). Users created by scripts/provisioning get no such context
and are born with tz=False, which then clashes with the required that hr
enforces in the Preferences view. Override res.users create to fall back
to the creator's tz so it is never empty.

Task #69983
@roboadhoc

Copy link
Copy Markdown
Contributor

Pull request status dashboard

@les-adhoc

Copy link
Copy Markdown
Contributor Author

Cierro: el enfoque de defaultear tz en res.users.create rompe 14 tests de hr_attendance (corrimiento de 1h porque el tz del usuario alimenta el cálculo de jornada/overtime del empleado). Vamos por setear tz en el provisioning de bases nuevas. Ver tarea #69983.

@les-adhoc les-adhoc closed this Jun 29, 2026
@les-adhoc les-adhoc deleted the 19.0-t-69983-les branch June 29, 2026 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants