change count to limit
This commit is contained in:
parent
ba8f84127c
commit
7aaffe5d63
|
@ -61,7 +61,7 @@ module CouchRest
|
||||||
# * The most recent 20 articles. Remember that the <tt>view_by :date</tt>
|
# * The most recent 20 articles. Remember that the <tt>view_by :date</tt>
|
||||||
# has the default option <tt>:descending => true</tt>.
|
# has the default option <tt>:descending => true</tt>.
|
||||||
#
|
#
|
||||||
# Article.by_date :count => 20
|
# Article.by_date :limit => 20
|
||||||
#
|
#
|
||||||
# * The raw CouchDB view reduce result for the custom <tt>:tags</tt> view.
|
# * The raw CouchDB view reduce result for the custom <tt>:tags</tt> view.
|
||||||
# In this case we'll get a count of the number of articles tagged "ruby".
|
# In this case we'll get a count of the number of articles tagged "ruby".
|
||||||
|
@ -131,7 +131,7 @@ module CouchRest
|
||||||
# opts<Hash>::
|
# opts<Hash>::
|
||||||
# View options, see <tt>CouchRest::Database#view</tt> options for more info.
|
# View options, see <tt>CouchRest::Database#view</tt> options for more info.
|
||||||
def first opts = {}
|
def first opts = {}
|
||||||
first_instance = self.all(opts.merge!(:count => 1))
|
first_instance = self.all(opts.merge!(:limit => 1))
|
||||||
first_instance.empty? ? nil : first_instance.first
|
first_instance.empty? ? nil : first_instance.first
|
||||||
end
|
end
|
||||||
|
|
||||||
|
|
|
@ -5,13 +5,13 @@ module CouchRest
|
||||||
@db = db
|
@db = db
|
||||||
end
|
end
|
||||||
|
|
||||||
def all_docs(count=100, &block)
|
def all_docs(limit=100, &block)
|
||||||
startkey = nil
|
startkey = nil
|
||||||
oldend = nil
|
oldend = nil
|
||||||
|
|
||||||
while docrows = request_all_docs(count+1, startkey)
|
while docrows = request_all_docs(limit+1, startkey)
|
||||||
startkey = docrows.last['key']
|
startkey = docrows.last['key']
|
||||||
docrows.pop if docrows.length > count
|
docrows.pop if docrows.length > limit
|
||||||
if oldend == startkey
|
if oldend == startkey
|
||||||
break
|
break
|
||||||
end
|
end
|
||||||
|
@ -20,13 +20,13 @@ module CouchRest
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
def key_reduce(view, count=2000, firstkey = nil, lastkey = nil, &block)
|
def key_reduce(view, limit=2000, firstkey = nil, lastkey = nil, &block)
|
||||||
# start with no keys
|
# start with no keys
|
||||||
startkey = firstkey
|
startkey = firstkey
|
||||||
# lastprocessedkey = nil
|
# lastprocessedkey = nil
|
||||||
keepgoing = true
|
keepgoing = true
|
||||||
|
|
||||||
while keepgoing && viewrows = request_view(view, count, startkey)
|
while keepgoing && viewrows = request_view(view, limit, startkey)
|
||||||
startkey = viewrows.first['key']
|
startkey = viewrows.first['key']
|
||||||
endkey = viewrows.last['key']
|
endkey = viewrows.last['key']
|
||||||
|
|
||||||
|
@ -37,7 +37,7 @@ module CouchRest
|
||||||
# we need to do an offset thing to find the next startkey
|
# we need to do an offset thing to find the next startkey
|
||||||
# otherwise we just get stuck
|
# otherwise we just get stuck
|
||||||
lastdocid = viewrows.last['id']
|
lastdocid = viewrows.last['id']
|
||||||
fornextloop = @db.view(view, :startkey => startkey, :startkey_docid => lastdocid, :count => 2)['rows']
|
fornextloop = @db.view(view, :startkey => startkey, :startkey_docid => lastdocid, :limit => 2)['rows']
|
||||||
|
|
||||||
newendkey = fornextloop.last['key']
|
newendkey = fornextloop.last['key']
|
||||||
if (newendkey == endkey)
|
if (newendkey == endkey)
|
||||||
|
@ -79,18 +79,18 @@ module CouchRest
|
||||||
|
|
||||||
private
|
private
|
||||||
|
|
||||||
def request_all_docs count, startkey = nil
|
def request_all_docs limit, startkey = nil
|
||||||
opts = {}
|
opts = {}
|
||||||
opts[:count] = count if count
|
opts[:limit] = limit if limit
|
||||||
opts[:startkey] = startkey if startkey
|
opts[:startkey] = startkey if startkey
|
||||||
results = @db.documents(opts)
|
results = @db.documents(opts)
|
||||||
rows = results['rows']
|
rows = results['rows']
|
||||||
rows unless rows.length == 0
|
rows unless rows.length == 0
|
||||||
end
|
end
|
||||||
|
|
||||||
def request_view view, count = nil, startkey = nil, endkey = nil
|
def request_view view, limit = nil, startkey = nil, endkey = nil
|
||||||
opts = {}
|
opts = {}
|
||||||
opts[:count] = count if count
|
opts[:limit] = limit if limit
|
||||||
opts[:startkey] = startkey if startkey
|
opts[:startkey] = startkey if startkey
|
||||||
opts[:endkey] = endkey if endkey
|
opts[:endkey] = endkey if endkey
|
||||||
|
|
||||||
|
|
|
@ -29,8 +29,8 @@ describe CouchRest::Database do
|
||||||
rs = @db.temp_view(@temp_view, :key => "wild")
|
rs = @db.temp_view(@temp_view, :key => "wild")
|
||||||
rs['rows'].length.should == 1
|
rs['rows'].length.should == 1
|
||||||
end
|
end
|
||||||
it "should work with a count" do
|
it "should work with a limit" do
|
||||||
rs = @db.temp_view(@temp_view, :count => 1)
|
rs = @db.temp_view(@temp_view, :limit => 1)
|
||||||
rs['rows'].length.should == 1
|
rs['rows'].length.should == 1
|
||||||
end
|
end
|
||||||
it "should work with multi-keys" do
|
it "should work with multi-keys" do
|
||||||
|
@ -42,9 +42,9 @@ describe CouchRest::Database do
|
||||||
describe "map/reduce query with _temp_view in Javascript" do
|
describe "map/reduce query with _temp_view in Javascript" do
|
||||||
before(:each) do
|
before(:each) do
|
||||||
@db.bulk_save([
|
@db.bulk_save([
|
||||||
{"beverage" => "beer", :count => 4},
|
{"beverage" => "beer", :limit => 4},
|
||||||
{"beverage" => "beer", :count => 2},
|
{"beverage" => "beer", :limit => 2},
|
||||||
{"beverage" => "tea", :count => 3}
|
{"beverage" => "tea", :limit => 3}
|
||||||
])
|
])
|
||||||
end
|
end
|
||||||
it "should return the result of the temporary function" do
|
it "should return the result of the temporary function" do
|
||||||
|
@ -109,8 +109,8 @@ describe CouchRest::Database do
|
||||||
rs = @db.view('first/test', :key => "wild")
|
rs = @db.view('first/test', :key => "wild")
|
||||||
rs['rows'].length.should == 1
|
rs['rows'].length.should == 1
|
||||||
end
|
end
|
||||||
it "should work with a count" do
|
it "should work with a limit" do
|
||||||
rs = @db.view('first/test', :count => 1)
|
rs = @db.view('first/test', :limit => 1)
|
||||||
rs['rows'].length.should == 1
|
rs['rows'].length.should == 1
|
||||||
end
|
end
|
||||||
it "should work with multi-keys" do
|
it "should work with multi-keys" do
|
||||||
|
|
|
@ -611,7 +611,7 @@ describe CouchRest::Model do
|
||||||
articles[1].title.should == 'not junk'
|
articles[1].title.should == 'not junk'
|
||||||
end
|
end
|
||||||
it "should be queryable with couchrest options" do
|
it "should be queryable with couchrest options" do
|
||||||
articles = Article.by_user_id_and_date :count => 1, :startkey => 'quentin'
|
articles = Article.by_user_id_and_date :limit => 1, :startkey => 'quentin'
|
||||||
articles.length.should == 1
|
articles.length.should == 1
|
||||||
articles[0].title.should == "even more interesting"
|
articles[0].title.should == "even more interesting"
|
||||||
end
|
end
|
||||||
|
|
|
@ -32,7 +32,7 @@ describe CouchRest::FileManager, "generating an app" do
|
||||||
CouchRest::FileManager.generate_app(@appdir)
|
CouchRest::FileManager.generate_app(@appdir)
|
||||||
end
|
end
|
||||||
it "should create an attachments directory" do
|
it "should create an attachments directory" do
|
||||||
Dir["#{@appdir}/*"].select{|x|x =~ /attachments/}.length.should == 1
|
Dir["#{@appdir}/*"].select{|x|x =~ /_attachments/}.length.should == 1
|
||||||
end
|
end
|
||||||
it "should create a views directory" do
|
it "should create a views directory" do
|
||||||
Dir["#{@appdir}/*"].select{|x|x =~ /views/}.length.should == 1
|
Dir["#{@appdir}/*"].select{|x|x =~ /views/}.length.should == 1
|
||||||
|
|
|
@ -28,7 +28,7 @@ describe CouchRest::Pager do
|
||||||
end
|
end
|
||||||
@db.bulk_save(@docs)
|
@db.bulk_save(@docs)
|
||||||
end
|
end
|
||||||
it "should yield total_docs / count times" do
|
it "should yield total_docs / limit times" do
|
||||||
n = 0
|
n = 0
|
||||||
@pager.all_docs(10) do |doc|
|
@pager.all_docs(10) do |doc|
|
||||||
n += 1
|
n += 1
|
||||||
|
@ -76,7 +76,7 @@ describe CouchRest::Pager do
|
||||||
end
|
end
|
||||||
|
|
||||||
it "should have a view" do
|
it "should have a view" do
|
||||||
@db.view('magic/number', :count => 10)['rows'][0]['key'].should == 0
|
@db.view('magic/number', :limit => 10)['rows'][0]['key'].should == 0
|
||||||
end
|
end
|
||||||
|
|
||||||
it "should yield once per key" do
|
it "should yield once per key" do
|
||||||
|
|
Loading…
Reference in a new issue