Solved

JavaScript: undefined function name

Posted on 2013-02-04
3
284 Views
Last Modified: 2013-02-04
Hi Experts,

On a .php page I have

<body leftmargin="0" topmargin="0" marginwidth="0" marginheight="0" onload="directPromo()">

Open in new window


but I get 'directPromo' is undefined.

When I change it to:
<body leftmargin="0" topmargin="0" marginwidth="0" marginheight="0" onload="document.directPromo()">

Open in new window


I get Object doesn't support property or method 'directPromo'

My directPromo is at the way bottom of my page, as follows:
<div id="divPromo" style="display:block">
	
		<form name="frmPromo" id="frmPromo" method="post">
			<p>
			<label>Promo Code:</label>
			<input type="text" name="txtPromo" size="8" " />
			<input type="hidden" name="sess" value="<?php echo $_SESSION['cart_sess']; ?>" />
			<input type="submit" value="Apply" onclick="changePromo()" name="btnApplyPromo" />
			</p>
			<center><label style="font-weight: bold; color:red;"><?php echo $_SESSION['pMSG']; ?></label></center>
		</form>

</div>
</body>
<script language="javascript">
function directPromo()
{
	<?php if ($_GET['promo'] != '') {?>
		document.frmPromo.txtPromo.value='<?php echo $_GET['promo']?>';
		
		str = '<?=$postURL?>' + 'changePromo.asp?goto=products';
		document.frmPromo.action = str;
		document.frmPromo.submit();
	<?php } ?>
}
</script)
</html>

Open in new window


Any ideas?
0
Comment
Question by:APD_Toronto
3 Comments
 
LVL 83

Expert Comment

by:Dave Baldwin
ID: 38852921
Why are you trying to change it?   'directPromo' is a standalone function you created so it will not be a property of anything in the 'document'.
0
 

Author Comment

by:APD_Toronto
ID: 38853295
I just need to execute 'directPromo' when the pg loads
0
 
LVL 58

Accepted Solution

by:
Gary earned 500 total points
ID: 38853314
Is this just a typo?
</script)
Should be
</script>

You can just add in before the closing script tag, assuming this would be the end of the page.
directPromo()

Also you should be using
<script type="text/javascript">

language= is deprecated so could be giving you problems
0

Featured Post

Gigs: Get Your Project Delivered by an Expert

Select from freelancers specializing in everything from database administration to programming, who have proven themselves as experts in their field. Hire the best, collaborate easily, pay securely and get projects done right.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Suggested Solutions

Title # Comments Views Activity
jquery datepciker end date add 6 days 6 34
Wordpress plugins not working in html 3 21
asp.net mvc5 6 15
alert(innerHTML); 8 13
Boost your ability to deliver ambitious and competitive web apps by choosing the right JavaScript framework to best suit your project’s needs.
Introduction This article is intended for those who are new to PHP error handling (https://www.experts-exchange.com/articles/11769/And-by-the-way-I-am-New-to-PHP.html).  It addresses one of the most common problems that plague beginning PHP develop…
The viewer will learn how to dynamically set the form action using jQuery.
The viewer will learn the basics of jQuery including how to code hide show and toggles. Reference your jQuery libraries: (CODE) Include your new external js/jQuery file: (CODE) Write your first lines of code to setup your site for jQuery…

776 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question