Re: [PATCH] libmsc,sms: Properly calculate and store validity and received timestamps for incoming SMS.

On Sat, Mar 8, 2014 at 5:49 PM, Holger Hans Peter Freyther holger@freyther.de wrote:

On Sat, Mar 08, 2014 at 04:53:12PM +0100, Alexander Chemeris wrote:

...

•         "(datetime('now'), %u, "
  

•         "(%s, %s, "
  

...

...

•         validity_timestamp,
  

•         received_timestamp, validity_timestamp,
  

Please do not change the created to be the received timestamp. If you want to have a received_timestamp then introduce it. In case you hurry up we can get it into schema version 4.

I don't see any immediate needs for having created timestamps, but I agree that it's a good idea to keep it. The updated patch is attached and the branch is updated.