thr3ads.net - Rails - Moving pagination to the Model [Feb 2008]

If this information is useful, please help other people find it:
Share via:

Neil

2008-Feb-29 17:11 UTC

Moving pagination to the Model

I''m using will_paginate in a Post class method;

[code]def self.list(page)
  paginate :per_page => 10,
           :page => page,
           :include => :user
end[/code]

I can happily use the method on my posts controller index action;

[code]
class PostsController < ApplicationController

def index
  @posts = Post.list(params[:page])

   respond_to do |format|
     format.html # index.html.erb
     format.js
   end
 end
[/code]

But I haven''t worked out how to do it on my Users controller show
action;

[code]
class UsersController < ApplicationController

  def show
    @user = User.find(params[:id])
    @posts = @user.posts.paginate :per_page => 10,
                                  :page => params[:page]
  end
[/code]

How I do I go about using the post.list class method on the post
object within the @posts variable (therefore DRYing up the pagination
and leaving it in the model)?
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Ruby on Rails: Talk" group.
To post to this group, send email to
rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
To unsubscribe from this group, send email to
rubyonrails-talk-unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
For more options, visit this group at
http://groups.google.com/group/rubyonrails-talk?hl=en
-~----------~----~----~----~------~----~------~--~---

Ilan Berci

2008-Feb-29 22:56 UTC

head link

Re: Moving pagination to the Model

Neil wrote:
> But I haven''t worked out how to do it on my Users controller show
> action;
> 
> [code]
> class UsersController < ApplicationController
> 
>   def show
>     @user = User.find(params[:id])
>     @posts = @user.posts.paginate :per_page => 10,
>                                   :page => params[:page]
>   end
> [/code]
> 
> How I do I go about using the post.list class method on the post
> object within the @posts variable (therefore DRYing up the pagination
> and leaving it in the model)?
The 1 to many association between posts can users that AR builds for you 
(as an array) can be extended with :extend

check out ActiveRecord::Base::has_many()  :extend

module PaginationExtension
  def paginate
    #your code here
  end
end

class User < ActiveRecord::Base
  has_many :posts, :extend PaginationExtension
end

@some_user.posts.paginate

hth

ilan
-- 
Posted via http://www.ruby-forum.com/.

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Ruby on Rails: Talk" group.
To post to this group, send email to
rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
To unsubscribe from this group, send email to
rubyonrails-talk-unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
For more options, visit this group at
http://groups.google.com/group/rubyonrails-talk?hl=en
-~----------~----~----~----~------~----~------~--~---

Neil Cauldwell

2008-Mar-02 07:48 UTC

head link

Re: Moving pagination to the Model

Thanks ilan. Just checked out the AP on associated extensions, but I 
have one question; where should I keep the PaginationExtension module? 
Posts are the only thing being paginated, so I presume I''d put the 
extension at the top of the post.rb. Is this correct? Or should I create 
a pagination_extension.rb?

-- 
Posted via http://www.ruby-forum.com/.

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Ruby on Rails: Talk" group.
To post to this group, send email to
rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
To unsubscribe from this group, send email to
rubyonrails-talk-unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
For more options, visit this group at
http://groups.google.com/group/rubyonrails-talk?hl=en
-~----------~----~----~----~------~----~------~--~---

Neil Cauldwell

2008-Mar-02 07:50 UTC

head link

Re: Moving pagination to the Model

Neil Cauldwell wrote:> Thanks ilan. Just checked out the AP on associated extensions, but I 
> have one question; where should I keep the PaginationExtension module? 
> Posts are the only thing being paginated, so I presume I''d put the
> extension at the top of the post.rb. Is this correct? Or should I create 
> a pagination_extension.rb?
Should read ''Just checked out the API on associated
extensions''.
-- 
Posted via http://www.ruby-forum.com/.

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Ruby on Rails: Talk" group.
To post to this group, send email to
rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
To unsubscribe from this group, send email to
rubyonrails-talk-unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
For more options, visit this group at
http://groups.google.com/group/rubyonrails-talk?hl=en
-~----------~----~----~----~------~----~------~--~---

Ilan Berci

2008-Mar-02 16:08 UTC

head link

Re: Moving pagination to the Model

Neil Cauldwell wrote:> Thanks ilan. Just checked out the AP on associated extensions, but I 
> have one question; where should I keep the PaginationExtension module? 
> Posts are the only thing being paginated, so I presume I''d put the
> extension at the top of the post.rb. Is this correct? Or should I create 
> a pagination_extension.rb?
If that is the only model using it, then it''s fine to keep it in that 
model file..  If on the other hand, it is to be used by more than one 
model then it should be in it''s own file withing app/model or lib/

I like to embed my extension modules within the model itself as:
class Foo < ActiveRecord::Base
  module SomeExtension
    #
  end

  has_many :somethings, :extend => SomeExtension
end

hth

ilan
-- 
Posted via http://www.ruby-forum.com/.

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Ruby on Rails: Talk" group.
To post to this group, send email to
rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
To unsubscribe from this group, send email to
rubyonrails-talk-unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
For more options, visit this group at
http://groups.google.com/group/rubyonrails-talk?hl=en
-~----------~----~----~----~------~----~------~--~---

Neil Cauldwell

2008-Mar-02 19:31 UTC

head link

Re: Moving pagination to the Model

It''s all working, thanks.

Sorry if I''m missing something basic, but can I somehow reuse the 
PaginationExtension on the Post.rb itself (the model being extended 
through the has_many)? I was hoping to keep the pagination settings in 
just one place.

The pagination_extension.rb resides in the model folder;

module PaginationExtension
  def pagination(page)
    paginate :per_page => 10,
             :page => page
  end
end

But I currently need to keep a class method on the Post.rb to keep it 
paginating on itself;

class Post < ActiveRecord::Base

  def self.pagination(page)
    paginate :per_page => 10,
             :page => page,
             :include => :user
  end

end

Any way round this?




-- 
Posted via http://www.ruby-forum.com/.

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Ruby on Rails: Talk" group.
To post to this group, send email to
rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
To unsubscribe from this group, send email to
rubyonrails-talk-unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
For more options, visit this group at
http://groups.google.com/group/rubyonrails-talk?hl=en
-~----------~----~----~----~------~----~------~--~---

Ilan Berci

2008-Mar-03 02:53 UTC

head link

Re: Moving pagination to the Model

Neil Cauldwell wrote:
> 
> module PaginationExtension
>   def pagination(page)
>     paginate :per_page => 10,
>              :page => page
>   end
> end
> 
> But I currently need to keep a class method on the Post.rb to keep it 
> paginating on itself;
> 
> class Post < ActiveRecord::Base
> 
>   def self.pagination(page)
>     paginate :per_page => 10,
>              :page => page,
>              :include => :user
>   end
> 
> end
> 
> Any way round this?
class Post < ActiveRecord::Base

  module PaginationExtension
    def pagination(page, options ={})
      paginate {:per_page => 10,:page => page}.merge options
    end
  end

  has_many :whatevers, :extend => PaginationExtension

  extend PaginationExtension  # no reason we can''t extend it ourselves,
  def pagination(page, options = {})
    super(page, {:include => :user}.merge options
  end

end

NOT TESTED,NOT TESTED.. use at your own risk..
I believe this may be a little drier and meets your criteria..

hth

ilan

-- 
Posted via http://www.ruby-forum.com/.

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Ruby on Rails: Talk" group.
To post to this group, send email to
rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
To unsubscribe from this group, send email to
rubyonrails-talk-unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
For more options, visit this group at
http://groups.google.com/group/rubyonrails-talk?hl=en
-~----------~----~----~----~------~----~------~--~---

Neil Cauldwell

2008-Mar-03 08:16 UTC

head link

Re: Moving pagination to the Model

Ilan Berci wrote:> Neil Cauldwell wrote:
> 
>> 
>> module PaginationExtension
>>   def pagination(page)
>>     paginate :per_page => 10,
>>              :page => page
>>   end
>> end
>> 
>> But I currently need to keep a class method on the Post.rb to keep it 
>> paginating on itself;
>> 
>> class Post < ActiveRecord::Base
>> 
>>   def self.pagination(page)
>>     paginate :per_page => 10,
>>              :page => page,
>>              :include => :user
>>   end
>> 
>> end
>> 
>> Any way round this?
> 
> class Post < ActiveRecord::Base
> 
>   module PaginationExtension
>     def pagination(page, options ={})
>       paginate {:per_page => 10,:page => page}.merge options
>     end
>   end
> 
>   has_many :whatevers, :extend => PaginationExtension
> 
>   extend PaginationExtension  # no reason we can''t extend it
ourselves,
>   def pagination(page, options = {})
>     super(page, {:include => :user}.merge options
>   end
> 
> end
> 
> NOT TESTED,NOT TESTED.. use at your own risk..
> I believe this may be a little drier and meets your criteria..
> 
> hth
> 
> ilan
My ''has_many :posts, :extend => PaginationExtension'' lives
in the
User.rb, but the post model is the one being extended with pagination.

I tried putting ''module PaginationExtension'' and the
''extend
PaginationExtension'' in the Post.rb whilst keeping the
''has_many :posts,
:extend => PaginationExtension'' in the User.rb, but I get a syntax 
error;

app/models/post.rb:5: syntax error, unexpected tASSOC, expecting
''}''
      paginate {:per_page => 10, :page => page}.merge options


-- 
Posted via http://www.ruby-forum.com/.

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Ruby on Rails: Talk" group.
To post to this group, send email to
rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
To unsubscribe from this group, send email to
rubyonrails-talk-unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
For more options, visit this group at
http://groups.google.com/group/rubyonrails-talk?hl=en
-~----------~----~----~----~------~----~------~--~---

Maybe Matching Threads

Search for more apparently analagous threads

Rails - Feb 2008 - Moving pagination to the Model

Moving pagination to the Model

Re: Moving pagination to the Model

Re: Moving pagination to the Model

Re: Moving pagination to the Model

Re: Moving pagination to the Model

Re: Moving pagination to the Model

Re: Moving pagination to the Model

Re: Moving pagination to the Model

Maybe Matching Threads