If you subscribe to multiple topics, you cannot define one callback per subions

So the new code will looks like <div class="snippet-clipboard-content notranslate

Make a lot of sence. Use jfarcand@apache.org <

I just sent you my sample at jfarcand@apache.org<

Cannot define one callback per request about atmosphere HOT 7 CLOSED

atmosphere commented on May 20, 2024

Cannot define one callback per request

from atmosphere.

Comments (7)

jfarcand commented on May 20, 2024

I like it. Will add it now. Do you have a sample you can share so I can test?

from atmosphere.

jfarcand commented on May 20, 2024

So the new code will looks like

       invokeCallback: function(response) {
        var call = function (index, func) {
            func(response);
        };


        if (typeof(jQuery.atmosphere.request.callback) == 'function') {
            jQuery.atmosphere.request.callback(response);
        } else {
            jQuery.atmosphere.log(logLevel, ["Invoking " + jQuery.atmosphere.callbacks.length + " callbacks"]);
            if (jQuery.atmosphere.callbacks.length > 0) {
                jQuery.each(jQuery.atmosphere.callbacks, call);
            }
        }
    }

Is that OK?

from atmosphere.

mjeanroy commented on May 20, 2024

Yes, this is the idea.

But maybe, "jQuery.atmosphere.callbacks" schould be call even if "jQuery.atmosphere.request.callback" is defined.
With this, you can define one callback per request and "global" callbacks for all request. I don't know if it is the best choice, but I think it can be usefull to have "global" callbacks.

How I fixed it :

invokeCallback: function(response) {
    var call = function (index, func) {
        func(response);
    };

    // Invoke global callbacks
    jQuery.atmosphere.log(logLevel, ["Invoking " + jQuery.atmosphere.callbacks.length + " callbacks"]);
    if (jQuery.atmosphere.callbacks.length > 0) {
        jQuery.each(jQuery.atmosphere.callbacks, call);
    }
    // Invoke request callback
    if (typeof(jQuery.atmosphere.request.callback) == 'function') {
        jQuery.atmosphere.request.callback(response);
    }
}

I can write a sample you can test, where can I send it to you ?

from atmosphere.

jfarcand commented on May 20, 2024

Make a lot of sence. Use [email protected]

Thanks a lot!

from atmosphere.

jfarcand commented on May 20, 2024

OK I've integrated it with a basic test. Still highly interested to a simple sample if you have a chance. Thanks a lot.

from atmosphere.

mjeanroy commented on May 20, 2024

I just sent you my sample at [email protected].
Thanks!

from atmosphere.