Skip to content
This repository has been archived by the owner on Dec 12, 2021. It is now read-only.

treat ActiveRecord::Relation as Array #917

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

Conversation

chrosciu
Copy link

This fixes the problem with Rails 4, where ActiveRecord::Relation is no longer recognized as Array, which gives problems when defining abilities with joins - see #916

@mhenrixon
Copy link

👍 merge this quickly please is cancan dead or what?

@Loremaster
Copy link

+1 for merge!

@dgilperez
Copy link

💯

@ghost
Copy link

ghost commented Aug 23, 2013

+1 Having the exact same issue after upgrading from Rails 3.2 to 4.0. Using the cancan gem version 1.6.10.

@michaldarda
Copy link

👍

@johncalvinyoung
Copy link

About time to merge. This has been a big annoyance for me.

@ghost
Copy link

ghost commented Sep 17, 2013

+1

2 similar comments
@noiseunion
Copy link

👍

@ricardodovalle
Copy link

👍

@nacengineer
Copy link

+1 and looking forward to @ryanb coming back from sabbatical

@sproot
Copy link

sproot commented Jan 27, 2014

+1

@jcoyne
Copy link

jcoyne commented Jan 28, 2014

I'm hitting this too. Please merge.

@ghost
Copy link

ghost commented Jan 28, 2014

If Railscasts.com is any indication, Ryan Bates is not actively working on projects right now. If Ryan could delegate responsibility for CanCan to someone else, that would be wonderful. If someone here could take the existing pull requests and begin to maintain this gem again, I would be grateful. Until that time, I have stopped using CanCan in any new applications or in any Rails 4 application. I have moved to Pundit.

It would appear that this gem is becoming abandonware, if it isn't already 😦

@nacengineer
Copy link

@BigNerdRanchDan @ryanb is on sabbatical due to burnout. He'll most likely be back, but it may be some time yet. Not quite abandonware yet.

@chrosciu
Copy link
Author

This change has been included in cancan successor: https://github.com/bryanrite/cancancan.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.