Added convenience methods for limit and order #54

Merged
merged 2 commits into from Dec 26, 2012

Projects

None yet

3 participants

@gmaliar

I've added two convenience methods for order and limit in Parse::Query so you could just one line the whole query.

@adelevie adelevie merged commit b06db0b into adelevie:master Dec 26, 2012

1 check passed

Details default The Travis build passed
@ericcj

these are great except for the naming obscures the attr_accessors for those fields so if anyone was accessing query.limit they now get an error from that "accessor" that says ArgumentError: wrong number of arguments (0 for 1). maybe name the builder methods something else or at least rename the attributes so you can still read them off the query without it raising an error

@adelevie
Owner
@ericcj

go for it

@adelevie
Owner

@gmaliar do you have any preferences for renaming the methods you created?

@gmaliar

@adelevie, @ericcj, Sorry guys for not being attentive, no I have no preferences. :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment