[FIX] base: barcode qweb field correct API

[kebeclibre]

Before this commit, the barcode Qweb Field had a
option called "type", which meant what is the mapping between
numbers and barcode, that mapping is called a symbology
https://www.barcoding.com/resources/barcoding-basics/barcode-symbologies/

This option "type" collided with what the qweb engine does
at

qweb > _compile_directive_field
that calls
ir_qweb > _get_field

It resulted in a crash where type = "barcode" was not a valid symbology
to pass to the barcode lib

It is safe to say that this barcode widget was never user, otherwise it would have crashed

This commit changes that non-functioning API, and the barcode renders well

OPW 2066870

Description of the issue/feature this PR addresses:

Current behavior before PR:

Desired behavior after PR is merged:

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

[Whenrow]

Maybe you could also apply the change in

odoo/addons/web/controllers/main.py

Lines 1625 to 1640 in 3cadfeb

	@http.route(['/report/barcode', '/report/barcode/<type>/<path:value>'], type='http', auth="public")
	def report_barcode(self, type, value, width=600, height=100, humanreadable=0):
	"""Contoller able to render barcode images thanks to reportlab.
	Samples:
	<img t-att-src="'/report/barcode/QR/%s' % o.name"/>
	<img t-att-src="'/report/barcode/?type=%s&amp;value=%s&amp;width=%s&amp;height=%s' %
	('QR', o.name, 200, 200)"/>
	:param type: Accepted types: 'Codabar', 'Code11', 'Code128', 'EAN13', 'EAN8', 'Extended39',
	'Extended93', 'FIM', 'I2of5', 'MSI', 'POSTNET', 'QR', 'Standard39', 'Standard93',
	'UPCA', 'USPS_4State'
	:param humanreadable: Accepted values: 0 (default) or 1. 1 will insert the readable value
	at the bottom of the output image
	"""
	try:
	barcode = request.env['ir.actions.report'].barcode(type, value, width=width, height=height, humanreadable=humanreadable)

[kebeclibre]

@Whenrow
Why ? there is a bug there as well ?
There is no collision between keys named "type", so I guess it is overkill to change that there.
Besides, I'm afraid that would change the controller's API which actually works as it is

[nim-odoo]

But export the translations 😄

[kebeclibre]

robodoo r+

[nim-odoo]

robodoo r-
no merge, wait for the forwardport

[kebeclibre]

robodoo r+

[robodoo]

Merged at b062ed4, thanks!