Review: Needs Information Hello Zehk,
It seems a good work but I have few doubts... Why do you need to add this extra check for 'posix' and for file types ['xdg-open', 'gnome-open', 'kde-open'] dont we have this check in _findPDFOpener except for the 'kde-open'opener (which can be added there itself) at line 107. AFAIS the current system do make a check for the applications to be loaded to open the files based on their types or am I missing something ? Thanks, -- https://code.launchpad.net/~openerp-community/openobject-client/zehk_use-of-xdg-open/+merge/77333 Your team OpenERP Community is subscribed to branch lp:~openerp-community/openobject-client/zehk_use-of-xdg-open. _______________________________________________ Mailing list: https://launchpad.net/~openerp-community Post to : [email protected] Unsubscribe : https://launchpad.net/~openerp-community More help : https://help.launchpad.net/ListHelp
