<div dir="ltr">Hi Ruben,<div><br></div><div>Thanks! for the proposed coding guideline on removal of php closing tag we will review this. Please do share if you have any supporting documentation.</div><div><br></div><div>I am not clear on how this is impacting the excel files download, Please do describe the issue in detail. Reporting it on <a href="http://code.vtiger.com">code.vtiger.com</a> with an explanation will be good.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Sep 4, 2020 at 3:57 AM Rubén A. Estrada Orozco <<a href="mailto:rulotec1@gmail.com">rulotec1@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Hi,<div><br></div><div>Today I had to fix a  problem with excel files downloaded from the documents module.  The files didn't display correctly, just a bunch of unreadable characters like these:</div><div><br></div><div><div><img src="cid:ii_kende2gz0" alt="image.png" width="542" height="239"><br></div></div><div><br></div><div>The problem was that there was a new line at the end of the config.inc.php file.</div><div><br></div><div>Wouldn't it be a good idea to get rid of the closing "?>" part of the php tag?</div><div><br></div><div>I think that in general it would be good to avoid closing php tags. There are projects like prestashop that set it as a coding guideline.</div><div><br></div><div><br></div><div><br clear="all"><div><div dir="ltr"><div dir="ltr">Saludos<div><br></div><div>Rubén</div></div></div></div></div></div>
_______________________________________________<br>
<a href="http://www.vtiger.com/" rel="noreferrer" target="_blank">http://www.vtiger.com/</a></blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr">With<br>Best Regards<br>Uma.S<br><div>Vtiger Team</div></div></div>