This may be a hot spot if you're loading a lot of AR objects so I'd rather not be creating a new string object every time can you change this to:

``` ruby
Base.default_timezone == :utc ? time : time.getlocal
```