Declared javascript function not binding correctly

Question

I have the follow code:

    $(document).ready(function() { 
    $(function detinator() {
                    $("select#arhpcnDetinator").jCombo("services/detinatori.php", { 
                            initial_text: "Detinator",
                            selected_value: $("input#arhpcndetinatorID").val()
                    });
});    
});

and everything is working perfect.

When I try to call same function on other event, it's stop working.

  $(document).ready(function() { 
    $(function detinator() {
                    $("select#arhpcnDetinator").jCombo("services/det.php", { 
                            initial_text: "Detinator",
                            selected_value: $("input#arhpcndetinatorID").val()
                    });
    $("#imag").click(detinator);
    });
});

HTML is:

<img src="Sync.png" id="imag" onclick="ziceva">

Please help me! Thank you in advance.

glortho · Accepted Answer · 2012-01-12 02:30:57Z

2

I would remove it from the double-wrapper you have there, like so:

$(document).ready(function() { 
    var detinator = function() {
      $("select#arhpcnDetinator").jCombo("services/det.php", { 
        initial_text: "Detinator",
        selected_value: $("input#arhpcndetinatorID").val()
      });
    };

    $("#imag").click(detinator);
});

answered Jan 12, 2012 at 2:30

glortho

13.2k8 gold badges51 silver badges46 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Steve Wellens Over a year ago

The "double wrap" was weird. I would move the function declaration out of the ready function.

James Montagne Over a year ago

This is not equivalent to his code unless you add a call to detinator. It's a very odd construct which will result in that function being called on ready. Though I doubt it's intentionally, his way would also add an additional click handler on each call.

bretterer · Accepted Answer · 2012-01-12 02:33:19Z

0

Would you be able to try something like this?

$(document).ready(function() { 

   $("#imag").click(function() {
                $("select#arhpcnDetinator").jCombo("services/det.php", { 
                        initial_text: "Detinator",
                        selected_value: $("input#arhpcndetinatorID").val()
                });

   });


});

answered Jan 12, 2012 at 2:33

bretterer

5,7825 gold badges34 silver badges54 bronze badges

1 Comment

Leonard Lepadatu Over a year ago

this is not what I want, because when DOM are loading, jCombo feed my select box. After I add a new record (on the other window) I must refresh/reload select list and recall jCombo(jSON).

auroranil · Accepted Answer · 2012-01-12 02:33:31Z

0

Try converting the detinator function into javascript.

$(document).ready(function() { 
    function detinator() {
            $("select#arhpcnDetinator").jCombo("services/det.php", { 
                initial_text: "Detinator",
                selected_value: $("input#arhpcndetinatorID").val()
            });
    }
    $("#imag").click(detinator());
});

answered Jan 12, 2012 at 2:33

auroranil

2,6716 gold badges26 silver badges35 bronze badges

2 Comments

James Montagne Over a year ago

You should not be calling detinator. You are passing to click the return value of detinator which is nothing. Also, the original code executes the function on ready.

Leonard Lepadatu Over a year ago

But I must to call detinator function :). When DOM are loading, jCombo feed my select list. After I add a new record (on the other window) I must refresh/reload select list and recall jCombo(jSON) with a click on #imag.

James Montagne · Accepted Answer · 2012-01-12 02:38:38Z

0

I think the open parenthesis { that you have after $("select#arhpcnDetinator").jCombo("services/det.php", is the problem. Remove it and try again..

edited Jan 12, 2012 at 2:38

James Montagne

78.9k14 gold badges115 silver badges132 bronze badges

answered Jan 12, 2012 at 2:36

yaza

1

1 Comment

James Montagne Over a year ago

That's not a parenthesis, it's a curly brace or bracket. And I see no issue with it.

Collectives™ on Stack Overflow

Declared javascript function not binding correctly

4 Answers 4

2 Comments

1 Comment

2 Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

2 Comments

1 Comment

2 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related