Uploaded image for project: 'Wicket'
  1. Wicket
  2. WICKET-2646

Better error reporting from StatelessChecker

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 1.4.5
    • 1.4.6, 1.5-M1
    • wicket
    • None

    Description

      Improvement for error messages from StatelessChecker:
      replace
      throw new IllegalArgumentException(msg + " Offending component: " + o);

      with
      throw new IllegalArgumentException(msg + " Offending component: " + ((Component)o)toString(true));

      Reasoning:
      Current message contains Component.toString(false) output that doesn't contain component path, thus making it harder to identify which component is to blame.

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            jdonnerstag Juegen Donnerstag
            slonopotamus Marat Radchenko
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment