Hey Derek, I know this confused me too. The current MappedText for SQL Server is being implemented as Varchar(MAX) in the database.
I always to m.description.toString and this way I get the clob reference. Excuse my ignorance, but shouldn¹t MappedText actually be using ³text² column type? Cheers, Tim On 17/03/2009 19:19, "Derek Chen-Becker" <[email protected]> wrote: > Actually, now I'm more confused. In the MappedText source it's using the JDBC > type VARCHAR. In MetaMapper's buildMapper method it appears that VARCHAR > should be retrieved via a resultSet.getString call, which should return the > String form of the data. I confirmed this in jTDS source: > > http://jtds.cvs.sourceforge.net/viewvc/jtds/jtds/src/java/net/sourceforge/jtds > /jdbc/Support.java?revision=1.1&view=markup&pathrev=MAIN > > Line 289. I have no idea how you're getting a real ClobImpl object, since the > source seems to indicate that it should just be a String: > > class MappedText[T<:Mapper[T]](val fieldOwner: T) extends MappedField[String, > T] > > Derek > > On Tue, Mar 17, 2009 at 2:08 PM, Derek Chen-Becker <[email protected]> > wrote: >> OK, check out the wip-dcb-datetime branch and test that. >> >> It looks like the CLOB object that jTDS returns doesn't override toString to >> return the contents of the CLOB. It does, however, implement the >> java.sql.Clob interface, so it may be possible to match that somehow and make >> it work. I'm diving into parts of Mapper that I haven't really worked with >> before, so no guarantees. Just to confirm, can you change your snippet to >> use: >> >> "description" -> m.description.toString >> >> and see if you get the same thing? >> >> Thanks, >> >> Derek >> >> >> >> >> On Tue, Mar 17, 2009 at 1:05 PM, Timothy Perrett <[email protected]> >> wrote: >>> >>> Hey Derek, >>> >>> Awesome - thanks. I knew the DataTime one would be a simple fix - I >>> just don't know enough about mapper. >>> >>> #17 is the one thats really hurting me right now... if you could fix >>> that I would be sooo grateful! >>> >>> Cheers, Tim >>> >>> On Mar 17, 6:02 pm, Derek Chen-Becker <[email protected]> wrote: >>>> > OK, actually, the change is in SqlServerDriver. I've made the change and >>>> I'm >>>> > running a build before committing. Once I push the branch could you check >>>> it >>>> > out and test? I'm looking at #17 right now, too. >>>> > >>>> > Derek >>>> > >>>> > On Tue, Mar 17, 2009 at 12:44 PM, Derek Chen-Becker >>>> > <[email protected]>wrote: >>>> > >>>>> > > I think the DateTime issue should be fixed pretty easily in >>>>> MappedDateTime >>>>> > > itself. Let me make a new branch and make a minor change. >>>> > >>>>> > > Derek >>>> > >>>>> > > On Tue, Mar 17, 2009 at 12:09 PM, Timothy Perrett >>>>> <[email protected] >>>>>> > > > wrote: >>>> > >>>>>> > >> *bump* >>>> > >>>>>> > >> Al, any progress on these? >>>> > >>>>>> > >> Cheers, Tim >>>> > >>>>>> > >> On Mar 10, 11:04 am, Timothy Perrett <[email protected]> >>>>>> wrote: >>>>>>> > >> > Is someone able to take ownership of these tickets? >>>> > >>>>>>> > >> > Cheers, Tim >>>> > >>>>>>> > >> > On Mar 9, 5:52 pm, Tim Perrett <[email protected]> wrote: >>>> > >>>>>>>> > >> > > Guys, >>>> > >>>>>>>> > >> > > Just logged a couple of bugs for SQL Server drivers: >>>> > >>>>>>>> > >> > >>>>>>>> >http://liftweb.lighthouseapp.com/projects/26102/tickets/18-sql-server. >>>>>> > >> .. >>>> > >>>>>>>> > >> > >>>>>>>> >http://liftweb.lighthouseapp.com/projects/26102/tickets/17-mappedtext. >>>>>> > >> .. >>>> > >>>>>>>> > >> > > Can someone take a look? This really is not my speciality. >>>> > >>>>>>>> > >> > > Cheers, Tim >>> >> > > > > > --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Lift" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/liftweb?hl=en -~----------~----~----~----~------~----~------~--~---
