Search code examples
asp.netuser-controlsloadcontrol

Asp.net Usercontrol LoadControl Issue


I am having an issue when using LoadControl( type, Params ). Let me explain...

I have a super simple user control (ascx)

<%@ Control Language="C#" AutoEventWireup="True" Inherits="ErrorDisplay" Codebehind="ErrorDisplay.ascx.cs" EnableViewState="false" %>

<asp:Label runat="server" ID="lblTitle" />
<asp:Label runat="server" ID="lblDescription" />

with code ( c# ) behind of:

public partial class ErrorDisplay : System.Web.UI.UserControl
{

    private Message _ErrorMessage;    

    public ErrorDisplay()
    {
    }

    public ErrorDisplay(Message ErrorMessage)
    {
        _ErrorMessage = ErrorMessage;
    }    

    protected override void OnPreRender(EventArgs e)
    {
        base.OnPreRender(e);

        if (_ErrorMessage != null)
        {
            lblTitle.Text = _ErrorMessage.Message;
            lblDescription.Text = _ErrorMessage.Description;
        }
    }
}

Elsewhere in my web application I am adding an instance of the usercontrol to the page using the following code:

divValidationIssues.Controls.Add(LoadControl(typeof(ErrorDisplay), new object[] { MessageDetails }));   

I am using the overloaded version of LoadControl because I want to pass the Message parameter to the constructor. All this appears to work ok.

However, when the OnPreRender() is fired on the ErrorDisplay usercontrol the lblTitle and lblDescription variables are both null, despite them having a markup equivalent. The message variable has been correctly populated.

Can anyone shed any light on why this may be happening?

Thanks

EDIT:

Just for clarity I'll also add that the code which is programatically adding the usercontrol to the page is running in response to a button press, so the 'hosting page' has progressed through Init, Page_Load and is now processing the event handlers.

I cannot add the usercontrols at an earlier asp lifecycle stage as they are being created in response to a button click event.


Solution

  • I have tried the following code as well - which yields the same result (i.e. both lblTitle and lblDescription are null)

    protected void Page_Load(object sender, EventArgs e)
    {
        if (_ErrorMessage != null)
        {
            lblTitle.Text = _ErrorMessage.Message;
            lblDescription.Text = _ErrorMessage.Description;
        }
    }
    

    I had the understanding that the LoadControl function brought the control it is loading up to the current 'state' of the page onto which it is being included on. hence the Init, Page_Load etc are all run as part of the LoadControl call.

    Interestingly this (unanswered) asp.net forums post exhibits the same problem as I am experiencing.

    MSDN Forums Post

    Additionally - From the MSDN:

    When you load a control into a container control, the container raises all of the added control's events until it has caught up to the current event. However, the added control does not catch up with postback data processing. For an added control to participate in postback data processing, including validation, the control must be added in the Init event rather than in the Load event.

    Therefore shouldn't LoadControl correctly initalise the control?

    EDIT:

    Ok, so I'm answering my own question here ..

    I found an answered version of the forum post I linked to above Here

    Essentially the answer is that the LoadControl( type, params ) cannot infer the 'page infront' ascx to parse and hence it doesn't bother initalising any of the controls. When you use the LoadControl( "ascx path" ) version it is given the page infront and hence does all the parsing and initalision.

    So in summary I need to change the code which is initalising the control and split it into seperate parts. I.e.

    Control ErrorCntrl = LoadControl("ErrorDisplay.ascx");
    ErrorCntrl.ID = SomeID;
    (ErrorCntrl as ErrorDisplay).SetErrorMessage = MessageDetail;
    divErrorContainer.Controls.Add(ErrorCntrl);
    

    And it should work ok.. It isn't as neat as my previous attempt, but at least it will work.

    I am still open to suggestions to improve the above.

    Cheers