Function works and then says not defined

Expand Post »

Hi,
I have some code to dynamically retrieve one of three AJAX files and insert it into a DIV

The Javascript is

javascript Syntax (Toggle Plain Text)
function createajax() {
	var Ajax;
	try{
		Ajax = new XMLHttpRequest();
	} catch (e){
		try{
			Ajax = new ActiveXObject("Msxml2.XMLHTTP");
		} catch (e) {
			try{
				Ajax = new ActiveXObject("Microsoft.XMLHTTP");
			} catch (e){
				alert("Error!!!");
				return false;
			}
		}
	}
	return Ajax;
}
 
function basicAjaxSwitch(File, Recipient, Animate) {
	if ( Animate == true ) {
		Animation(document.getElementById(Recipient)).duration(100).from('opacity', 1).to('opacity', 0).go();
	}
	basicAjaxSwitch = createajax();
	basicAjaxSwitch.onreadystatechange = function(){
		if(basicAjaxSwitch.readyState == 4){
			if ( Animate == true ) {
				Animation(document.getElementById(Recipient)).duration(100).checkpoint().duration(100).from('opacity', 0).to('opacity', 1).go();
				setTimeout('document.getElementById("'+Recipient+'").innerHTML = basicAjaxSwitch.responseText;', 100);
			} else {
				document.getElementById(Recipient).innerHTML = basicAjaxSwitch.responseText;
			}
		}
	}
	basicAjaxSwitch.open("GET", "/ajax/"+File, true);
	basicAjaxSwitch.send(null);
}
 
function changesidebarinfo(changeto) {
	if ( changeto == 'login' ) {
		Animation(document.getElementById('main_layout_accountlinks_background')).to('left', '0px').duration(100).go();
		Animation(document.getElementById('main_layout_accountlinks_login')).duration(100).to('color', '#FFF').go();
		Animation(document.getElementById('main_layout_accountlinks_signup')).duration(100).to('color', '#000').go();
		Animation(document.getElementById('main_layout_accountlinks_about')).duration(100).to('color', '#000').go();
		basicAjaxSwitch('contentinfo/login.php', 'main_layout_rsbinfo_container', true);
	} else if ( changeto == 'join' ) {
		Animation(document.getElementById('main_layout_accountlinks_background')).to('left', '50px').duration(100).go();
		Animation(document.getElementById('main_layout_accountlinks_login')).duration(100).to('color', '#000').go();
		Animation(document.getElementById('main_layout_accountlinks_signup')).duration(100).to('color', '#FFF').go();
		Animation(document.getElementById('main_layout_accountlinks_about')).duration(100).to('color', '#000').go();
		basicAjaxSwitch('contentinfo/join.php', 'main_layout_rsbinfo_container', true);
	} else if ( changeto == 'about' ) {
		Animation(document.getElementById('main_layout_accountlinks_background')).to('left', '100px').duration(100).go();
		Animation(document.getElementById('main_layout_accountlinks_login')).duration(100).to('color', '#000').go();
		Animation(document.getElementById('main_layout_accountlinks_signup')).duration(100).to('color', '#000').go();
		Animation(document.getElementById('main_layout_accountlinks_about')).duration(100).to('color', '#FFF').go();
		basicAjaxSwitch('contentinfo/about.php', 'main_layout_rsbinfo_container', true);
	}
}
function createajax() {
	var Ajax;
	try{
		Ajax = new XMLHttpRequest();
	} catch (e){
		try{
			Ajax = new ActiveXObject("Msxml2.XMLHTTP");
		} catch (e) {
			try{
				Ajax = new ActiveXObject("Microsoft.XMLHTTP");
			} catch (e){
				alert("Error!!!");
				return false;
			}
		}
	}
	return Ajax;
}

function basicAjaxSwitch(File, Recipient, Animate) {
	if ( Animate == true ) {
		Animation(document.getElementById(Recipient)).duration(100).from('opacity', 1).to('opacity', 0).go();
	}
	basicAjaxSwitch = createajax();
	basicAjaxSwitch.onreadystatechange = function(){
		if(basicAjaxSwitch.readyState == 4){
			if ( Animate == true ) {
				Animation(document.getElementById(Recipient)).duration(100).checkpoint().duration(100).from('opacity', 0).to('opacity', 1).go();
				setTimeout('document.getElementById("'+Recipient+'").innerHTML = basicAjaxSwitch.responseText;', 100);
			} else {
				document.getElementById(Recipient).innerHTML = basicAjaxSwitch.responseText;
			}
		}
	}
	basicAjaxSwitch.open("GET", "/ajax/"+File, true);
	basicAjaxSwitch.send(null);
}

