Code review comment for lp:~florent.x/openobject-server/trunk-tools-file_open

Revision history for this message
Olivier Dony (Odoo) (odo-openerp) wrote :

Ouch, thanks for taking a stab at that [awful] legacy code (dating back from 2007[1]).

As I mentioned on the bug, I have a work in progress to change the file location mechanism to give higher priority to extracted/directory modules rather than zip files, for the various reasons exposed in comment #3.

This might cause a bit of conflict in your branch, but not too much normally.
I should be done very soon, and will notify you, so we can then proceed to review/merge the second part of the fix for bug 928376 - your patch.

Thanks!

[1] rev.632 revid:ced-96fb1d71e2b7282f46324766dd733aa1181e9e00 and related ones

« Back to merge proposal