Review: Needs Information

Slight preference for setting image to NULL in that case. Otherwise, looks 
alright. Have you tried finding out what encoding the image is in, to see if 
you could recover the images? That is, if the images are rendered properly in 
the interface for those employees. Maybe you could post an example string in 
that case?


-- 
https://code.launchpad.net/~savoirfairelinux-openerp/openupgrade-addons/hr_employee_image/+merge/192728
Your team Savoir-faire Linux' OpenERP is subscribed to branch 
lp:~savoirfairelinux-openerp/openupgrade-addons/hr_employee_image.

-- 
Mailing list: https://launchpad.net/~savoirfairelinux-openerp
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~savoirfairelinux-openerp
More help   : https://help.launchpad.net/ListHelp

Reply via email to