[17.0][FIX] l10n_es_partner: Show comercial

[david-s73]

Se ha tenido que quitar el context en el if ya que al contener el context no añadia el nombre comercial en el nombre completo, entonces al hacer la busqueda de contactos no se podia buscar por el comercial ya que no lo contenia en nombre comercial.

[OCA-git-bot]

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

[pedrobaeza]

Pero si quitas el contexto saldrá en sitios donde no debe... La solución debería ser pasar ese contexto cuando se hace la búsqueda (name_search?).

[david-s73]

@pedrobaeza
En la versión 16.0 el filtro que se utilizaba para buscar el nombre del cliente utiliza en display_name mientras que en la 17.0 utiliza el complete_name, entonces si no le pasas el comercial al complete_name no encontrara el contacto ya que el complete_name no contiene el comercial

16.0:

17.0:

En odoo base cuando hacen el compute del complete_name quitan los context que pudiera contener entonces nunca entraría en el if con ese contexto.

https://github.com/odoo/odoo/blob/17.0/odoo/addons/base/models/res_partner.py#L363

[david-s73]

@pedrobaeza He hecho cambios en el código

[pedrobaeza]

Dime por favor el caso que daba problemas antes para probarlo ahora.

[david-s73]

Por ejemplo tenia varios clientes que tenia como comercial "Pegaso" y al hacer la búsqueda por el nombre simplemente no lo encontraba, como lo explique arriba esta funcionalidad se encontraba en la versión 16 pero se ha perdido en esta versión

[pedrobaeza]

Hola, ya he reproducido el problema:

• Voy a un contacto.
• Coloco un nombre comercial.
• Me voy a crear una factura.
• Busco en el campo cliente por ese nombre comercial, y no aparece.

Pero he probado en el runboat de este PR, y sigue ocurriendo lo mismo.

[david-s73]

Si los contactos ya existen se tiene que recalcular el complete_name ya que al estar creado no lo contiene pero si crea un nuevo contacto si que añade al complete_name el comercial

[pedrobaeza]

Pero entonces habrá que solucionar ese problema también, incluyendo por ejemplo comercial en el depends de complete_name.

[david-s73]

@pedrobaeza He hecho un migrations para que se recalcule el complete_name de todos los clientes los cuales tengan el campo comercial con información asi que los clientes que ya estaban creados tendran el comercial en el complete_name, ademas de añadir en el @api.depends el comecial asi cuando se cambien se calculara el complete_name

[pedrobaeza]

Hola, David, gracias por los cambios. Probando en runboat, el caso ya funciona, pero existe un problema de concepto, y es que esto depende de si se ha puesto que en el nombre mostrado salga el nombre comercial, pero se puede deshabilitar el mismo con el parámetro de sistema, y que entonces deje de funcionar.

Creo que el cambio hecho en la migración a 17 no es correcto, y se debería volver a un sistema en el que en la búsqueda se incluyera el nombre comercial sin importar si se muestra o no.

[david-s73]

Los cambios son similares a como se hace en la versión 16.0 pero en vez de utilizar el complete_name en la versión anterior se utiliza el display_name, utilizando también el parámetro del sistema l10n_es_partner.name_pattern, la única solución que podíamos hacer es añadir en la búsqueda de nombre el comercial por si el parámetro del sistema se deshabilita

[david-s73]

@pedrobaeza He cambiado el manifest por el post-migration ademas he visto un par de ejemplos que cambian el manifest por el migrations.

1. https://github.com/OCA/l10n-spain/pull/3867/files
2. https://github.com/OCA/l10n-spain/pull/3554/files