Hey Rick,

      My proposed idea in that article requires you to actually
redefine the Ajax.Request.  If we can solidify the idea we could
submit the code as a patch to core but as of now you've gotta either
replace it in the source library itself or just redefine it after the
library but before any implementation.

Review that demo page to see how I had done it and you could probably
get it working.



On Jul 14, 1:57 pm, Rick <dun...@googlemail.com> wrote:
> hi
>
> > But leaving aside status 0, just a logical point.  The third condition
> > in your updated code:
>
> > >     return (status >= 200 && status < 300 && status != 0); <--------
>
> > ...is irrelevant and will never be evaluated.  If 200 <= x < 300, x
> > cannot be zero.
>
> i can't believe what piece rubbish of code i have written, my brain
> must have bin offline :'(
>
> i think the zero comes from that part:
> ------- line: 1499--------------------------------------
>   getStatus: function() {
>     try {
>       return this.transport.status || 0;
>     } catch (e) { return 0 }
>   }
> -----------------------------------------------------------
>
> so Firefox must leave the status undefined.
>
> @ Matt Foster : i tried your patch but it didn't work for me,
> but maybe its my fault ... maybe it was a bad day of coding for me :)
>
> i will try it again .... yes the Ajax part off prototype is a bit
> uncomfortable
>
> by the way.... sorry I am not a native English speaker, so don't be
> mad with me ;)
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"Prototype & script.aculo.us" group.
To post to this group, send email to prototype-scriptaculous@googlegroups.com
To unsubscribe from this group, send email to 
prototype-scriptaculous+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/prototype-scriptaculous?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to