openerp-community-reviewer team mailing list archive

Thread
Date

Re: [Merge] lp:~yann-papouin/ocb-server/6.1-bug-1029344-translation-sync-abstract-model into lp:ocb-server/6.1

To: Yann Papouin <yann.papouin@xxxxxxxxx>
From: "Holger Brunn \(Therp\)" <hbrunn@xxxxxxxx>
Date: Mon, 10 Feb 2014 15:08:06 -0000
In-reply-to: <20140206095926.19733.71957.launchpad@ackee.canonical.com>
Reply-to: mp+205104@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Needs Information code review

Wouldn't this let us miss translations in all non-auto (ie report) descendants of AbstractModel?

I don't think we can infer what we want from the classes properties alone. Why not ask the database if objmodel._table exists?
-- 
https://code.launchpad.net/~yann-papouin/ocb-server/6.1-bug-1029344-translation-sync-abstract-model/+merge/205104
Your team OpenERP Community Backports Team is subscribed to branch lp:ocb-server/6.1.

References

[Merge] lp:~yann-papouin/ocb-server/6.1-bug-1029344-translation-sync-abstract-model into lp:ocb-server/6.1
From: Yann Papouin, 2014-02-06