[FIX] l10n_es_partner: Corregido mostrar dirección completa tras rellenar nombre comercial

[JuaniFreedoo]

En los formularios de pedidos y facturas, cuando el campo "nombre comercial" (comercial) está rellenado, los campos de dirección del cliente (calle, ciudad, código postal, etc.) desaparecen del formulario y también de las impresiones PDF.

Se ha corregido la función para que esto no suceda

[OCA-git-bot]

Hi @pedrobaeza,
some modules you are maintaining are being modified, check this out!

[pedrobaeza]

Hola, Juani. Gracias por el PR. No debes cambiar cosas que no sean el parche exacto (como lo del default), ni añadir líneas que hagan fallar pre-commit. Mira que tanto eso como el CI están en rojo.

[JuaniFreedoo]

Hola, Juani. Gracias por el PR. No debes cambiar cosas que no sean el parche exacto (como lo del default), ni añadir líneas que hagan fallar pre-commit. Mira que tanto eso como el CI están en rojo.

Buenas Pedro, gracias por responder. Dejo el default como estaba, una pregunta ¿El return lo vuelvo a poner?

[pedrobaeza]

Sí, por favor, limita totalmente los cambios a lo estrictamente necesario.

[etobella]

Por mi OK,pero en un solo commit please

[pedrobaeza]

Según entiendo, con esto, si el pattern es poner el nombre comercial detrás, lo va a colocar detrás de la dirección y de todo, ¿no? Eso no sería del todo correcto.

[JuaniFreedoo]

Por mi OK,pero en un solo commit please

Ya he hecho el squash

Gracias

[JuaniFreedoo]

Según entiendo, con esto, si el pattern es poner el nombre comercial detrás, lo va a colocar detrás de la dirección y de todo, ¿no? Eso no sería del todo correcto.

Creía haberte entendido @pedrobaeza pero no sé exactamente a lo que te refieres, yo en las pruebas que he hecho lo veo bien tanto en formulario como en el PDF

[pedrobaeza]

Pones patrón %(name)s (%(comercial_name)s).

En caso de direcciones completas, va a salir:

Empresa
Calle lo que sea
Ciudad, NNNNN, País (Nombre comercial)

en lugar de:

Empresa (Nombre comercial)
Calle lo que sea
Ciudad, NNNNN, País

[JuaniFreedoo]

Pones patrón %(name)s (%(comercial_name)s).

En caso de direcciones completas, va a salir:

Empresa
Calle lo que sea
Ciudad, NNNNN, País (Nombre comercial)

en lugar de:

Empresa (Nombre comercial)
Calle lo que sea
Ciudad, NNNNN, País

Está saliendo bien, (Nombre Comercial) Empresa , yo al menos lo veo bien
A ver si lo prueba alguien más y lo vemos

[pedrobaeza]

Juani, fíjate en el primer paso. Tienes que dar la vuelta al patrón.

[JuaniFreedoo]

Juani, fíjate en el primer paso. Tienes que dar la vuelta al patrón.

Te refieres a que le pase así el diccionario?

Perdona si no termino de enterder, pero en principio daría igual si lo estamos cogiendo arriba desde el parámetro, no?

[pedrobaeza]

Primero replica el error con los pasos que comento a ver si es real o no. Luego la solución será más compleja.