[ 
https://issues.apache.org/jira/browse/CB-11391?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15321264#comment-15321264
 ] 

Soeren Unruh commented on CB-11391:
-----------------------------------

Yes, an enum should also make it "clearer" what the intention is instead of 
handling {{null}} as "follow default policy" in the various places. Something 
like {{ALLOW, DENY, DEFAULT}}, maybe.

> cordova-plugin-whitelist Android does not pass sonarqube scan
> -------------------------------------------------------------
>
>                 Key: CB-11391
>                 URL: https://issues.apache.org/jira/browse/CB-11391
>             Project: Apache Cordova
>          Issue Type: Bug
>          Components: Plugin Whitelist
>    Affects Versions: Master
>         Environment: Android
>            Reporter: Rob Close
>            Priority: Minor
>             Fix For: Master
>
>
> The Android whitelist plugin does not pass sonarqube 
> (http://www.sonarqube.org/) scan.
> The problem is "Correctness - Suspicious reference comparison of Boolean 
> values", which sonarqube considers major.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to