Clone
Jacques Le Roux <jacques.le.roux@les7arts.com>
committed
on 16 Jun
Improved: Checks if the value passed to checkStringForHtmlSafe is not null (OFBIZ-11822)
This was reported by SpotBugs in Eclipse to prevent… Show more
Improved: Checks if the value passed to checkStringForHtmlSafe is not null (OFBIZ-11822)

This was reported by SpotBugs in Eclipse to prevent a possible NPE.

It seems to me that it's more complex than that because of the

PolicyFactory::sanitize methods signatures where @Nullable annotation is

used. So the SpotBugs warning remains, anyway can't hurt.

BTW found 92 issues reported by SpotBugs in Eclipse, among them 33 are "scary"

I had to add 1 to tasks.checkstyleMain.maxErrors because of

https://github.com/apache/ofbiz-framework/pull/203 that I pulled after the push

was rejected by the pre-push local Git hook

Show less