Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

This project is archived and is in readonly mode.

autosave by default

#2035

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?

Reported by Lawrence Pit · February 21st, 2009 @ 05:41 AM

State: hold
Milestone: 2.x
Assigned to: Eloy Duran Eloy Duran
Importance: none

Activity

  1. Lawrence Pit
    Lawrence Pit

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

    February 21st, 2009 @ 06:27 AM

  2. Michael Koziarski
    Michael Koziarski

    I'd like to hold off investigating this till 3.0

    February 22nd, 2009 @ 02:28 AM

  3. Eloy Duran
    Eloy Duran
    • 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.

    February 22nd, 2009 @ 04:00 PM