openerp-community-reviewer team mailing list archive

[Leonardo Pistone - camptocamp <leonardo.pistone@xxxxxxxxxxxxxx>]

Your point makes sense, Romain.

I would rather leave it as it is, since:

1. This is just a refactor with a few tests added. It is probably better to keep the existing functionality for now.

2. I'm not sure that can happen often in practice, and the implementation could use string manipulation (brittle) or new field (complex).

Thanks!
-- 
https://code.launchpad.net/~camptocamp/purchase-wkfl/7.0-merge-po-hooks-lep/+merge/216745
Your team Purchase Core Editors is subscribed to branch lp:purchase-wkfl.