Code review comment for lp:~openerp-dev/openobject-addons/trunk-bug-794412-ron

if order.shipped:
   this is generic as it check if the products or services (or other future type of product) are delivered.

for picking in order.picking_ids:
   this only works for stockable products which is not good.

Several modules can inherit and enhance the order.shipped.