<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>