<div dir="ltr">Thanks a lot for the feedback! Also really cool to see so much positive feedback! <div>I wasn't exactly sure about that topic, but this feedback encourages me to spend some time on it every now and then. I also think this will improve quality and performance and I'm happy to contribute some time. The process will take a while and will be done gradually...</div><div><br></div><div>I started a [WIP] merge request, open for anyone to contribute, review, comment: <a href="http://code.vtiger.com/vtiger/vtigercrm/merge_requests/25">http://code.vtiger.com/vtiger/vtigercrm/merge_requests/25</a> </div></div><div class="gmail_extra"><br><div class="gmail_quote">On 2 February 2016 at 18:40, Sutharsan Jeganathan <span dir="ltr"><<a href="mailto:ajstharsan@gmail.com" target="_blank">ajstharsan@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div><div>Hi Tim<br><br></div>What I feel is the "Warning" is the essential stuff has to be fixed. Also in PHP7 the warnings are some how merging with Fatal errors. Others such as strict and notices might not be serious issue, if disabled at php.ini, but anyway impact on memory usages, thus performance too.<br><br></div>I think  code optimization (and cleanup too) has to be considered as a separate project, since there are thousands of php files scattered with different standards.<br><br></div>Thanks<br></div>Sutharsan Jeganathan<br></div><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Tue, Feb 2, 2016 at 3:10 PM, Stacey Johnson <span dir="ltr"><<a href="mailto:stacey.johnson110@gmail.com" target="_blank">stacey.johnson110@gmail.com</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5"><div dir="ltr">Great stuff Tim. Lets watch merging performance ..<div><br></div><div>Stacey</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Feb 2, 2016 at 12:02 AM, Tim Mohrbach <span dir="ltr"><<a href="mailto:preexo@googlemail.com" target="_blank">preexo@googlemail.com</a>></span> 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></blockquote></div><br></div>
<br></div></div>_______________________________________________<br>
<a href="http://www.vtiger.com/" rel="noreferrer" target="_blank">http://www.vtiger.com/</a><br></blockquote></div><br></div>
<br>_______________________________________________<br>
<a href="http://www.vtiger.com/" rel="noreferrer" target="_blank">http://www.vtiger.com/</a><br></blockquote></div><br></div>