YADA icon indicating copy to clipboard operation
YADA copied to clipboard

is this where everything gets simpler?

Open christopheleroy opened this issue 6 years ago • 0 comments

I think line 1012 in this file is where a column name that is missing will be counted as "Null", bringing a cryptic "NullPointerException" and a 500 to the Yada Web developer that forgot to add a column/field in their payload.

https://github.com/Novartis/YADA/blob/c3e5f486294f86ba90ae816857fb3ac0ce923583/yada-api/src/main/java/com/novartis/opensource/yada/util/QueryUtils.java#L1012

Line 1014 will throw the Exception when the null is consumed by the for statement.

It would be awesome if the developer got YadaException like "column (column name) was not found in the payload for (query name)" instead of the enraging NullPointerException

Pretty please.

christopheleroy avatar Nov 02 '18 16:11 christopheleroy