Community
Participate
Working Groups
A ComboRidget gets bound to a model. If no rendering-Method is provided, the toString-Methode will be used for rendering the items. If one of the items in the model is null, all following items will not be displayed. In this case the ComboRidget should throw a NullPointerException, otherwise the developer might be confused.
Created attachment 141874 [details] throw NullPointerException in ComboRidget
http://www.compeople.eu/support/rundv/reports/show_bug.cgi?id=11
please ignore previous url. it points to a private bugzilla and was added by error
patch is committed, thanks Steffen
closing bug
Reopening, because there is a problem with the patch: String.valueOf(valueObject) never returns null. If value object is null it will return "null", so the NullPointerException in the added code will never be thrown: + String returnValue = String.valueOf(valueObject); + + if (null == returnValue) { + throw new NullPointerException("Value in model is null"); + } It would also be nice to have a test case or snippet reproducing the issue described in the bugzilla.
Created attachment 143384 [details] Added improved Patch + Testcase
Steffen, thanks for the updated patch! I've committed it. Steffen+Thorsten: one thing to be aware of is that on 'production' conditions the exception will be written straight into the log, but to be shown on the console. This happens when launching _without_ -consolelog in the 'program arguments' and _with_ -Driena.development=false in the 'VM Arguments'. If this is a problem please reopen.
