[6.0.3] Incorrect Variable naming in product/pricelist.py - price_get_multi

Hi this bit of code, while it doesn't cause a bug is really confusing. Bear in mind this is in an awful function of 98 statements and 54 local variables, so it is hard enough to understand already.

The variable pricelist_version_ids is unnecessary, but even if it were necessary, it should correctly be called pricelist_ids. In fact the whole first block of code could just be reduced to an - if not pricelist_ids:
pricelist_ids = self.pool.get('product.pricelist').search(cr, uid, [], context=context).

next we have plversions_search_args. I get why it has been seperated out, but it is a bit unnecessary as well. It is only used one time and only on the next line so why create the variable.

Now there is plversions_ids, which should more correctly be called pricelist_version_ids (but that was already taken unnecessarily as per above) but I suppose it is still explanatory.

       if pricelist_ids:
            pricelist_version_ids = pricelist_ids
        else:
            # all pricelists:
            pricelist_version_ids = self.pool.get('product.pricelist').search(cr, uid, [], context=context)

        pricelist_version_ids = list(set(pricelist_version_ids))
        plversions_search_args = [
            ('pricelist_id', 'in', pricelist_version_ids),
            '|',
            ('date_start', '=', False),
            ('date_start', '<=', date),
            '|',
            ('date_end', '=', False),
            ('date_end', '>=', date),
        ]

        plversion_ids = product_pricelist_version_obj.search(cr, uid, plversions_search_args)