Adding start/offset option for queries

[no-reply]

Solr supports paginated results with the start option. This adds that
to the find/query methods.

[dchandekstark]

Recommend using loaded?' here instead of@loaded'

[no-reply]

changed.

It should be noted that @loaded is used throughout this module. These _value methods could probably stand to be refactored. A lot of repetition.

[jcoyne]

Can we call it offset so it matches ActiveRecord? See http://guides.rubyonrails.org/active_record_querying.html#limit-and-offset

[no-reply]

I like 'offset' better, in general, but I called it start so it matches Solr. It looks like we're already not matching Solr's parameters on 'order'/'sort', though.

What's the thinking? Better to match AR or Solr? Or should there be an alias?

[tpendragon]

What's the thinking? Better to match AR or Solr? Or should there be an alias?

Consistency is key, and since AF is already matching AR's order/sort my vote's for #offset without an alias.

[dchandekstark]

+1 to matching AR, per https://wiki.duraspace.org/display/hydra/Hydra+Stack+-+The+Hierarchy+of+Promises#HydraStack-TheHierarchyofPromises-ActiveFedora:ActiveModelPatternforFedora.

[mjgiarlo]

+1 to making Hydra feel more like vanilla Rails and less like niche cultural heritageware.

[no-reply]

Done.