[Lift] Re: Changes to ajax handling?

2009-07-03 Thread marius d.

Yeah that what happens when you're coding late in the night ... the
null monster bites you.

Marius

On Jul 2, 7:47 pm, Kris Nuttycombe  wrote:
> Marius, that fix appears to work fine, thank you!
>
> Bloody nulls. :)
>
> Kris
>
> On Thu, Jul 2, 2009 at 7:17 AM, marius d. wrote:
>
> > Corrected and pushed. Please do an update and give it a try.
>
> > Br's,
> > Marius
>
> > On Jul 2, 8:56 am, "marius d."  wrote:
> >> On Jul 2, 12:40 am, Kris Nuttycombe  wrote:
>
> >> > To answer my own question, looking over the commit log 28595307 looks
> >> > extremely suspicious. I can revert it locally but would prefer a
> >> > mainline fix, and don't want to attempt it myself. Marius?
>
> >> > Kris
>
> >> > On Wed, Jul 1, 2009 at 3:33 PM, Kris
>
> >> > Nuttycombe wrote:
> >> > > Hi, all,
>
> >> > > Have there been recent changes made to liftAjax.js? I am encountering
> >> > > new errors from an ajaxSelect() today that were not present yesterday.
> >> > > Firebug complains:
>
> >> > > Use of getBoxObjectFor() is deprecated. Try to use
> >> > > element.getBoundingClientRect() if possible.
> >> > > [Break on this error] undefined
> >> > > liftAjax.js (line 111)
> >> > > aboutToSend.responseType is null
> >> > > lift_doAjaxCycle()()liftAjax.js (line 111)
> >> > > lift_ajaxHandler()()liftAjax.js (line 21)
> >> > > onchange(change )add_trig...Gyw%3D%3D (line 2)
> >> > > [Break on this error] if (aboutToSend.responseType.toLowerCase() === 
> >> > > "json")
>
> >> That is my commt but right now I don,t see how it affects other
> >> things ... But I,ll definitely look on it today. Note that there is no
> >> intent for ajax behavior changes .. Just an augmentation ...
>
> >> > > Thanks,
>
> >> > > Kris
--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
"Lift" group.
To post to this group, send email to liftweb@googlegroups.com
To unsubscribe from this group, send email to 
liftweb+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/liftweb?hl=en
-~--~~~~--~~--~--~---



[Lift] Re: Changes to ajax handling?

2009-07-02 Thread Kris Nuttycombe

Marius, that fix appears to work fine, thank you!

Bloody nulls. :)

Kris

On Thu, Jul 2, 2009 at 7:17 AM, marius d. wrote:
>
> Corrected and pushed. Please do an update and give it a try.
>
> Br's,
> Marius
>
> On Jul 2, 8:56 am, "marius d."  wrote:
>> On Jul 2, 12:40 am, Kris Nuttycombe  wrote:
>>
>>
>>
>> > To answer my own question, looking over the commit log 28595307 looks
>> > extremely suspicious. I can revert it locally but would prefer a
>> > mainline fix, and don't want to attempt it myself. Marius?
>>
>> > Kris
>>
>> > On Wed, Jul 1, 2009 at 3:33 PM, Kris
>>
>> > Nuttycombe wrote:
>> > > Hi, all,
>>
>> > > Have there been recent changes made to liftAjax.js? I am encountering
>> > > new errors from an ajaxSelect() today that were not present yesterday.
>> > > Firebug complains:
>>
>> > > Use of getBoxObjectFor() is deprecated. Try to use
>> > > element.getBoundingClientRect() if possible.
>> > > [Break on this error] undefined
>> > > liftAjax.js (line 111)
>> > > aboutToSend.responseType is null
>> > > lift_doAjaxCycle()()liftAjax.js (line 111)
>> > > lift_ajaxHandler()()liftAjax.js (line 21)
>> > > onchange(change )add_trig...Gyw%3D%3D (line 2)
>> > > [Break on this error] if (aboutToSend.responseType.toLowerCase() === 
>> > > "json")
>>
>> That is my commt but right now I don,t see how it affects other
>> things ... But I,ll definitely look on it today. Note that there is no
>> intent for ajax behavior changes .. Just an augmentation ...
>>
>> > > Thanks,
>>
>> > > Kris
> >
>

--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
"Lift" group.
To post to this group, send email to liftweb@googlegroups.com
To unsubscribe from this group, send email to 
liftweb+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/liftweb?hl=en
-~--~~~~--~~--~--~---



[Lift] Re: Changes to ajax handling?

2009-07-02 Thread marius d.

Corrected and pushed. Please do an update and give it a try.

