[Lift] Re: Possible lift-json bug in Xml.toJson

2009-10-06 Thread Joni Freeman

Hi,

I fixed this but it will slip M6. You can build a fixed version from
branch if you need it now. Please see the changeset 'Scala XML
support...':

http://github.com/dpp/liftweb/commits/joni_wip_xml

Cheers Joni

On Oct 5, 9:22 pm, harryh  wrote:
> Xml.toJson (in M5) is converting this:
>
> http://harryh.org/img/icons/foo.png
>
> to this:
>
> JField("icon", JObject(Nil))
>
> Is there some special handling of URLs going on here that might be
> causing me problems?
>
> -harryh
--~--~-~--~~~---~--~~
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: Possible lift-json bug in Xml.toJson

2009-10-05 Thread David Pollak
On Mon, Oct 5, 2009 at 1:19 PM, Joni Freeman  wrote:

>
> Yes, that's exactly what happens here. David, do you know if this
> should be considered a bug in Scala XML or should lift-json be able to
> merge those Text() nodes?
>

I don't think it's a bug in Scala's XML representation.  I think that the
json library needs to handle this case as well as the Group() related cases
(e.g., Group(Text(..), Text(..)))


>
> Cheers Joni
>
> On Oct 5, 9:46 pm, David Pollak  wrote:
> > Hmmm looks like there might be multiple Text() nodes in the 
> node
> > and the parser isn't picking them up as one piece of text.
> > Might be worthy of a ticket.
> >
> >
> >
> > On Mon, Oct 5, 2009 at 11:31 AM, harryh  wrote:
> >
> > > More info.  If the code that generates the XML looks like this:
> >
> > > {"http://harryh.org"+theUri}
> >
> > > everything is fine, but it breaks when like this:
> >
> > > http://harryh.org{theUrl}
> >
> > > -harryh
> >
> > > On Oct 5, 2:22 pm, harryh  wrote:
> > > > Xml.toJson (in M5) is converting this:
> >
> > > > http://harryh.org/img/icons/foo.png
> >
> > > > to this:
> >
> > > > JField("icon", JObject(Nil))
> >
> > > > Is there some special handling of URLs going on here that might be
> > > > causing me problems?
> >
> > > > -harryh
> >
> > --
> > Lift, the simply functional web frameworkhttp://liftweb.net
> > Beginning Scalahttp://www.apress.com/book/view/1430219890
> > Follow me:http://twitter.com/dpp
> > Surf the harmonics
> >
>


-- 
Lift, the simply functional web framework http://liftweb.net
Beginning Scala http://www.apress.com/book/view/1430219890
Follow me: http://twitter.com/dpp
Surf the harmonics

--~--~-~--~~~---~--~~
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: Possible lift-json bug in Xml.toJson

2009-10-05 Thread Joni Freeman

Yes, that's exactly what happens here. David, do you know if this
should be considered a bug in Scala XML or should lift-json be able to
merge those Text() nodes?

Cheers Joni

On Oct 5, 9:46 pm, David Pollak  wrote:
> Hmmm looks like there might be multiple Text() nodes in the  node
> and the parser isn't picking them up as one piece of text.
> Might be worthy of a ticket.
>
>
>
> On Mon, Oct 5, 2009 at 11:31 AM, harryh  wrote:
>
> > More info.  If the code that generates the XML looks like this:
>
> > {"http://harryh.org"+theUri}
>
> > everything is fine, but it breaks when like this:
>
> > http://harryh.org{theUrl}
>
> > -harryh
>
> > On Oct 5, 2:22 pm, harryh  wrote:
> > > Xml.toJson (in M5) is converting this:
>
> > > http://harryh.org/img/icons/foo.png
>
> > > to this:
>
> > > JField("icon", JObject(Nil))
>
> > > Is there some special handling of URLs going on here that might be
> > > causing me problems?
>
> > > -harryh
>
> --
> Lift, the simply functional web frameworkhttp://liftweb.net
> Beginning Scalahttp://www.apress.com/book/view/1430219890
> Follow me:http://twitter.com/dpp
> Surf the harmonics
--~--~-~--~~~---~--~~
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: Possible lift-json bug in Xml.toJson

2009-10-05 Thread David Pollak
Hmmm looks like there might be multiple Text() nodes in the  node
and the parser isn't picking them up as one piece of text.
Might be worthy of a ticket.

On Mon, Oct 5, 2009 at 11:31 AM, harryh  wrote:

>
> More info.  If the code that generates the XML looks like this:
>
> {"http://harryh.org"+theUri}
>
> everything is fine, but it breaks when like this:
>
> http://harryh.org{theUrl}
>
> -harryh
>
> On Oct 5, 2:22 pm, harryh  wrote:
> > Xml.toJson (in M5) is converting this:
> >
> > http://harryh.org/img/icons/foo.png
> >
> > to this:
> >
> > JField("icon", JObject(Nil))
> >
> > Is there some special handling of URLs going on here that might be
> > causing me problems?
> >
> > -harryh
> >
>


-- 
Lift, the simply functional web framework http://liftweb.net
Beginning Scala http://www.apress.com/book/view/1430219890
Follow me: http://twitter.com/dpp
Surf the harmonics

--~--~-~--~~~---~--~~
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: Possible lift-json bug in Xml.toJson

2009-10-05 Thread Naftoli Gugenheim

Is there a difference in the NodeSeq that generates?

On Mon, Oct 5, 2009 at 2:31 PM, harryh  wrote:
>
> More info.  If the code that generates the XML looks like this:
>
> {"http://harryh.org"+theUri}
>
> everything is fine, but it breaks when like this:
>
> http://harryh.org{theUrl}
>
> -harryh
>
> On Oct 5, 2:22 pm, harryh  wrote:
>> Xml.toJson (in M5) is converting this:
>>
>> http://harryh.org/img/icons/foo.png
>>
>> to this:
>>
>> JField("icon", JObject(Nil))
>>
>> Is there some special handling of URLs going on here that might be
>> causing me problems?
>>
>> -harryh
> >
>

--~--~-~--~~~---~--~~
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: Possible lift-json bug in Xml.toJson

2009-10-05 Thread harryh

More info.  If the code that generates the XML looks like this:

{"http://harryh.org"+theUri}

everything is fine, but it breaks when like this:

http://harryh.org{theUrl}

-harryh

On Oct 5, 2:22 pm, harryh  wrote:
> Xml.toJson (in M5) is converting this:
>
> http://harryh.org/img/icons/foo.png
>
> to this:
>
> JField("icon", JObject(Nil))
>
> Is there some special handling of URLs going on here that might be
> causing me problems?
>
> -harryh
--~--~-~--~~~---~--~~
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
-~--~~~~--~~--~--~---