On Mon, Feb 1, 2010 at 3:39 PM, Lachlan Deck <[email protected]> wrote:
> On 02/02/2010, at 10:29 AM, David Pollak wrote: > > > On Sun, Jan 31, 2010 at 7:04 PM, Jarod Liu <[email protected]> wrote: > > > >> david, > >> sorry, i closed the ticket. do you accept my proposal > >> > > If we make the change, it will subtly break all the apps that are relying > on > > the current behavior. > > Only if they're running from a snapshot :) > No, there will be breakage changing from one version of Lift to another. > > > There are two options: overload the calls with a "useGetColumnLabel" flag > or > > have some sort of global setting for the two behaviors. > > > > Do folks have thoughts? > > Make a note of the change in the release notes for the next version. It > seems like the suggested fix matches a user's modelling expectations. i.e., > if they've modelled the label different from the actual column name isn't > that what they expect to reference in code? > We are not going to do this. We don't expect our users to read release notes for subtle stuff like this. We use the Scala type system to signal breakage... if there's an API behavior change, the code should cease to compile when changing versions. > > with regards, > -- > > Lachlan Deck > > -- > 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]<liftweb%[email protected]> > . > For more options, visit this group at > http://groups.google.com/group/liftweb?hl=en. > > -- 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 [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.
