openerp-community-reviewer team mailing list archive

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

Yannick Vaucher @ Camptocamp has proposed merging lp:~camptocamp/carriers-deliveries/7.0-fix-1296678 into lp:carriers-deliveries.

Requested reviews:
  Stock and Logistic Core Editors (stock-logistic-core-editors)
Related bugs:
  Bug #1296678 in Carriers And Deliveries Management: "Postlogistics Warnings are wrongly unwrapped"
  https://bugs.launchpad.net/carriers-deliveries/+bug/1296678

For more details, see:
https://code.launchpad.net/~camptocamp/carriers-deliveries/7.0-fix-1296678/+merge/212396

[FIX] postlogistics warnings unwrapping
-- 
https://code.launchpad.net/~camptocamp/carriers-deliveries/7.0-fix-1296678/+merge/212396
Your team Stock and Logistic Core Editors is requested to review the proposed merge of lp:~camptocamp/carriers-deliveries/7.0-fix-1296678 into lp:carriers-deliveries.

=== modified file 'delivery_carrier_label_postlogistics/postlogistics/web_service.py'
--- delivery_carrier_label_postlogistics/postlogistics/web_service.py	2014-03-21 16:09:52 +0000
+++ delivery_carrier_label_postlogistics/postlogistics/web_service.py	2014-03-24 12:04:00 +0000
@@ -383,7 +383,7 @@
                 })
 
             if hasattr(item, 'Warnings') and item.Warnings:
-                for warning in item.Warnings:
+                for warning in item.Warnings.Warning:
                     message = '[%s] %s' % (warning.Code, warning.Message)
                     warning_messages.append(message)