Javascript checkbox crazy if statement

Question

I've got a command.cgi that returns a website url, if the checkbox with id="simple" is checked then redirect the user to that page, if not, add the html formatted link to a container. The problem is that it doesn't redirect the user to a html formatted link when the checkbox is checked and when it does, it redirects the user to a formatted html link, it's crazy. Any solution? Thanks

<script type="text/javascript">
 $(document).ready(function() {
      $("#sudo").click(function(event){
          if ($("#simple").attr('checked')==1){
                    $.get("/cgi-bin/command.cgi",
                     { cmd: $("#cmd").val()},
                     function(data) {
        window.location.href=data;
                     });
                }else{
                    $.get("/cgi-bin/command.cgi",
                     { cmd: $("#cmd").val()},
                     function(data) {
                        $('#resultado p').prepend("<a href=\"" + data + "\" target=\"_blank\">" + data + "</a><br><br>");
                        $("#cmd").val('');
                        $("#cmd").focus();
                     });  
                }


      });
      $("#cmd").keyup(function(e){
        if(e.keyCode == 13){
            $("#sudo").click();
        }
    });
   });
</script>

Sudhir Bastakoti · Accepted Answer · 2012-01-13 04:37:32Z

1

Try:


$("#sudo").click(function(event){
    $.get("/cgi-bin/command.cgi",
     { cmd: $("#cmd").val()},
     function(data) {
        if($("#simple").is(":checked")) {
            window.location.href=data;
        }
        else {
            $('#resultado p').prepend("<a href=\"" + data + "\" target=\"_blank\">" + data + "</a><br><br>");
            $("#cmd").val('');
            $("#cmd").focus();
        }
     });
});

answered Jan 13, 2012 at 4:37

Sudhir Bastakoti

100k15 gold badges163 silver badges169 bronze badges

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

Collectives™ on Stack Overflow

Javascript checkbox crazy if statement

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related