https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35647

Marcel de Rooy <m.de.r...@rijksmuseum.nl> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Signed Off                  |Failed QA
                 CC|                            |m.de.r...@rijksmuseum.nl

--- Comment #6 from Marcel de Rooy <m.de.r...@rijksmuseum.nl> ---
    if (!loaded && counter_wait < 50) {// Do not wait more than 5 seconds
        window.setTimeout(function(){wait_for_images(cb);}, 100);
    } else {
        if (counter_wait >= 50 ) {
            console.log("Could not retrieve the images")
        }
        cb();
    }
Shouldnt you differentiate loaded is true and !loaded && counter_wait > 50 ?
Now you always go the callback?

$(window).load(function() {
    if ( PREF_IntranetCoce && PREF_CoceProviders ) {
        wait_for_images(verify_cover_images);
    }
});
What about the other source of images? Amazon or whatever? Is that condition
really correct?

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to