openerp-community-reviewer team mailing list archive

["Yannick Vaucher @ Camptocamp" <yannick.vaucher@xxxxxxxxxxxxxx>]

carrier_id isn't required.

If you are asking for that line:
151	+        license = picking.carrier_id.postlogistics_license_id

orm.browse_null

Implements __getattr__ that way.
    def __getattr__(self, name):
        return None  # XXX: return self ?

So even if carrier_id returns orm.browse_null asking for one of its attributes is None
-- 
https://code.launchpad.net/~camptocamp/carriers-deliveries/7.0-delivery_carrier_label_postlogistics-imp-licenses-yvr/+merge/202334
Your team Stock and Logistic Core Editors is requested to review the proposed merge of lp:~camptocamp/carriers-deliveries/7.0-delivery_carrier_label_postlogistics-imp-licenses-yvr into lp:carriers-deliveries.