jQuery .each loop string or object

Question

Why does this not work?

$( ["blog","user","forum"] ).each(function(num,opt) {
    if ( window.location.pathname.indexOf(opt) != -1 ) {
        $('#rb-' + opt).attr('checked','checked');
        return false;
      }
});

When I input $('#rb-blog').attr('checked','checked'); it works as expected?

console.log(typeof opt) produces string and the expected value.

--- UPDATE ---

I've just seen the html is being written to the page via ajax and is executed on .ready() :( Thanks for the help all, much appreciated.

Your code looks just fine. Are you sure `window.location.pathname` contains the option? — João Silva, Sep 10 '12 at 14:19
Is your code in a `ready`or `load` block? Your checkboxes may not be initialized yet. — Michael Laffargue, Sep 10 '12 at 14:20
Like @ComputerArts mentioned, it should be either an id, type or classname - [".blog",".user",".forum"], or #blog etc.. — karthikr, Sep 10 '12 at 14:23
Could you please provide also the html code of the web page? — Muffo, Sep 10 '12 at 14:25
@karthikr not necessarily, the [each](http://api.jquery.com/jQuery.each/) method is generic, it works with any sort of array, it doesn't *have* to be elements. — James, Sep 10 '12 at 14:28
Thx guys. `window.location.pathname` is not null and contains the path as expected. — Jongosi, Sep 10 '12 at 14:45

Ties · Answer 1 · 2012-09-10T14:44:30.473

1

What the problem could be if the page is not fully loaded and #rb-blog is not available yet.

$(document).ready(function(){
    $( ["blog","user","forum"] ).each(function(num,opt) {
        if ( window.location.pathname.indexOf(opt) != -1 ) {
            $('#rb-' + opt).attr('checked','checked');
            return false;
        }
    });
});

edited Sep 10 '12 at 14:44

answered Sep 10 '12 at 14:29

Ties

5,726
3
28
37

score 0 · Answer 2 · edited May 23 '17 at 12:18

As already noted in the comments, return false will actually exit the "blog", "user", "forum" loop and thus stop checking the checkboxes once one pathname.indexOf condition is true.

Also you might want to add a console.log(window.location.pathname); to make certain this variable contains what you are checking on. Perhaps it is a casing issue?

If you want to know which literals are present in pathname you could use this:

var isPresent = [];
$( ["blog","user","forum"] ).each(function(num,opt) {
    if ( window.location.pathname.indexOf(opt) != -1 ) {
        $('#rb-' + opt).attr('checked','checked');
        isPresent.push(opt);
    }
});

If you just want to know if one of the literals is present in pathname:

var isAtLeastOneIsPresent = false;
$( ["blog","user","forum"] ).each(function(num,opt) {
    if ( window.location.pathname.indexOf(opt) != -1 ) {
        $('#rb-' + opt).attr('checked','checked');
        isAtLeastOneIsPresent = true;
    }
});

Thx, exiting the loop after the first match is the desired behaviour. — Jongosi, Sep 10 '12 at 15:59

score 0 · Accepted Answer · edited May 23 '17 at 10:24

Solution: the HTML content was not yet written to the page, so now we wait for ajax requests to complete, then we call the function to update the select value, like so...

// apply checked to the selected element
function setAsChecked() {
    $.each(["blog","user","forum"], function(num,opt){
        if (window.location.pathname.indexOf(opt) != -1) {
            $('#rb-' + opt, '.radio_menu').attr('checked','checked');
            return false;
        }
    });
}

// $('.radio_menu').ajaxStop(function(){
//  setAsChecked();
// });

// UPDATE! ajaxStop() catches all our ajax events - not cool. Instead, use when
$.when( $.ajax("") ).done(function(){
    setAsChecked();
});

Maybe that saves someone else a headache!

--- EDIT ---

BE WARNED! This solution caused a major headache when used with CakePHP. The layout disappeared when we called this function in the footer.

See this thread: CakePHP no layout on back and forward button

jQuery .each loop string or object

3 Answers3

Linked