[master][14.0][13.0] [REGRESSION] "Label" instead of (product) "Description" on invoices

[primes2h]

Impacted versions:

master, 14.0, 13.0

Steps to reproduce:

Create an invoice, add a line, select a product.

Current behavior:

Product description column is named "Label".

This is a regression.
In any other documents (RFQ, PO, SO, etc.) (all versions) and invoices (v.12.0), the column is correctly named "Description".

Expected behavior:

Product description column must be named "Description".

[pedrobaeza]

I personally don't consider that a regression, as now the invoice is a journal entry, and the popup form is a general form view for journal items. Being an understandable label similar to description, creating an specific view for that and all the needed stuff for showing it, it's oversized for the benefit.

Anyways, I let others to give their opinion.

[primes2h]

I personally don't consider that a regression, as now the invoice is a journal entry, and the popup form is a general form view for journal items. Being an understandable label similar to description,

This is the main point, It's not so clear from a user point of view IMO.

If a user wants to add a new invoice line (w/o selecting a product), he'd expect to enter a "Description of the line" not a "Label" of the line. (as it is on SO, Quotations, etc.)

creating an specific view for that and all the needed stuff for showing it, it's oversized for the benefit.

Why would you need a specific view for that?

Changing this string should be enough:

odoo/addons/account/models/account_move.py

Line 3191 in 10b5a5f

name = fields.Char(string='Label', tracking=True)

[pedrobaeza]

Why would you need a specific view for that?

Changing this string should be enough:

odoo/addons/account/models/account_move.py

Line 3191 in 10b5a5f

name = fields.Char(string='Label', tracking=True)

No, because that string is shared for journal items too.

[Yenthe666]

& changing labels in stable is generally not allowed because it breaks translations their compatibility :)

[primes2h]

Why would you need a specific view for that?
Changing this string should be enough:

odoo/addons/account/models/account_move.py

Line 3191 in 10b5a5f

name = fields.Char(string='Label', tracking=True)

No, because that string is shared for journal items too.

Ok, but using "Description" for journal items too seems good to me, much better than having Label on invoice line.
What do you think?

[pedrobaeza]

I personally prefer the other.

[primes2h]

& changing labels in stable is generally not allowed because it breaks translations their compatibility :)

Sure, but not in master branch, AFAIK. ;-)
P.S. i could open a PR if needed.

[Yenthe666]

Do feel free to make a PR against master, then the accounting team from Odoo can make the final call :)