jQuery 1.4.2 IE8 error on page expected ')'

Question

Hi all,

I'm having an issue with IE8 (nothing new there). The error I'm getting is not making much sense as it does not occur on FF or Chrome. Here's the code:

function remComp(id, trade) {
            $.ajax({
                url: "functions/removePriceSurveyComparison.php",
                type: "POST",
                async: true,
                data: "id="+id,
                dataType: "xml",
                success: function(xmlData) {
                    if ($("success", xmlData).text() == "true") {
                        loadComps(trade);
                    }// TODO create error handler
                }
            });
        }

In this function it is complaining about the line where the success callback is defined. This function has not even been called yet though? But when it does get called, it works perfectly fine, although still creates new errors?

The function that is being called though is:

        function loadComps(trade) {
            $.ajax({
                url: "functions/loadPriceSurveyComparisons.php",
                type: "POST",
                async: true,
                data: "trade="+trade,
                cache: false,
                dataType: "html",
                success: function(comps) {
                    $("#current"+trade).html(comps);
                }
            });
        }

The second function is basically getting called 3 times when the page loads. Any advice?

Here is the complete script block as well:

        function remComp(id, trade) {
            $.ajax({
                url: "functions/removePriceSurveyComparison.php",
                type: "POST",
                async: true,
                data: "id="+id,
                dataType: "xml",
                success: function(xmlData) {
                    if ($("success", xmlData).text() == "true") {
                        loadComps(trade);
                    }// TODO create error handler
                }
            });
        }

        function addComp(trade, albId, compId) {
            $.ajax({
                url: "functions/addPriceSurveyComparison.php",
                type: "POST",
                async: true,
                data: "trade="+trade+"&albId="+albId+"&compId="+compId,
                cache: false,
                dataType: "xml",
                success: function(xmlData) {
                    if ($("success", xmlData).text() == "true") {
                        loadComps(trade);
                    }// TODO add an error handler
                }
            });
        }

        function updateComp(id, trade) {
            var albId = $("select#albProd"+id).val();
            var compId = $("select#compProd"+id).val();

            $.ajax({
                url: "functions/updatePriceSurveyComparison.php",
                type: "POST",
                async: true,
                data: "id="+id+"&albId="+albId+"&compId="+compId,
                cache: false,
                dataType: "xml",
                success: function(xmlData) {
                    if ($("success", xmlData).text() == "true") {
                        // reload table for this trade
                        loadComps(trade);
                    }// TODO create error handler
                }
            });
        }

        // function that loads all of the comparisons for a specific trade
        function loadComps(trade) {
            $.ajax({
                url: "functions/loadPriceSurveyComparisons.php",
                type: "POST",
                async: true,
                data: "trade="+trade,
                cache: false,
                dataType: "html",
                success: function(comps) {
                    $("#current"+trade).html(comps);
                }
            });
        }

        // define document.ready function
        $(document).ready(function() {
            // load all of the comparisons for each trade
            <?php
            foreach ($trades as $trade) {
                echo "loadComps(\"$trade\");\n";
                ?>
                $("#addComp<?php echo $trade; ?>").click(function(e) {
                    e.preventDefault();
                    addComp("<?php echo $trade; ?>", $("#albProd<?php echo $trade; ?>").val(), $("#compProd<?php echo $trade; ?>").val());
                });
                <?php
            }
            ?>
        });