[JWebUnit-development] [ jwebunit-Patches-3590252 ] Imprive exception message when setting form by
Brought to you by:
henryju
|
From: SourceForge.net <no...@so...> - 2012-11-27 15:56:16
|
Patches item #3590252, was opened at 2012-11-26 18:33 Message generated for change (Comment added) made by henryju You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=497984&aid=3590252&group_id=61302 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. >Category: Core >Group: Release 3.2 >Status: Closed >Resolution: Fixed Priority: 5 Private: No Submitted By: Kenny MacLeod (skaffman) >Assigned to: Julien HENRY (henryju) Summary: Imprive exception message when setting form by ID Initial Comment: When you call setWorkingForm(nameorId, index), and no form is found by those arguments, then you get a rather unhelpful message: "Attempted to set form to null." Attached is a patch (against v3.1) which checks for the null pre-emptively, and throws a more useful exception. ---------------------------------------------------------------------- >Comment By: Julien HENRY (henryju) Date: 2012-11-27 07:56 Message: I prefer to do the same than for setWorkingForm(nameOrId): assert that form is present before trying to define it as default working form. Anyway thanks for reporting the issue. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=497984&aid=3590252&group_id=61302 |