Br's,
Marius

On Jul 2, 8:56 am, "marius d."  wrote:
> On Jul 2, 12:40 am, Kris Nuttycombe  wrote:
>
>
>
> > To answer my own question, looking over the commit log 28595307 looks
> > extremely suspicious. I can revert it locally but would prefer a
> > mainline fix, and don't want to attempt it myself. Marius?
>
> > Kris
>
> > On Wed, Jul 1, 2009 at 3:33 PM, Kris
>
> > Nuttycombe wrote:
> > > Hi, all,
>
> > > Have there been recent changes made to liftAjax.js? I am encountering
> > > new errors from an ajaxSelect() today that were not present yesterday.
> > > Firebug complains:
>
> > > Use of getBoxObjectFor() is deprecated. Try to use
> > > element.getBoundingClientRect() if possible.
> > > [Break on this error] undefined
> > > liftAjax.js (line 111)
> > > aboutToSend.responseType is null
> > > lift_doAjaxCycle()()liftAjax.js (line 111)
> > > lift_ajaxHandler()()liftAjax.js (line 21)
> > > onchange(change )add_trig...Gyw%3D%3D (line 2)
> > > [Break on this error] if (aboutToSend.responseType.toLowerCase() === 
> > > "json")
>
> That is my commt but right now I don,t see how it affects other
> things ... But I,ll definitely look on it today. Note that there is no
> intent for ajax behavior changes .. Just an augmentation ...
>
> > > Thanks,
>
> > > Kris
--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
"Lift" group.
To post to this group, send email to liftweb@googlegroups.com
To unsubscribe from this group, send email to 
liftweb+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/liftweb?hl=en
-~--~~~~--~~--~--~---



[Lift] Re: Changes to ajax handling?

2009-07-01 Thread marius d.



On Jul 2, 12:40 am, Kris Nuttycombe  wrote:
> To answer my own question, looking over the commit log 28595307 looks
> extremely suspicious. I can revert it locally but would prefer a
> mainline fix, and don't want to attempt it myself. Marius?
>
> Kris
>
> On Wed, Jul 1, 2009 at 3:33 PM, Kris
>
> Nuttycombe wrote:
> > Hi, all,
>
> > Have there been recent changes made to liftAjax.js? I am encountering
> > new errors from an ajaxSelect() today that were not present yesterday.
> > Firebug complains:
>
> > Use of getBoxObjectFor() is deprecated. Try to use
> > element.getBoundingClientRect() if possible.
> > [Break on this error] undefined
> > liftAjax.js (line 111)
> > aboutToSend.responseType is null
> > lift_doAjaxCycle()()liftAjax.js (line 111)
> > lift_ajaxHandler()()liftAjax.js (line 21)
> > onchange(change )add_trig...Gyw%3D%3D (line 2)
> > [Break on this error] if (aboutToSend.responseType.toLowerCase() === "json")

That is my commt but right now I don,t see how it affects other
things ... But I,ll definitely look on it today. Note that there is no
intent for ajax behavior changes .. Just an augmentation ...

> > Thanks,
>
> > Kris
--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
"Lift" group.
To post to this group, send email to liftweb@googlegroups.com
To unsubscribe from this group, send email to 
liftweb+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/liftweb?hl=en
-~--~~~~--~~--~--~---



[Lift] Re: Changes to ajax handling?

2009-07-01 Thread Kris Nuttycombe

To answer my own question, looking over the commit log 28595307 looks
extremely suspicious. I can revert it locally but would prefer a
mainline fix, and don't want to attempt it myself. Marius?

Kris

On Wed, Jul 1, 2009 at 3:33 PM, Kris
Nuttycombe wrote:
> Hi, all,
>
> Have there been recent changes made to liftAjax.js? I am encountering
> new errors from an ajaxSelect() today that were not present yesterday.
> Firebug complains:
>
> Use of getBoxObjectFor() is deprecated. Try to use
> element.getBoundingClientRect() if possible.
> [Break on this error] undefined
> liftAjax.js (line 111)
> aboutToSend.responseType is null
> lift_doAjaxCycle()()liftAjax.js (line 111)
> lift_ajaxHandler()()liftAjax.js (line 21)
> onchange(change )add_trig...Gyw%3D%3D (line 2)
> [Break on this error] if (aboutToSend.responseType.toLowerCase() === "json") {
>
>
> Thanks,
>
> Kris
>

--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
"Lift" group.
To post to this group, send email to liftweb@googlegroups.com
To unsubscribe from this group, send email to 
liftweb+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/liftweb?hl=en
-~--~~~~--~~--~--~---