Bug reports
Version 3.x > Bug reports > FIX: Minor HTML render bug in CMS.CMSEcommerce.OrderInfoProvider.GetProductList View modes: 
User avatar
Member
Member
Mufasa - 2/3/2009 4:40:33 PM
   
FIX: Minor HTML render bug in CMS.CMSEcommerce.OrderInfoProvider.GetProductList
The default code that renders the HTML for the product list for an invoice renders too many table cells in the header and footer rows if the renderDiscount flag is false. The lines:

builder.Append(" <thead><tr><th colspan=\"6\" class=\"headerBorder\"> </tr><tr>");
builder.Append(" <td colspan=\"6\" class=\"bottomBorder\"> ");

should both replace the string part: \"6\"

with \"" + (5 + (renderDiscount ? 1 : 0)) + "\"

Example: builder.Append(" <td colspan=\"" + (5 + (renderDiscount ? 1 : 0)) + "\" class=\"bottomBorder\"> ");


A minor bug, but I thought I'd mention it so I don't have to fix it again for my sites in the future. Besides, HTML e-mail clients are horrible with rendering even valid HTML, let alone HTML that isn't quite perfect.

User avatar
Kentico Consulting
Kentico Consulting
kentico_borisp - 2/4/2009 4:53:54 AM
   
RE:FIX: Minor HTML render bug in CMS.CMSEcommerce.OrderInfoProvider.GetProductList
Hello,

Thank you for the report, this issue should be fixed in the 4.0 version.

Best Regards,
Boris Pocatko