+1, I suggest a Jira to handle all cases like that

Jacques


Le 09/01/2017 à 10:22, Devanshu Vyas a écrit :
Hello Devs,

I found an inconsistency in the code for string comparison
*statusId.equals("PRUN_COMPLETED")* whereas it should be written as
*"PRUN_COMPLETED".equals(statusId)*
cause the former can throw NullPointerException if the variable found to be
NULL.

This code pattern can be found at several places and if you all agree with
this I can provide a patch for correcting code.

Let me know your thoughts.

Thanks & Regards,
Devanshu Vyas.


Reply via email to