Merge lp:~fabien-morin/unifield-server/fm-us-3142 into lp:unifield-server

Proposed by jftempo
Status: Merged
Merged at revision: 4453
Proposed branch: lp:~fabien-morin/unifield-server/fm-us-3142
Merge into: lp:unifield-server
Diff against target: 31 lines (+4/-4)
2 files modified
bin/addons/consumption_calculation/report/product_likely_expire_xls.mako (+1/-1)
bin/addons/report_webkit/webkit_report.py (+3/-3)
To merge this branch: bzr merge lp:~fabien-morin/unifield-server/fm-us-3142
Reviewer Review Type Date Requested Status
UniField Reviewer Team Pending
Review via email: mp+327968@code.launchpad.net
To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'bin/addons/consumption_calculation/report/product_likely_expire_xls.mako'
--- bin/addons/consumption_calculation/report/product_likely_expire_xls.mako 2014-03-14 16:20:38 +0000
+++ bin/addons/consumption_calculation/report/product_likely_expire_xls.mako 2017-07-24 15:33:28 +0000
@@ -131,7 +131,7 @@
131 <Cell ss:StyleID="line"><Data ss:Type="String">${formatLang(i.available_qty) or 0.00} (${(formatLang(i.expired_qty) or 0.00)})</Data></Cell>131 <Cell ss:StyleID="line"><Data ss:Type="String">${formatLang(i.available_qty) or 0.00} (${(formatLang(i.expired_qty) or 0.00)})</Data></Cell>
132 % endif132 % endif
133 % if not i.expired_qty:133 % if not i.expired_qty:
134 <Cell ss:StyleID="line"><Data ss:Type="Number">${formatLang(i.available_qty) or 0.00}</Data></Cell>134 <Cell ss:StyleID="line"><Data ss:Type="Number">${i.available_qty or 0.00}</Data></Cell>
135 % endif135 % endif
136% endfor136% endfor
137<Cell ss:StyleID="line_number"><Data ss:Type="Number">${line.in_stock or 0.}</Data></Cell>137<Cell ss:StyleID="line_number"><Data ss:Type="Number">${line.in_stock or 0.}</Data></Cell>
138138
=== modified file 'bin/addons/report_webkit/webkit_report.py'
--- bin/addons/report_webkit/webkit_report.py 2017-05-10 11:55:36 +0000
+++ bin/addons/report_webkit/webkit_report.py 2017-07-24 15:33:28 +0000
@@ -513,11 +513,11 @@
513 namespaces=namespaces)513 namespaces=namespaces)
514 for cell in number_cells:514 for cell in number_cells:
515 # if space in the in Numbers, remove them515 # if space in the in Numbers, remove them
516 forbidden_chars = [' ', u'\xa0']516 forbidden_chars = [' ', u'\xc2\xa0', u'\xa0']
517 for char in forbidden_chars:517 for char in forbidden_chars:
518 if isinstance(cell.text, str) and char in cell.text:518 if isinstance(cell.text, basestring) and char in cell.text:
519 error_message = 'Line %s of document %s is corrupted, a '\519 error_message = 'Line %s of document %s is corrupted, a '\
520 'Number cannot contain characters or spaces: %s' % \520 'Number cannot contain characters or spaces: %r' % \
521 (cell.sourceline, report_name, cell.text)521 (cell.sourceline, report_name, cell.text)
522 logger.warning(error_message)522 logger.warning(error_message)
523 cell.text = cell.text.replace(char, '')523 cell.text = cell.text.replace(char, '')

Subscribers

People subscribed via source and target branches