Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix build #2

Open
wants to merge 95 commits into
base: master
Choose a base branch
from
Open

Fix build #2

wants to merge 95 commits into from

Conversation

psergus
Copy link

@psergus psergus commented Feb 19, 2016

No description provided.

Zachary Scott and others added 30 commits February 24, 2014 03:04
We can announce maintenance policy in README.
Now it looks like what a gem generated with `bundle gem` would look
like.

Also:
- Change requirements back to ActiveRecord 3.1, seen as the gem is not
  ready for prime time with Rails 4;
- Silence tests;
- Indicate which license this gem has.
Modernized the gem, and a few other tweaks.
- Add build status badge
- Mention that the gem also works for Rails 3.1
Add Zachary to the authors and change the gem homepage.
Zachary Scott and others added 30 commits August 25, 2014 17:19
The values of the result hash are all reflection objects. Adding a
boolean there breaks stuff.
Remove #unscoped to avoid clearing the where_values prematurely.
The dependent destroy callback mechanism in Rails calls destroy! instead
of destroy for has_many associations. With the existing implementation
of acts_as_paranoid, this would unintentionally cause dependent objects
to be hard-destroyed. This change makes destroy and destroy! identical,
and adds a separate destroy_fully! method for full destroy.
By merging the scope, the original code would re-apply the default
non-deleted scope, causing the scope to always return zero records. Use
regular scope chaining instead.
This is to avoid conflict with minitest versions required by the
different versions of Rails.
Conflicts:
	lib/acts_as_paranoid/core.rb
…r the main model first and then its dependents" see: 60c236e?diff=split
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants