2
99%
master: 99%

Ran 10 Apr 2014 09:53PM UTC

Files 11

Run time 2s

Badge

Embed ▾

Committed 10 Apr 2014 09:33PM UTC coverage: 97.357% (+1.7%) from 95.631%

Job # 1.9.3, PAGINATOR=will_paginate

Build Type

push

travis-ci

Committed by

davidcelis

Commit Message

Change interface for Rails pagination

This fixes #7 but is also related to a more dire issue. `after_filter`
was misunderstood to provide access to the response before the body had
been written. In actuality, the response has already been fully
constructed by the time `after_filter` is called, which means that the
collection has already been queried and written as json (or XML). In
order to maintain a nice interface without using after_filter, the
method signature of `paginate` has changed to act like `render`:

```rails
class API::v1::PostsController < ApplicationController
  def index
    # Uncomment this line to change the default per_page from 10 to 25:
    # params[:per_page] ||= 25
    posts = Post.all

    paginate json: posts
  end
end
```

At the end, `paginate` simply passes its arguments on to `render` so all
of `render's` options are supported. The collection is assumed to live
either in the argument hash's `:json` key or an `:xml` key.

Signed-off-by: David Celis <me@davidcel.is>

Run Details

221 of 227 relevant lines covered (97.36%)

7.58 hits per line

davidcelis / api-pagination / 45 / 2
99%
master: 99%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 45.2 (1.9.3, PAGINATOR=will_paginate)

davidcelis / api-pagination / 45 / 2 99% master: 99%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 45.2 (1.9.3, PAGINATOR=will_paginate)

davidcelis / api-pagination / 45 / 2
99%
master: 99%

README BADGES
x