Code review comment for lp:~bcim/account-invoicing/7.0-account_invoice_line_default_account

On 09/18/2013 02:28 PM, Ronald Portier (Therp) wrote:
> Just on the version numbering.
>
> It is needed the way it is. As it is the ONLY way of knowing that this module was adapted to and tested on the specified version.
>
If a module is in a 7.0 branch and has the 'installable' flag to True,
it means that it is adapted and tested on the branch's version.

> The core action of automatically adding a version number is a completely futile exercise, as it will be 7.0 on for all modules on an 7.0 installation and all 6.1 on a 6.1 installation etc.
>

I don't agree with that. "Bumping" the version has a - not futile ̣-
meaning. It, obviously, means that a new version is available, and when
a major version is upgraded, it warns about possible issues like the
need to upgrade the schema of the database. That's even used by the new
OpenERP Apps to inform the users that a new version of their module is
available.

An example in the official addons: the version of the stock module
displayed in OpenERP is: 7.0.1.1.1. In the manifest, the version is 1.1.1.

--
Guewen Baconnier
Business Solutions Software Developer

Camptocamp SA
PSE A, CH-1015 Lausanne
Phone: +41 21 619 10 39
Office: +41 21 619 10 10
http://www.camptocamp.com/