<div dir="ltr"><span style="font-family:arial,helvetica,sans-serif">I did not understand about "common strategy" to update the code base.</span><div><span style="font-family:arial,helvetica,sans-serif"><span class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br></span></span></div><div><span style="font-family:arial,helvetica,sans-serif"><span class="gmail_default" style="font-family:arial,helvetica,sans-serif">You can create a project at <a href="http://code.vtiger.com">code.vtiger.com</a> and submit the changes.</span></span><span class="gmail_default" style="font-family:arial,helvetica,sans-serif"></span><span style="font-family:arial,helvetica,sans-serif"></span><span class="gmail_default" style="font-family:arial,helvetica,sans-serif"></span><span style="font-family:arial,helvetica,sans-serif"></span></div><div><span style="font-family:arial,helvetica,sans-serif">M<span class="gmail_default" style="font-family:arial,helvetica,sans-serif">R is merged to master.</span></span></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Aug 2, 2021 at 3:21 PM Sukhdev Mohan <<a href="mailto:s.mohan@myti.it">s.mohan@myti.it</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="auto"><div>I agree with Mariusz. My email was to inform and try to figure out a common strategy to update the code base.</div><div dir="auto"><br></div><div dir="auto"><br><div class="gmail_quote" dir="auto"><div dir="ltr" class="gmail_attr">Il lun 2 ago 2021, 11:16 Prasad <<a href="mailto:prasad@vtiger.com" target="_blank">prasad@vtiger.com</a>> ha scritto:<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"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">neverending story and issues pending for years... nothing has changed</blockquote><div> </div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Subjective opinion. No comments.<br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">If you have a MR feel free to contribute <a href="https://code.vtiger.com/vtiger/vtigercrm" rel="noreferrer" target="_blank">here</a>.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Aug 2, 2021 at 1:26 PM Mariusz Krzaczkowski | YetiForce Sp. z o.o. <<a href="mailto:m.krzaczkowski@yetiforce.com" rel="noreferrer" target="_blank">m.krzaczkowski@yetiforce.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 style="font-size:10pt;font-family:Verdana,Geneva,sans-serif">
<p>neverending story and issues pending for years... nothing has changed</p>
<div id="gmail-m_-2316224947220780494m_5129269139602956716gmail-m_1125499433335101044signature">--<br>
<div>
<div style="padding:0px 5px">
<div style="font-family:Tahoma,Geneva,sans-serif;font-size:12.5px;color:rgb(1,55,77);font-weight:200">Z poważaniem / Kind regards</div>
<div style="padding-top:7px;font-family:Tahoma,Geneva,sans-serif;font-size:21.67px;color:rgb(1,55,77);font-weight:bold">Mariusz Krzaczkowski</div>
<div style="padding-top:4px;font-family:Tahoma,Geneva,sans-serif;font-size:14.17px;color:rgb(147,207,216);font-weight:bold">Director of Product Development</div>
<div style="padding-top:8px;font-family:Tahoma,Geneva,sans-serif;font-size:10px;color:rgb(1,55,77);font-weight:400"><span style="margin-right:6px"><a style="color:rgb(1,55,77);text-decoration:none" rel="noreferrer">+48 884 998 123</a></span> | <span style="margin-left:6px"> <a style="color:rgb(1,55,77);text-decoration:none" rel="noreferrer">m.krzaczkowski@yetiforce.com</a></span></div>
<div style="padding-top:6px;padding-bottom:11px"><a style="font-family:Tahoma,Geneva,sans-serif;font-size:10px;color:rgb(1,55,77);font-weight:bold;text-decoration:none" rel="noreferrer">www.yetiforce.com</a></div>
<div style="width:70px;height:4px;background:rgb(235,235,235)"> </div>
<div style="padding-top:18px">
<table style="border-collapse:collapse;border-spacing:0px" border="0" cellspacing="0">
<tbody>
<tr>
<td><a style="text-decoration:none" rel="noreferrer"><img style="font-family: Tahoma, Geneva, sans-serif; font-weight: 400; font-size: 12px; color: rgb(1, 55, 77); border: none;" src="cid:17b0621d91ed5e51771" alt="YetiForce"></a></td>
<td><img src="cid:17b0621d91e6766c10a2" alt=""></td>
<td style="padding-right:12px"><a style="text-decoration:none" href="https://linkedin.com/company/yetiforce" rel="noreferrer noopener noreferrer" target="_blank"><img style="font-family: Tahoma, Geneva, sans-serif; font-weight: 400; font-size: 12px; color: rgb(1, 55, 77); border: none;" src="cid:17b0621d91e7d1e9a4a3" alt="linkedin"></a></td>
<td style="padding-right:12px"><a style="text-decoration:none" href="https://facebook.com/YetiForceCRM/" rel="noreferrer noopener noreferrer" target="_blank"><img style="font-family: Tahoma, Geneva, sans-serif; font-weight: 400; font-size: 12px; color: rgb(1, 55, 77); border: none;" src="cid:17b0621d91ef752438d4" alt="facebook"></a></td>
<td style="padding-right:12px"><a style="text-decoration:none" href="https://twitter.com/yetiforceen" rel="noreferrer noopener noreferrer" target="_blank"><img style="font-family: Tahoma, Geneva, sans-serif; font-weight: 400; font-size: 12px; color: rgb(1, 55, 77); border: none;" src="cid:17b0621d91e3e42cb755" alt="twitter"></a></td>
<td style="padding-right:12px"><a style="text-decoration:none" href="https://www.youtube.com/channel/UCnt7DLyYtcSBkC2sk2CJBXw" rel="noreferrer noopener noreferrer" target="_blank"><img style="font-family: Tahoma, Geneva, sans-serif; font-weight: 400; font-size: 12px; color: rgb(1, 55, 77); border: none;" src="cid:17b0621d91efa18efe26" alt="youtube"></a></td>
<td><a style="text-decoration:none" href="https://github.com/YetiForceCompany/YetiForceCRM" rel="noreferrer noopener noreferrer" target="_blank"><img style="font-family: Tahoma, Geneva, sans-serif; font-weight: 400; font-size: 12px; color: rgb(1, 55, 77); border: none;" src="cid:17b0621d91ed970f3a47" alt="github"></a></td>
</tr>
</tbody>
</table>
</div>
<div style="padding-top:13px;font-family:Tahoma,Geneva,sans-serif;font-size:10px;font-weight:bold;max-width:620px;border:none"><img title="Nagrody / Awards" src="cid:17b0621d91e1e5dbd358" alt="Nagrody / Awards: HIGH PERFORMER IN CRM CATEGORY, MOST AFFORDABLE CRM SYSTEM IN THE WORLD IN 2017, SUPPLIER OF THE YEAR 2017 IN THE IT CATEGORY, GUARANTEE OF USER SATISFACTION, RISING STAR AMONG CRM SYSTEMS, PLATINIUM MEDAL FROM THE SENSIOLABINSIGHTS"></div>
<div style="padding-top:6px;font-family:Tahoma,Geneva,sans-serif;font-size:9px;color:rgb(1,55,77);font-weight:200;font-style:italic;max-width:608px;line-height:1.2">The content of this message is confidential and must not be disclosed. If the recipient of this message is not the intended recipient, employee, or intermediary authorized to forward it to the intended recipient, please note that all distribution, copying, or any other use of this message is forbidden. If you received this message by mistake, please inform the sender immediately by responding to this message, and delete all copies of this message.</div>
</div>
</div>
</div>
<p><br></p>
<p id="gmail-m_-2316224947220780494m_5129269139602956716gmail-m_1125499433335101044reply-intro">W dniu 2021-08-02 09:13, Prasad napisał(a):</p>
<blockquote type="cite" style="padding:0px 0.4em;border-left:2px solid rgb(16,16,255);margin:0px">
<div id="gmail-m_-2316224947220780494m_5129269139602956716gmail-m_1125499433335101044replybody1">
<div dir="ltr">
<div style="font-family:arial,helvetica,sans-serif">MR Contributions are welcome.</div>
</div>
<br>
<div>
<div dir="ltr">On Mon, Aug 2, 2021 at 12:25 PM Mariusz Krzaczkowski | YetiForce Sp. z o.o. <<a href="mailto:m.krzaczkowski@yetiforce.com" rel="noreferrer noreferrer" target="_blank">m.krzaczkowski@yetiforce.com</a>> wrote:</div>
<blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div style="font-size:10pt;font-family:Verdana,Geneva,sans-serif">
<p>@Sukhdev Mohan</p>
<p>See here (you can see what needs to be improved)</p>
<p><a href="https://insight.symfony.com/projects/df801b71-93c7-42cc-b67c-eba1e9e02e37/analyses/1?status=stats" rel="noopener noreferrer noreferrer" target="_blank">https://insight.symfony.com/projects/df801b71-93c7-42cc-b67c-eba1e9e02e37/analyses/1?status=stats</a> </p>
<p><a href="https://insight.symfony.com/projects/df801b71-93c7-42cc-b67c-eba1e9e02e37/analyses/1" rel="noopener noreferrer noreferrer" target="_blank">https://insight.symfony.com/projects/df801b71-93c7-42cc-b67c-eba1e9e02e37/analyses/1</a> </p>
<div id="gmail-m_-2316224947220780494m_5129269139602956716gmail-m_1125499433335101044v1gmail-m_4038912516141398720signature">--<br>
<div>
<div style="padding:0px 5px">
<div style="font-family:Tahoma,Geneva,sans-serif;font-size:12.5px;color:rgb(1,55,77);font-weight:200">Z poważaniem / Kind regards</div>
<div style="padding-top:7px;font-family:Tahoma,Geneva,sans-serif;font-size:21.67px;color:rgb(1,55,77);font-weight:bold">Mariusz Krzaczkowski</div>
<div style="padding-top:4px;font-family:Tahoma,Geneva,sans-serif;font-size:14.17px;color:rgb(147,207,216);font-weight:bold">Director of Product Development</div>
<div style="padding-top:8px;font-family:Tahoma,Geneva,sans-serif;font-size:10px;color:rgb(1,55,77);font-weight:400"><span style="margin-right:6px"><a style="color:rgb(1,55,77);text-decoration:none" href="#m_-2316224947220780494_m_5129269139602956716_m_1125499433335101044_NOP" rel="noreferrer noreferrer">+48 884 998 123</a></span> | <span style="margin-left:6px"> <a style="color:rgb(1,55,77);text-decoration:none" href="#m_-2316224947220780494_m_5129269139602956716_m_1125499433335101044_NOP" rel="noreferrer noreferrer">m.krzaczkowski@yetiforce.com</a></span></div>
<div style="padding-top:6px;padding-bottom:11px"><a style="font-family:Tahoma,Geneva,sans-serif;font-size:10px;color:rgb(1,55,77);font-weight:bold;text-decoration:none" href="#m_-2316224947220780494_m_5129269139602956716_m_1125499433335101044_NOP" rel="noreferrer noreferrer">www.yetiforce.com</a></div>
<div style="width:70px;height:4px;background:rgb(235,235,235)"> </div>
<div style="padding-top:18px">
<table style="border-collapse:collapse;border-spacing:0px" border="0" cellspacing="0">
<tbody>
<tr>
<td><a style="text-decoration:none" href="#m_-2316224947220780494_m_5129269139602956716_m_1125499433335101044_NOP" rel="noreferrer noreferrer"><img style="font-family: Tahoma, Geneva, sans-serif; font-weight: 400; font-size: 12px; color: rgb(1, 55, 77); border: none;" alt="YetiForce"></a></td>
<td><img></td>
<td style="padding-right:12px"><a style="text-decoration:none" href="https://linkedin.com/company/yetiforce" rel="noopener noreferrer noreferrer" target="_blank"><img style="font-family: Tahoma, Geneva, sans-serif; font-weight: 400; font-size: 12px; color: rgb(1, 55, 77); border: none;" alt="linkedin"></a></td>
<td style="padding-right:12px"><a style="text-decoration:none" href="https://facebook.com/YetiForceCRM/" rel="noopener noreferrer noreferrer" target="_blank"><img style="font-family: Tahoma, Geneva, sans-serif; font-weight: 400; font-size: 12px; color: rgb(1, 55, 77); border: none;" alt="facebook"></a></td>
<td style="padding-right:12px"><a style="text-decoration:none" href="https://twitter.com/yetiforceen" rel="noopener noreferrer noreferrer" target="_blank"><img style="font-family: Tahoma, Geneva, sans-serif; font-weight: 400; font-size: 12px; color: rgb(1, 55, 77); border: none;" alt="twitter"></a></td>
<td style="padding-right:12px"><a style="text-decoration:none" href="https://www.youtube.com/channel/UCnt7DLyYtcSBkC2sk2CJBXw" rel="noopener noreferrer noreferrer" target="_blank"><img style="font-family: Tahoma, Geneva, sans-serif; font-weight: 400; font-size: 12px; color: rgb(1, 55, 77); border: none;" alt="youtube"></a></td>
<td><a style="text-decoration:none" href="https://github.com/YetiForceCompany/YetiForceCRM" rel="noopener noreferrer noreferrer" target="_blank"><img style="font-family: Tahoma, Geneva, sans-serif; font-weight: 400; font-size: 12px; color: rgb(1, 55, 77); border: none;" alt="github"></a></td>
</tr>
</tbody>
</table>
</div>
<div style="padding-top:13px;font-family:Tahoma,Geneva,sans-serif;font-size:10px;font-weight:bold;max-width:620px;border:none"><img title="Nagrody / Awards" alt="Nagrody / Awards: HIGH PERFORMER IN CRM CATEGORY, MOST AFFORDABLE CRM SYSTEM IN THE WORLD IN 2017, SUPPLIER OF THE YEAR 2017 IN THE IT CATEGORY, GUARANTEE OF USER SATISFACTION, RISING STAR AMONG CRM SYSTEMS, PLATINIUM MEDAL FROM THE SENSIOLABINSIGHTS"></div>
<div style="padding-top:6px;font-family:Tahoma,Geneva,sans-serif;font-size:9px;color:rgb(1,55,77);font-weight:200;font-style:italic;max-width:608px;line-height:1.2">The content of this message is confidential and must not be disclosed. If the recipient of this message is not the intended recipient, employee, or intermediary authorized to forward it to the intended recipient, please note that all distribution, copying, or any other use of this message is forbidden. If you received this message by mistake, please inform the sender immediately by responding to this message, and delete all copies of this message.</div>
</div>
</div>
</div>
<p><br></p>
<p id="gmail-m_-2316224947220780494m_5129269139602956716gmail-m_1125499433335101044v1gmail-m_4038912516141398720reply-intro">W dniu 2021-08-02 08:43, Prasad napisał(a):</p>
<blockquote style="padding:0px 0.4em;border-left:2px solid rgb(16,16,255);margin:0px">
<div id="gmail-m_-2316224947220780494m_5129269139602956716gmail-m_1125499433335101044v1gmail-m_4038912516141398720replybody1">
<div dir="ltr">
<div style="font-family:arial,helvetica,sans-serif">Great to hear that.</div>
</div>
<br>
<div>
<div dir="ltr">On Mon, Aug 2, 2021 at 11:28 AM Sukhdev Mohan <<a href="mailto:s.mohan@myti.it" rel="noreferrer noreferrer" target="_blank">s.mohan@myti.it</a>> wrote:</div>
<blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div dir="auto">I've just done a preliminary analysis. Work has yet to be started, given that there is a lot to work on, there will be MRs coming your way. It should be a team effort thoughm</div>
<br>
<div>
<div dir="ltr">Il lun 2 ago 2021, 07:42 Prasad <<a href="mailto:prasad@vtiger.com" rel="noreferrer noreferrer" target="_blank">prasad@vtiger.com</a>> ha scritto:</div>
<blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div dir="ltr">
<div style="font-family:arial,helvetica,sans-serif">Submit MR to increase the compatibility.</div>
</div>
<br>
<div>
<div dir="ltr">On Mon, Aug 2, 2021 at 2:59 AM Sukhdev Mohan <<a href="mailto:s.mohan@myti.it" rel="noreferrer noreferrer" target="_blank">s.mohan@myti.it</a>> wrote:</div>
<blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div>
<div>
<div dir="auto">Hi all,<br><br>I've done a static analysis for the new release. The idea is to highlight are of interventions, defining a starting point to decrease the technical debt this project has piled up.<br><br>I've tried to run the project (clean install) with php7.4 and mysql 8 + docker, install started unfortunately I had 504 during the installation but I'm working on it, when it's finished I'll be sharing my GitHub repo with a working docker file with LEMP. I've tried PHP8 too just out of curiosity. As expected it doesn't even launch the wizard: a lot of fatal errors. For the moment I can tell these 2:<br>1. PearDatabase: use of curly braces instead of []<br>2. AntlrLexer: them Ian class seems to be extending a base class but it's following the definition in the base class for matchAny and Re. I couldn't find the php version for this project, so if you guys wrote it, look at it.<br><br>Now coming back to static Analysis, I've done it with PHPInsights for the modules folder (launch this command from cam root folder - after install through composer - : php -d memory_limit=4000M ./vendor/bin/phpinsights analyse modules ), you can find it zipped as a json file, the tool did truncated some but since it's easy to launch please do relaunch it. - although it will take a bit... the generated file is about 85MB...<br><br>Some thoughts:<br>As expected there is no proper implementation of  OOP (No visibility keywords, no namespaces), there is extensive use of global variables and super global variables such as $_REQUEST - which is needless to say harmful. Other than this there are too many nested loops (in some part nested loop level is 7 whereas recommended is 2 or 3, this has huge implications on performance!), many declared and unused variables.<br><br>It's safe to say that if PHP7.4 is still letting some of the old practices go on, PHP8 will break the code badly, since most of the code is legacy. I was hoping that at least there would be composer but that's for the next release, hopefully.<br><br>What are your thoughts? How can we start to approach?</div>
</div>
<div><br>
<div><span style="color:rgb(198,198,198)"><strong><span>Sukhdev Mohan</span></strong></span><span></span><br><span style="color:rgb(153,153,153)"><strong><span>Developer</span></strong></span></div>
</div>
</div>
_______________________________________________<br><a href="http://www.vtiger.com/" rel="noopener noreferrer noreferrer" target="_blank">http://www.vtiger.com/</a></blockquote>
</div>
_______________________________________________<br><a href="http://www.vtiger.com/" rel="noopener noreferrer noreferrer" target="_blank">http://www.vtiger.com/</a></blockquote>
</div>
_______________________________________________<br><a href="http://www.vtiger.com/" rel="noopener noreferrer noreferrer" target="_blank">http://www.vtiger.com/</a></blockquote>
</div>
</div>
<br>
<div style="margin:0px;padding:0px;font-family:monospace">_______________________________________________<br><a href="http://www.vtiger.com/" rel="noopener noreferrer noreferrer" target="_blank">http://www.vtiger.com/</a></div>
</blockquote>
</div>
_______________________________________________<br><a href="http://www.vtiger.com/" rel="noopener noreferrer noreferrer" target="_blank">http://www.vtiger.com/</a></blockquote>
</div>
</div>
<br>
<div style="margin:0px;padding:0px;font-family:monospace">_______________________________________________<br><a href="http://www.vtiger.com/" rel="noopener noreferrer noreferrer" target="_blank">http://www.vtiger.com/</a></div>
</blockquote>
</div>
_______________________________________________<br>
<a href="http://www.vtiger.com/" rel="noreferrer noreferrer" target="_blank">http://www.vtiger.com/</a></blockquote></div>
_______________________________________________<br>
<a href="http://www.vtiger.com/" rel="noreferrer noreferrer" target="_blank">http://www.vtiger.com/</a></blockquote></div></div></div>
_______________________________________________<br>
<a href="http://www.vtiger.com/" rel="noreferrer" target="_blank">http://www.vtiger.com/</a></blockquote></div>