account_tax_include module in branch addons was removed on 28/12/2010

[OpenBMS JSC]

account_tax_include module in branch addons was removed on 28/12/2010, although the document (http://doc.openerp.com/technical_guide/account_tax_include.html#module-account_tax_include) says this is an official and quality certified module.

        ------------------------------------------------------------
        revno: 3363.111.101 [merge]
        committer: <email address hidden>
        branch nick: ron-dev-addons2
        timestamp: Tue 2010-12-28 14:58:40 +0530
        message:
          [MERGE]trunk-dev-addons2
        removed:
          account_tax_include/

Is there any reason that this is no longer available in branch addons?

[Raphaël Valyi - http://www.akretion.com (rvalyi)]

Phong: the short answer is that what was doing that module regarding to
included taxes (eg limited support) is now included in the core account of
OpenERP v6 module, it's even a bit better and cleaner though it might still
not be enough in may countries (for instance in Brazil we had to override
its computation methods).
So I guess that in theory, if you stick on v5, you stick with the module.
Now if you have a maintenance contract with OpenERP to migrate to v6, then
you will be able to just drop the module.

On Thu, Dec 30, 2010 at 5:06 PM, Phong Nguyen <email address hidden>wrote:

> Public bug reported:
>
> account_tax_include module in branch addons was removed on 28/12/2010,
> although the document
> (http://doc.openerp.com/technical_guide/account_tax_include.html#module-
> account_tax_include) says this is an official and quality certified
> module.
>
> ------------------------------------------------------------
> revno: 3363.111.101 [merge]
> committer: <email address hidden>
> branch nick: ron-dev-addons2
> timestamp: Tue 2010-12-28 14:58:40 +0530
> message:
> [MERGE]trunk-dev-addons2
> removed:
> account_tax_include/
>
> Is there any reason that this is no longer available in branch addons?
>
> ** Affects: openobject-addons
> Importance: Undecided
> Status: New
>
> --
> You received this bug notification because you are subscribed to
> OpenObject Addons.
> https://bugs.launchpad.net/bugs/695771
>
> Title:
> account_tax_include module in branch addons was removed on 28/12/2010
>
> Status in OpenObject Addons Modules:
> New
>
> Bug description:
> account_tax_include module in branch addons was removed on 28/12/2010,
> although the document (
> http://doc.openerp.com/technical_guide/account_tax_include.html#module-account_tax_include)
> says this is an official and quality certified module.
>
> ------------------------------------------------------------
> revno: 3363.111.101 [merge]
> committer: <email address hidden>
> branch nick: ron-dev-addons2
> timestamp: Tue 2010-12-28 14:58:40 +0530
> message:
> [MERGE]trunk-dev-addons2
> removed:
> account_tax_include/
>
> Is there any reason that this is no longer available in branch addons?
>
>
>
>
>

[OpenBMS JSC (openbmsjsc)]

Raphaël, thank you for your quick answer. I think it'd be better if there are a clear noes about what will be dropped and reason why, and the old modules can be accessible some where during the migration/clean up, just like a deprecation feature. The reason I brought this up is to avoid the confusion between controlled removal and unintended regression bugs introduced during the merges of various branches (e.g. issues recently introduced between 4067->4077, that breaks many things, and holding back testing)

Is the feature of account_tax_include available in the core account module yet, or it's something that has not been committed and merge to the main trunk?

[Fabien (Open ERP) (fp-tinyerp)]

Available in trunk

[OpenBMS JSC (openbmsjsc)]

Hi,

I tried with the latest code from repository and now I see lots of following lines in the logfile, and the product doesn't have a checked-box to indicate price is tax-included as previously.

[2011-01-01 11:52:38,819][crdev] DEBUG:addons.account.account:Status: SELECT
[2011-01-01 11:53:13,601][crdev] WARNING:warning:Deprecated, use compute_all(...)['taxes'] instead of compute(...) to manage prices with tax included

Please advise how to work around this?