JS function not seen

rodmcleay · #1 (**permalink**) October 23rd, 2009, 02:24 AM

Hi All,
I have a typical JS rollover menu that is giving me a Object Expected error.
I am not able to tell what the issue is.
I have simplified things a little to test the code.
So on the mouseover i now call a blanc test function with no parameters and the error still occurs.
I'm really not sure how to debug this any further?
The blow snippet is includes the test function and onmouseover="test();"

The error occuring on the call to the test() function.
I would really appreciate any advise.
Best Regards,
Rod

Code:


function test()
{
alert('boo');
}
</script>
<divstyle="width: 760px; padding: 0px30px0px16px;">
<tablecellpadding="7"cellspacing="7"style="margin-top: -20px;padding-top:10px; padding-right:30px;"align="center"border="0">
<tr>
<td>
<ahref="#"><spanonmouseover="test();onMouseOverMenu(this);"onmouseout="onMouseMenuOut(this);"class="mnuItemNormal">Gallery</span></a>
</td>

Full Menu Code

Code:

<script language="javascript" type="text/javascript">
    function onMouseOverMenu(obj)
    {
        obj.className = 'mnuItemOver';
    }
    
    function onMouseMenuOut(obj)
          
        obj.className = 'mnuItemNormal';        
    }
    function test()
    {
    alert('boo');
    }
</script>     
<div style="width: 760px; padding: 0px 30px 0px 16px;">
    <table cellpadding="7" cellspacing="7" style="margin-top: -20px;padding-top:10px; padding-right:30px;" align="center" border="0" >
        <tr>
            <td>
                <a href="#" ><span onmouseover="test();onMouseOverMenu(this);" onmouseout="onMouseMenuOut(this);" class="mnuItemNormal">Gallery</span></a>
            </td>
            <td>
                <a href="Create.aspx" id="LightWindowLink" class="lightwindow" params="lightwindow_type=external,lightwindow_width=880,lightwindow_height=656"  title="" author="" caption="" >
                     <span class="mnuItemNormal" onmouseover="onMouseOverMenu(this);" onmouseout="onMouseMenuOut(this);">Create one</span></a>
            </td>
            <td class="mnuItemNormal">
                <a href="http://jaal.allaboutlife.com.au/Jesus_has_Answer.aspx" class="mnuItemNormal" target="_parent">
                    <span onmouseover="onMouseOverMenu(this);" onmouseout="onMouseMenuOut(this);"  class="mnuItemNormal"> View TV ad</span></a>
            </td>
            <td class="mnuItemNormal">
                <a href="http://jaal.allaboutlife.com.au/whoisjesus.aspx" class="mnuItemNormal" target="_parent">
                    <span class="mnuItemNormal" onmouseover="onMouseOverMenu(this);" onmouseout="onMouseMenuOut(this);">Who is Jesus</span></a>
            </td>
            <td class="mnuItemNormal">
                <a href="http://jaal.allaboutlife.com.au/Orde...pelofMark.aspx"
                    target="_parent">
                    <span class="mnuItemNormal" onmouseover="onMouseOverMenu(this);" onmouseout="onMouseMenuOut(this);">Gospel Mark</span></a>
            </td>
            <td class="mnuItemNormal">
                <a href="http://jaal.allaboutlife.com.au/whats_on.aspx" target="_parent" >
                    <span onmouseover="onMouseOverMenu(this);" onmouseout="onMouseMenuOut(this);" class="mnuItemNormal">What's On</span></a>
            </td>
            
        </tr>
    </table>
</div>

Old Pedant · #2 (**permalink**) October 23rd, 2009, 02:37 AM

One very minor change and it worked for me:

Code:

    function onMouseMenuOut(obj)
    { /* YOU WERE MISSING THE LEFT BRACE HERE */          
        obj.className = 'mnuItemNormal';        
    }

You know, any decent debugger--or even the stupid ones MS has--would have found that immediately.

I recommend that you start using FireFox with the Firebug plugin. You would have found that in about 5 seconds.

Old Pedant · #3 (**permalink**) October 23rd, 2009, 02:39 AM

Oh, and the "object" that was giving the "object expected" error was indeed that function.

Functions *are* objects in JS, so when the function didn't compile, it didn't exist. Ergo, every time you referenced it..."object expected".

rodmcleay · #4 (**permalink**) October 23rd, 2009, 02:43 AM

Thanks very much for that.
I had tried that, but in conjunction with other changes, best to start from the beginning I guess.
I didn't write the code and thought I had introduced the missing bracket in my changes.
Many thanks you saved me a heap.
Regards,
Rod

Need to download code?

Navigation