<div dir="ltr">This is a great idea - but demands plenty of time to achieve across the product <div>and extensions already developed.For now tuning PHP configuration looks better.<div><br></div><div>By all means continue with the changes - we would be happy to work on the merge-request.</div><div><br></div><div>Regards,</div><div>Prasad</div></div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr">--</div><div dir="ltr"><span style="border-collapse:collapse;font-family:arial,sans-serif;font-size:13px"><a href="http://www.facebook.com/vtiger" style="color:rgb(0,0,204)" target="_blank">FB</a> I </span><a href="http://twitter.com/vtigercrm" style="color:rgb(0,0,204)" target="_blank">Twit</a> I <a href="https://www.linkedin.com/company/1270573?trk=tyah" style="color:rgb(0,0,204)" target="_blank">LIn</a> I <a href="https://blogs.vtiger.com" style="color:rgb(0,0,204)" target="_blank">Blog</a> I <a href="https://www.vtiger.com/" style="color:rgb(0,0,204)" target="_blank">Website</a></div></div></div></div></div></div></div></div></div></div></div>
<br><div class="gmail_quote">On Mon, Feb 1, 2016 at 10:41 PM, Alan Bell <span dir="ltr"><<a href="mailto:alan.bell@libertus.co.uk" target="_blank">alan.bell@libertus.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">excellent idea, something I have been meaning to do for ages, I am sure that in some places, in the tighter loops, the process of evaluating the error and deciding whether to log it probably causes a performance hit, so fixing this probably makes things faster. It certainly makes real problems easier to find if you can turn on error reporting and not get tons of cruft from minor issues.<div><div class="h5"><br>
<br>
<br>
On 01/02/16 17:02, Tim Mohrbach wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
As everyone knows, vtiger is throwing notices, warnings, deprecation warnings and strict errors all over the place. I always thought it would improve the overall quality of the project if the code lines which cause this were improved someday. I was a little bored in the last hour (don’t ask) and tackled all of those notices that were thrown only when vtiger is being opened the very first time, for installation. It’s only a couple of changes and they were gone.<br>
<br>
@vtiger core developer team: Now I’m wondering if this would actually be of any interest to the vtiger team? Would such changes actually stand any chance of making it into the master?<br>
@vtiger core developer team: Also, I’m wondering which way of securing the lines would be preferred. I used a few different approaches, sometimes declaring a return variable beforehand with default values, sometimes returning a ternary expression, some changes make the code longer, some easier to read, some are obviously dull, but please could you comment which way of securing those code lines you would prefer?<br>
@Everyone else: what do you think about this topic? Interesting, boring, good, stupid?<br>
<br>
So if I’m wasting my time here, please let me know then I won’t do this again next time I’m bored, but in case you are interested, let me know some feedback on the way those things should be adjusted.<br>
<br>
I commited all changes to make the install start page notice and warning free to this branch: <a href="http://code.vtiger.com/preexo/vtigercrm/commit/7f7486074d0fb4549ca9524cf36f65d8e4bd5bab" rel="noreferrer" target="_blank">http://code.vtiger.com/preexo/vtigercrm/commit/7f7486074d0fb4549ca9524cf36f65d8e4bd5bab</a><br>
<br>
<br>
<br>
_______________________________________________<br>
<a href="http://www.vtiger.com/" rel="noreferrer" target="_blank">http://www.vtiger.com/</a><br>
</blockquote>
<br></div></div>
_______________________________________________<br>
<a href="http://www.vtiger.com/" rel="noreferrer" target="_blank">http://www.vtiger.com/</a></blockquote></div><br></div>