openerp-community-reviewer team mailing list archive

[Alejandro Santana <alejandrosantana@xxxxxxxxx>]

Alejandro Santana has proposed merging lp:~alejandrosantana/ocb-addons/6.1-ocb-addons-fix-product-label-barcode-populated-with-wrong-data into lp:ocb-addons/6.1.

Commit message:
[FIX] Fixes product label report barcode field: When printing product labels, the barcode is populated with Reference data instead of EAN13 data. Fixes bug: https://bugs.launchpad.net/openobject-addons/+bug/1249483

Requested reviews:
  OpenERP Community Backports Team (ocb)

For more details, see:
https://code.launchpad.net/~alejandrosantana/ocb-addons/6.1-ocb-addons-fix-product-label-barcode-populated-with-wrong-data/+merge/194613

[FIX] Fixes product label report barcode field: When printing product labels, the barcode is populated with Reference data instead of EAN13 data. Fixes bug: https://bugs.launchpad.net/openobject-addons/+bug/1249483

This fix set the correct variable to barcode, changing:
 <barCode><xsl:value-of select="code"/></barCode>
with:
<barCode><xsl:value-of select="ean13"/></barCode>
-- 
https://code.launchpad.net/~alejandrosantana/ocb-addons/6.1-ocb-addons-fix-product-label-barcode-populated-with-wrong-data/+merge/194613
Your team OpenERP Community Backports Team is requested to review the proposed merge of lp:~alejandrosantana/ocb-addons/6.1-ocb-addons-fix-product-label-barcode-populated-with-wrong-data into lp:ocb-addons/6.1.

=== modified file 'product/report/product_label.xsl'
--- product/report/product_label.xsl	2011-01-14 00:11:01 +0000
+++ product/report/product_label.xsl	2013-11-09 00:21:31 +0000
@@ -70,7 +70,7 @@
 				</td>
 			</tr><tr>
 				<td>
-				<barCode><xsl:value-of select="code"/></barCode>
+				<barCode><xsl:value-of select="ean13"/></barCode>
 				</td><td>
 					<para style="nospace"><xsl:value-of select="product"/></para><xsl:text>, </xsl:text>
 					<para style="nospace"><xsl:value-of select="variant"/></para>