Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.3.4
    • Fix Version/s: 1.3.5, 1.4-RC1
    • Component/s: None
    • Labels:
      None

      Description

      WicketMessage: Error attaching this container for rendering: [MarkupContainer [Component id = feedbackul, page = nl.topicus.tripleapoc.web.pages.deelnemer.intake.IntakeStap1Personalia, path = 11:layLeft:feedback:feedbackul.FeedbackPanel$2, isVisible = true, isVersioned = false]]

      Root cause:

      java.lang.NullPointerException
      at org.apache.wicket.markup.html.panel.FeedbackPanel.newMessageDisplayComponent(FeedbackPanel.java:324)
      at org.apache.wicket.markup.html.panel.FeedbackPanel$MessageListView.populateItem(FeedbackPanel.java:88)
      at org.apache.wicket.markup.html.list.ListView.onPopulate(ListView.java:573)
      at org.apache.wicket.markup.repeater.AbstractRepeater.onBeforeRender(AbstractRepeater.java:127)
      at org.apache.wicket.Component.internalBeforeRender(Component.java:1003)
      at org.apache.wicket.Component.beforeRender(Component.java:1035)
      at org.apache.wicket.MarkupContainer.onBeforeRenderChildren(MarkupContainer.java:1589)
      at org.apache.wicket.Component.onBeforeRender(Component.java:3684)
      at org.apache.wicket.Component.internalBeforeRender(Component.java:1003)
      at org.apache.wicket.Component.beforeRender(Component.java:1035)
      at org.apache.wicket.MarkupContainer.onBeforeRenderChildren(MarkupContainer.java:1589)
      at org.apache.wicket.Component.onBeforeRender(Component.java:3684)
      at org.apache.wicket.Component.internalBeforeRender(Component.java:1003)
      at org.apache.wicket.Component.prepareForRender(Component.java:2168)
      at org.apache.wicket.ajax.AjaxRequestTarget.respondComponent(AjaxRequestTarget.java:747)
      at org.apache.wicket.ajax.AjaxRequestTarget.respondComponents(AjaxRequestTarget.java:649)
      at org.apache.wicket.ajax.AjaxRequestTarget.respond(AjaxRequestTarget.java:564)
      at org.apache.wicket.request.AbstractRequestCycleProcessor.respond(AbstractRequestCycleProcessor.java:104)
      at org.apache.wicket.RequestCycle.processEventsAndRespond(RequestCycle.java:1177)
      at org.apache.wicket.RequestCycle.step(RequestCycle.java:1248)
      at org.apache.wicket.RequestCycle.steps(RequestCycle.java:1349)
      at org.apache.wicket.RequestCycle.request(RequestCycle.java:493)
      at org.apache.wicket.protocol.http.WicketFilter.doGet(WicketFilter.java:387)
      at org.apache.wicket.protocol.http.WicketFilter.doFilter(WicketFilter.java:199)

        Activity

        Hide
        Igor Vaynberg added a comment -

        i think IAE is fine

        Show
        Igor Vaynberg added a comment - i think IAE is fine
        Hide
        Martijn Dashorst added a comment -

        this happened when I added the message of an exception that occurred to the feedback queue using error(e.getMessage()); the a NPE has a null-message, so this instructed Wicket to create a FeedbackMessage(..., null, ...). Later on this causes a NPE in the feedback panel.

        Though I think that empty messages are useless to users, the framework should not crash upon them. If we don't accept null messages, we should throw an IllegalArgumentException from the FeedbackMessage constructor, which allows the developer to pinpoint where he went wrong, instead of having to step through all code.

        Any objections to throwing an IllegalArgumentException from the FeedbackMessage constructor?

        Show
        Martijn Dashorst added a comment - this happened when I added the message of an exception that occurred to the feedback queue using error(e.getMessage()); the a NPE has a null-message, so this instructed Wicket to create a FeedbackMessage(..., null, ...). Later on this causes a NPE in the feedback panel. Though I think that empty messages are useless to users, the framework should not crash upon them. If we don't accept null messages, we should throw an IllegalArgumentException from the FeedbackMessage constructor, which allows the developer to pinpoint where he went wrong, instead of having to step through all code. Any objections to throwing an IllegalArgumentException from the FeedbackMessage constructor?

          People

          • Assignee:
            Martijn Dashorst
            Reporter:
            Martijn Dashorst
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development