Require paginate-extension for Array #305

Closed
wants to merge 1 commit into from

2 participants

@drobin

When the gem is loaded, then the paginate-method for the Array-class is not required.

This patch will fix it. Any comments? Cheers! Robin.

@drobin drobin Require paginate-extension for Array
When the gem is loaded, then the paginate-method for the Array-class is
not required.
26be3a2
@mislav
Owner

In 3.0 I stopped auto-loading the Array extension because many people got tripped up by selecting all results from the database and paginating the resulting Array, defeating the whole purpose of pagination (which should happen on db level).

Also, core extensions should be avoided if possible. I think the current behavior is alright. You have to explicitly load it if you need it.

@mislav mislav closed this Apr 10, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment