Closed TonyCrousty closed 11 months ago
Hi @TonyCrousty , thank you very much for making us aware of these. The first one is clear to me, but the other two I'm not so sure about. If it is possible to give me one of those full flows in a way you mask any confidential information that would be ideal. You can also mail them to me via halmandevlabs@gmail.com
Hi, just mailed you the complete flow source
@TonyCrousty We have resolved bugs in the MissingNullHandler rule in the latest version, v2.11. However, the first lookup 'Get_all_opport' is still coming up as a violation. We will definitely still aim to enhance the rule so to support these types of use cases where a default value is assigned, but for now I recommend you define it as an exception.
@TonyCrousty do you know how to define the exception?
For now I am closing this issue. @TonyCrousty please do let us know if you find the current solution acceptable
Hi, in the flow I scanned, it looks 3 cases of false positive matched :
Get_all_opport (recordLookups)
Indeed, there is no decision on 'null' value of the result, but a variable is assigned with a count on the collection. If the collection is null, then the count is '0'. And a decision manages this case. Assignement extract :
Get_etab_from_Rubis (recordLookups)
is a recordLookup with a outputReference. This recordLookup is immediately followed with a 'null' check decision
Initial_lead_data (recordLookups)
same case than previous one. The only difference is the here, the 'null' check in the decision is part of several other conditions.
I can provide the full flow source if needed.
Regards,