[Vtigercrm-developers] Hotfix 2 Bug

[Martin Allen]

Tested and confirmed - if you set the variable in your config file for
max_scheduled_workflows to 20, you can still only create 10 scheduled
workflows.

Martin Allen


*01392 248692 - Main Office01392 690659 - Direct Line*

*[image: signature2]*

Have you visited our website recently? https://www.clystnet.com
<http://www.clystnet.com/>


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


On Wed, 14 Oct 2020 at 08:53, Uma S <uma.s at vtiger.com> wrote:

> Hi Martin,
>
> No! max_scheduled_workflows is a constant which is compared with getScheduledWorkflowsCount().
> So that it always remains the same.
>
> On Wed, Oct 14, 2020 at 1:00 PM Martin Allen <martin.allen at clystnet.com>
> wrote:
>
>> Hi - I've just looked at the commit for fixing the
>> max_scheduled_workflows issue, and I suspect it has a bug -not tested it
>> yet, but it looks pretty obvious what will happen.
>>
>> In getMaxAllowedScheduledWorkflows() it uses vglobal() to SET
>> max_scheduled_workflows to 10, then uses the same function to retrieves the
>> value in max_scheduled_workflows. Which will always be 10 as you just set
>> it to 10!
>>
>> It basically will now ignore whatever value you set in config.inc.php !
>>
>> There should be a separate function to SET a global variable, and a
>> different one to RETRIEVE the variable (with a default value returned if
>> not set, which im guessing is what was intended here)
>>
>> Martin Allen
>>
>>
>> *01392 248692 - Main Office01392 690659 - Direct Line*
>>
>> *[image: signature2]*
>>
>> Have you visited our website recently? https://www.clystnet.com
>> <http://www.clystnet.com/>
>>
>>
>> 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
>> _______________________________________________
>> http://www.vtiger.com/
>
>
>
> --
> With
> Best Regards
> Uma.S
> Vtiger Team
> _______________________________________________
> http://www.vtiger.com/
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.vtigercrm.com/pipermail/vtigercrm-developers/attachments/20201015/f6aebf36/attachment-0001.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: image001.png
Type: image/png
Size: 10550 bytes
Desc: not available
URL: <http://lists.vtigercrm.com/pipermail/vtigercrm-developers/attachments/20201015/f6aebf36/attachment-0001.png>