<div dir="ltr">Thanks! Henry and Ruben for your inputs, We will review.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Sep 4, 2020 at 9:30 PM 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">I've opened the issue here: <a href="https://code.vtiger.com/vtiger/vtigercrm/issues/1477" target="_blank">https://code.vtiger.com/vtiger/vtigercrm/issues/1477</a><div><br></div><div>Thanks for the links Henry!</div><div><br clear="all"><div><div dir="ltr"><div dir="ltr">Saludos<div><br></div><div>Rubén</div></div></div></div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Sep 4, 2020 at 7:19 AM Henry Cumbicus Rivera <<a href="mailto:hcumbicusr@gmail.com" target="_blank">hcumbicusr@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">I agree with Rubén, in some cases it caused me problems and the solution was simply to remove the closing tag.<br><br><div>Cases and documentation:<br><div><a href="https://www.php.net/basic-syntax.phptags" target="_blank">https://www.php.net/basic-syntax.phptags</a><br></div><div><a href="https://www.php.net/basic-syntax.instruction-separation" target="_blank">https://www.php.net/basic-syntax.instruction-separation</a><br></div><div><a href="https://stackoverflow.com/questions/4410704/why-would-one-omit-the-close-tag#:~:text=The%20reason%20you%20should%20leave,not%20add%20extra%20white%2Dspace" target="_blank">https://stackoverflow.com/questions/4410704/why-would-one-omit-the-close-tag#:~:text=The%20reason%20you%20should%20leave,not%20add%20extra%20white%2Dspace</a>.<br></div><div><a href="https://gulvi.com/serie/trucos-php/capitulo/por-que-no-poner-etiqueta-cierre-php" target="_blank">https://gulvi.com/serie/trucos-php/capitulo/por-que-no-poner-etiqueta-cierre-php</a><br></div><div><br></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">El vie., 4 sept. 2020 a las 1:53, Uma S (<<a href="mailto:uma.s@vtiger.com" target="_blank">uma.s@vtiger.com</a>>) escribió:<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 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" target="_blank">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" target="_blank">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"><div dir="ltr">With<br>Best Regards<br>Uma.S<br><div>Vtiger Team</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"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><br><div>---------------------------------------------------------------</div><div>Henry C.</div><div>Tel.: 956727976</div></div></div></div></div></div></div></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>
<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>