This project is archived and is in readonly mode.

#2035 ✓hold
Lawrence Pit

autosave by default

Reported by Lawrence Pit | February 21st, 2009 @ 05:41 AM | in 2.x

Now that we have the autosave option available, it would be nice if that option is enabled by default for all reflections.

I.e. reflect_on_all_autosave_associations should by default select all reflections except for those which have explicitely stated reflection.options[:autosave] == false.

As that would break backwards compability, a behaviour switch needs to be introduced, so that with one config line you get this behaviour for all your models.

Are there any negatives with having autosave on by default for all reflections though?

Comments and changes to this ticket

  • Lawrence Pit

    Lawrence Pit February 21st, 2009 @ 06:27 AM

    Perhaps add autosave as a new rails default in config/initializers/new_rails_defaults to make it the default for rails 3 ?

  • Michael Koziarski

    Michael Koziarski February 22nd, 2009 @ 02:28 AM

    I'd like to hold off investigating this till 3.0

  • Eloy Duran

    Eloy Duran February 22nd, 2009 @ 04:00 PM

    • State changed from “new” to “hold”

    I agree about holding this off for now, as I don't see a particular urge to enable it by default.

    But because I don't have a strong opinion on it, maybe you should discuss your idea and the rationale behind it on the core mailing list to see what other might have to say on the subject.

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile »

<h2 style="font-size: 14px">Tickets have moved to Github</h2>

The new ticket tracker is available at <a href="https://github.com/rails/rails/issues">https://github.com/rails/rails/issues</a>

Pages