Fixing "Microsoft JScript runtime error: '__pendingCallbacks[...].async' is null or not an object"

Introduction

I recently hit a runtime error bug while using ASP.NET callback components.

There is a bug in the Microsoft ASP.NET standard library.

Fixing the bug is rather complicated because you have to change the source code you don't have access to.

I could not find a complete step by step article on how to fix the error, this is why I decided to post the entire code.

Background

When running a page, I would hit an error:

"Microsoft JScript runtime error: '__pendingCallbacks[...].async' 
					is null or not an object"

The error occurs on this line.

// ORIGINAL CODE FROM MICROSOFT 
function WebForm_CallbackComplete() {
    for (i = 0; i < __pendingCallbacks.length; i++) {
        callbackObject = __pendingCallbacks[i];
        if (callbackObject && callbackObject.xmlRequest && 
		(callbackObject.xmlRequest.readyState == 4)) {
            WebForm_ExecuteCallback(callbackObject);
            if (!__pendingCallbacks[i].async) {
                __synchronousCallBackIndex = -1;
            }
            __pendingCallbacks[i] = null;
            var callbackFrameID = "__CALLBACKFRAME" + i;
            var xmlRequestFrame = document.getElementById(callbackFrameID);
            if (xmlRequestFrame) {
                xmlRequestFrame.parentNode.removeChild(xmlRequestFrame);
            }
        }
    }
} 

The Problem

It seems that the error is in the for (i=0; and is the core of the problem. The i variable is a global variable and it can get changed within the loop.

The fix is to add a var keyword to make the variable local.
Looking on the internet, you can find that some people also suggest to move the WebForm_ExecuteCallback lower in the loop.

The Fix

In order to fix your page, you just have to insert this code in the beginning of your page. I personally put it in the body.

<script type="text/javascript">
//<![CDATA[
var GlvDelayedNextPageNo;

function WebForm_CallbackComplete_SyncFixed() {
     // the var statement ensure the variable is not global
     for (var i = 0; i < __pendingCallbacks.length; i++) {
        callbackObject = __pendingCallbacks[i];
        if (callbackObject && callbackObject.xmlRequest && 
			(callbackObject.xmlRequest.readyState == 4)) {
            // SyncFixed: line move below // WebForm_ExecuteCallback(callbackObject);
            if (!__pendingCallbacks[i].async) { 
                __synchronousCallBackIndex = -1;
            }
            __pendingCallbacks[i] = null;
            var callbackFrameID = "__CALLBACKFRAME" + i;
            var xmlRequestFrame = document.getElementById(callbackFrameID);
            if (xmlRequestFrame) {
                xmlRequestFrame.parentNode.removeChild(xmlRequestFrame);
            }
            // SyncFixed: the following statement has been moved down from above;
            WebForm_ExecuteCallback(callbackObject);
        }
    }
}

var OnloadWithoutSyncFixed = window.onload;

window.onload = function Onload(){
    if (typeof (WebForm_CallbackComplete) == "function") {
        // Set the fixed version
        WebForm_CallbackComplete = WebForm_CallbackComplete_SyncFixed;
        // CallTheOriginal OnLoad
        if (OnloadWithoutSyncFixed!=null) OnloadWithoutSyncFixed();
    }
}
//]]>
</script> 

Points of Interest

This was because while overloading the onload function, you could change the behaviour of the rest of your components.

The right method is to override the onload but within the new onload, call the previous one.

This is not rocket science, but also probably not obvious for everyone.

You must Sign In to use this message board.

FAQ    Noise Tolerance Very HighHighMediumLowVery Low   Layout Expand Posts & RepliesThread ViewNo JavascriptNo JS + Preview   Per page 102550

Msgs 1 to 2 of 2 (Total in Forum: 2) (Refresh) FirstPrevNext

Typo schweeneh 11:07 22 Sep '08   Just thought I'd mention that the script to fix the problem has a typo in it. you've got qcallbackObject instead of callbackObject, which throws a javascript error if you try to use the script. Thanks for the fix though, works great. Sign In·View Thread·PermaLink 3.75/5 (3 votes) Re: Typo Pascal Ganaye 14:31 1 Jun '09   And... 6 month later I read this and fixed the article Thanks for the positive feedback Thanks for helping to fix the article Sign In·View Thread·PermaLink

Last Visit: 16:05 9 Feb '10     Last Update: 16:05 9 Feb '10 1