MappedText real_i_set_!

[Andrew Mullins]

Should this method check that there has been a change to the data before marking it as dirty...?

https://github.com/lift/framework/blob/master/persistence/mapper/src/main/scala/net/liftweb/mapper/MappedBinary.scala#L107

Something like:

protected def real_i_set_!(value: String): String = {

  if (value != data.get) {

    data() = value

    this.dirty_?(true)

  }

  value

}

[David Pollak]

Please open a ticket and assign it to me.

--
Lift, the simply functional web framework: http://liftweb.net
Code: http://github.com/lift
Discussion: http://groups.google.com/group/liftweb
Stuck? Help us help you: https://www.assembla.com/wiki/show/liftweb/Posting_example_code

--

Visi.Pro, Cloud Computing for the Rest of Us http://visi.pro

Lift, the simply functional web framework http://liftweb.net

Follow me: http://twitter.com/dpp
Blog: http://goodstuff.im

[Andrew Mullins]

Done. http://www.assembla.com/spaces/liftweb/tickets/1189

Thanks!