Razones frecuentes para el rechazo de un Pull Request con sus Soluciones

[christopherBinaural]

Agreguen partes de codigo, explicaciones cortas o cualquier cosa que alguien les haya comentado en una revision de codigo y como lo corrigieron.

Solo agreguen cosas que previamente no se encuentren en este discussion, evitemos duplicar.

Esta es como una guia de buenas practicas.

Objetivo del discussion

Cuando alguien nuevo del equipo necesite subir un Pull Request, pueda revisar una lista de posibles correcciones.

[christopherBinaural]

No utilizar self.env.company para obtener el valor de un campo de configuracion

Comentario: Cuando se quiere acceder a una variable en res.company desde res.partner u otro record que tenga el campo company_id (Es decir que sea multicompañia), es mejor acceder a la variable de la siguiente manera:

Si: self.company_id.sale_journal_id
No: self.env.company.sale_journal_id

Porque el company que se encuentra en el env es la compañia de la sesion actual del usuario, mientras que es mas seguro utilizar la misma compañia con la que fue creado el registros, la cual es la utilizada para el filtro de multicompañia.

[Carlos-Lin-Binaural]

Evitar las busquedas por ORM por el nombre, código o valores quemados

Tengan en cuenta que por mas que les diga el consultor que "Busquenlo por nombre, el cliente trabaja así", no le hagan caso y tomen como sugerencia hacerlo un campo Configuración Multicompañía, y allí puedan hacer su busqueda directamente por un solo campo y sí de repente llega a fallar por mala configuración u otra cosa sea por el lado del cliente o consultor

Manera Incorrecta:

Manera Correcta:

• En caso de que el modelo no posea un campo relacionado a la compañia (company_id (res.company)) pueden sacar la compañia actual usando el self.env.company
• Sí el modelo tiene el company_id ya tienen como acceder a la configuración de la compañia

[christopherBinaural]

Al comparar entre montos que poseen decimales utilizar el float_compare con el digits pasado por argumentos:

float_compare(move_line.qty_done, move_line.demanded_qty, precision_digits=digits) > 0:

Compara move_line.qty_done y move_line.demanded_qty con la precisión especificada por digits.

Devuelve 1 si move_line.qty_done es mayor que move_line.demanded_qty, -1 si es menor y 0 si son iguales.

Ejemplo

from odoo.tools.float_utils import float_compare
                float_compare(
                    move_line.qty_done, move_line.demanded_qty, precision_digits=digits,
                    ...
                )
for move_line in self:
    if (round(move_line.qty_done, 2) > move_line.demanded_qty ):

Mejorable

Asi