<div dir="ltr">Hi Martin,<br><div><br></div><div>That's a good idea.</div><div><br></div><div>I went through the reference which looks interesting.</div><div><br></div><div>Let's review and check how best we can include this to our product.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Oct 16, 2020 at 9:22 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">Hi,<div><br></div><div>I think the previous solution was better. It was good to have max_scheduled_workflows in config.inc.php. </div><div><br></div><div>in <span style="font-family:verdana,sans-serif">getMaxAllowedScheduledWorkflow</span><span style="font-family:verdana,sans-serif">s() having something like:</span></div><div><span style="font-family:verdana,sans-serif"><br></span></div><blockquote style="margin:0px 0px 0px 40px;border:none;padding:0px"><div><span style="font-family:verdana,sans-serif">if (empty(</span>max_scheduled_workflows)) {</div><div>   return 10;</div><div>}</div><div>return max_scheduled_workflows;</div></blockquote><div><span style="font-family:verdana,sans-serif"><br></span></div><div><span style="font-family:verdana,sans-serif">would have fixed the issue.</span></div><div><span style="font-family:verdana,sans-serif"><br></span></div><div><span style="font-family:verdana,sans-serif">As Alan mentioned, there are already variables that are editable in config.inc.php through "</span>Settings->Configuration Editor". Why not use that mechanism that already exists?</div><div><br></div><div>>> Let's introduce the api setMaxAllowedScheduledWorkflows() api.</div><div>Where should we then call the function? in config.inc.php?</div><div>It feels like we are overcomplicating the whole thing :(</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, Oct 16, 2020 at 9:12 AM Martin Allen <<a href="mailto:martin.allen@clystnet.com" target="_blank">martin.allen@clystnet.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"><div class="gmail_default" style="font-family:verdana,sans-serif">Would it be a good idea to start using a package like this that handles environment variables - something like <a href="https://github.com/vlucas/phpdotenv" target="_blank">https://github.com/vlucas/phpdotenv</a> which is used in many other PHP frameworks (Laravel etc). </div><div><div dir="ltr"><div dir="ltr"><div><div dir="ltr"><p style="font-size:12.8px;color:rgb(80,0,80)"><span style="color:navy;font-family:Verdana,sans-serif;font-size:10pt">Martin Allen</span></p><p style="font-size:12.8px;color:rgb(80,0,80)"><b style="font-size:12.8px"><span lang="EN-US" style="font-size:10pt;font-family:Verdana,sans-serif;color:rgb(51,51,153)">01392 248692 - Main Office<br>01392 690659 - Direct Line</span></b><br></p><p style="font-size:12.8px;color:rgb(80,0,80)"><b><span style="font-size:10pt;font-family:Verdana,sans-serif;color:rgb(51,51,153)"><img width="288" height="65" alt="signature2"></span></b><span style="color:rgb(31,73,125)"></span></p><p style="font-size:12.8px;color:rgb(80,0,80)"><span lang="EN-US" style="font-size:10pt;font-family:Verdana,sans-serif;color:navy">Have you visited our website recently? <a href="http://www.clystnet.com/" style="color:rgb(17,85,204)" target="_blank">https://www.clystnet.com</a></span><span style="color:rgb(31,73,125)"></span></p><p style="font-size:12.8px;color:rgb(80,0,80)"><span lang="EN-US" style="font-size:10pt;font-family:Verdana,sans-serif;color:navy"><br></span><span lang="EN-US" style="font-size:10pt;font-family:Verdana,sans-serif;color:rgb(153,153,153)">The information in this email is confidential If you are not the intended recipient, you must not read or use that information. This email and any attachments are believed to be virus free however no responsibility is accepted by Clystnet for any loss or damage arising in any way from receipt or use thereof. Clystnet Ltd (company reg number 7164503) is based at Silverdown Park, Fair Oak Close, Clyst Honiton, EX5 2UX</span></p></div></div></div></div></div><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, 16 Oct 2020 at 12:37, Uma S <<a href="mailto:uma.s@vtiger.com" target="_blank">uma.s@vtiger.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 Martin,<br><div><br></div><div>vglobals works with $GLOBALS of php.</div><div><br></div><div>>>A<span style="font-family:verdana,sans-serif">re you aiming to remove these variables from config.inc.php completely? </span></div><div><span style="font-family:verdana,sans-serif">I think we should not alter config.inc.php. For now '</span>max_scheduled_workflows'</div><div><span style="font-family:verdana,sans-serif"><br></span></div><div>>><span style="font-family:verdana,sans-serif">In that case I have to ask the question: how are they supposed to be changed?</span></div><div>To set the values of vglobal, you need to pass a key value pair. i.e., vglobal(key, value).</div><div><br></div><div>Let's introduce the api setMaxAllowedScheduledWorkflows() api.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Oct 16, 2020 at 2:54 PM Martin Allen <<a href="mailto:martin.allen@clystnet.com" target="_blank">martin.allen@clystnet.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"><div class="gmail_default" style="font-family:verdana,sans-serif">Thanks Uma, </div><div class="gmail_default" style="font-family:verdana,sans-serif"><br></div><div class="gmail_default" style="font-family:verdana,sans-serif">I agree that taking a look at how the vglobal function works is a good idea. </div><div class="gmail_default" style="font-family:verdana,sans-serif"><br>To me it makes sense that variables like this ARE set in the config.inc.php, and function like vglobal is used to retrieve these variables (and if they are not set, then use a default value). Is this what your intention is, or are you aiming to remove these variables from config.inc.php completely? </div><div class="gmail_default" style="font-family:verdana,sans-serif">In that case have to ask the question how are they supposed to be changed? (I thought the intention of the original bug was to have environment variables like this in a single unified location e.g. config.inc.php)<br></div><div class="gmail_default" style="font-family:verdana,sans-serif"><br></div><div><div dir="ltr"><div dir="ltr"><div><div dir="ltr"><p style="font-size:12.8px;color:rgb(80,0,80)"><span style="color:navy;font-family:Verdana,sans-serif;font-size:10pt">Martin Allen</span></p><p style="font-size:12.8px;color:rgb(80,0,80)"><b style="font-size:12.8px"><span lang="EN-US" style="font-size:10pt;font-family:Verdana,sans-serif;color:rgb(51,51,153)">01392 248692 - Main Office<br>01392 690659 - Direct Line</span></b><br></p><p style="font-size:12.8px;color:rgb(80,0,80)"><b><span style="font-size:10pt;font-family:Verdana,sans-serif;color:rgb(51,51,153)"><img width="288" height="65" alt="signature2"></span></b><span style="color:rgb(31,73,125)"></span></p><p style="font-size:12.8px;color:rgb(80,0,80)"><span lang="EN-US" style="font-size:10pt;font-family:Verdana,sans-serif;color:navy">Have you visited our website recently? <a href="http://www.clystnet.com/" style="color:rgb(17,85,204)" target="_blank">https://www.clystnet.com</a></span><span style="color:rgb(31,73,125)"></span></p><p style="font-size:12.8px;color:rgb(80,0,80)"><span lang="EN-US" style="font-size:10pt;font-family:Verdana,sans-serif;color:navy"><br></span><span lang="EN-US" style="font-size:10pt;font-family:Verdana,sans-serif;color:rgb(153,153,153)">The information in this email is confidential If you are not the intended recipient, you must not read or use that information. This email and any attachments are believed to be virus free however no responsibility is accepted by Clystnet for any loss or damage arising in any way from receipt or use thereof. Clystnet Ltd (company reg number 7164503) is based at Silverdown Park, Fair Oak Close, Clyst Honiton, EX5 2UX</span></p></div></div></div></div></div><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, 16 Oct 2020 at 09:42, Uma S <<a href="mailto:uma.s@vtiger.com" target="_blank">uma.s@vtiger.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">>>Err, the Settings->Configuration Editor alters config.inc.php...<div>Let's figure out a way to overcome this.<br></div><div><br></div><div>>>Please do have an option to write the data in config.inc.php.</div><div>Typo! Please don't have an option to write the data in config.inc.php.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Oct 16, 2020 at 1:59 PM Alan Lord <<a href="mailto:alanslists@gmail.com" target="_blank">alanslists@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">On 16/10/2020 09:22, Uma S wrote:<br>
> Hi Martin,<br>
> <br>
> Sorry! I think maybe I am not clear on the point.<br>
> <br>
> As it's empathize not a good practice to alter config.inc.php, We have <br>
> removed that from config.template.php. Which you can see in our commit <br>
<br>
Err, the Settings->Configuration Editor alters config.inc.php...<br>
<br>
Al<br>
_______________________________________________<br>
<a href="http://www.vtiger.com/" rel="noreferrer" target="_blank">http://www.vtiger.com/</a><br>
</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>
<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>
<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>