function changesidebarinfo(changeto) {
	if ( changeto == 'login' ) {
		Animation(document.getElementById('main_layout_accountlinks_background')).to('left', '0px').duration(100).go();
		Animation(document.getElementById('main_layout_accountlinks_login')).duration(100).to('color', '#FFF').go();
		Animation(document.getElementById('main_layout_accountlinks_signup')).duration(100).to('color', '#000').go();
		Animation(document.getElementById('main_layout_accountlinks_about')).duration(100).to('color', '#000').go();
		basicAjaxSwitch('contentinfo/login.php', 'main_layout_rsbinfo_container', true);
	} else if ( changeto == 'join' ) {
		Animation(document.getElementById('main_layout_accountlinks_background')).to('left', '50px').duration(100).go();
		Animation(document.getElementById('main_layout_accountlinks_login')).duration(100).to('color', '#000').go();
		Animation(document.getElementById('main_layout_accountlinks_signup')).duration(100).to('color', '#FFF').go();
		Animation(document.getElementById('main_layout_accountlinks_about')).duration(100).to('color', '#000').go();
		basicAjaxSwitch('contentinfo/join.php', 'main_layout_rsbinfo_container', true);
	} else if ( changeto == 'about' ) {
		Animation(document.getElementById('main_layout_accountlinks_background')).to('left', '100px').duration(100).go();
		Animation(document.getElementById('main_layout_accountlinks_login')).duration(100).to('color', '#000').go();
		Animation(document.getElementById('main_layout_accountlinks_signup')).duration(100).to('color', '#000').go();
		Animation(document.getElementById('main_layout_accountlinks_about')).duration(100).to('color', '#FFF').go();
		basicAjaxSwitch('contentinfo/about.php', 'main_layout_rsbinfo_container', true);
	}
}

And the HTML that triggers the actions is

HTML Syntax (Toggle Plain Text)
<div id="main_layout_accountlinks">
        	<div id="main_layout_accountlinks_background">&nbsp;</div>
        	<div id="main_layout_accountlinks_login" onclick="changesidebarinfo('login');">Log In</div>
            <div id="main_layout_accountlinks_signup" onclick="changesidebarinfo('join');">Join</div>
            <div id="main_layout_accountlinks_about" onclick="changesidebarinfo('about');">About</div>
        </div>
<div id="main_layout_accountlinks">
        	<div id="main_layout_accountlinks_background">&nbsp;</div>
        	<div id="main_layout_accountlinks_login" onclick="changesidebarinfo('login');">Log In</div>
            <div id="main_layout_accountlinks_signup" onclick="changesidebarinfo('join');">Join</div>
            <div id="main_layout_accountlinks_about" onclick="changesidebarinfo('about');">About</div>
        </div>

When I click one of the 3 buttons (doesn't matter which) the result is exactly as intended. Then when I click another (or the same) button, nothing happens and Firebug gives the error

text Syntax (Toggle Plain Text)

basicAjaxSwitch is not a function
changesidebarinfo("login")site.js (line 61)
function onclick(event) { changesidebarinfo("login"); }(click clientX=816, clientY=41)2 (line 2)
[Break on this error] basicAjaxSwitch('contentinfo/login.php...'main_layout_rsbinfo_container', true);\n

Regards,
Sam Rudge

Last edited by samarudge; Nov 3rd, 2009 at 11:44 am.

Similar Threads

dont understand a simple function pointer code in C++
how this function works for poisson dist?? in C++
Using function parameters defined in if statement in C++
DLL Function Call in C++
Need a way to make 1 function accessible to all in Python

samarudge

Reputation Points: 26

Solved Threads: 31

Posting Whiz

Offline

354 posts
since May 2008

Permalink

Nov 3rd, 2009

Re: Function works and then says not defined

Inside the function basicAjaxSwitch you are defining a variable basicAjaxSwitch so it's being overwritten. Just use a different variable name inside the function.

ShawnCplus

Sponsor

Reputation Points: 520

Solved Threads: 268

Code Monkey

Offline

1,564 posts
since Apr 2005

This thread is solved

Either the thread starter or a moderator has marked this thread as solved. You can most likely trust the responses and answers given. There is most likely no reason for any further responses to be posted here. If you have a related question, please start a new thread in this forum instead.

This thread is more than three months old

No one has posted to this discussion for at least three months. Please let old threads die and do not reply to them unless you feel you have something new and valuable to contribute that absolutely must be added to make the discussion complete. Otherwise, please start a new thread in this forum instead.

Previous Thread in JavaScript / DHTML / AJAX Forum Timeline: New web design need help connecting sql database to display content in a form mask

Next Thread in JavaScript / DHTML / AJAX Forum Timeline: Aptana Studio Nokia WRT RSS Reader help

DaniWeb Message

Cancel Changes

User Name
Password