Does Garner support binding ActiveRecord models?

[artsy]

Does Garner support binding ActiveRecord models?

[mfunaro]

After some more spelunking through the code, I see that my use case is not implemented yet. No need to look into this for me. I'm going to see if I can replicate some of the Mongoid features in ActiveRecord.

[dblock]

Would be really happy to see complete ActiveRecord support! Currently we say:

Garner provides rudimentary support for ActiveRecord. To use ActiveRecord models for Garner bindings, use Garner::Mixins::ActiveRecord::Base. You can set it up in an initializer:

require "garner/mixins/active_record"

module ActiveRecord
  class Base
    include Garner::Mixins::ActiveRecord::Base
  end
end

[fancyremarker]

Hey @mfunaro, I'm using Garner w/ ActiveRecord, but can confirm that binding to classes won't work out of the box (just instances). In order to bind to classes, the ActiveRecord mixin would need to implement ActiveRecord::Base.proxy_binding as the Mongoid mixin does. For Mongoid, this looks like:

# Only find the latest if we can order by :updated_at
return nil unless fields['updated_at']
only(:_id, :_type, :updated_at).order_by(updated_at: :desc).first

It'd be similar for ActiveRecord. Something like this would work (though it's not optimized):

return nil unless columns.map(&:name).include?('updated_at')
order('updated_at DESC').first

[mfunaro]

Thanks for the responses. @fancyremarker, I did end up doing something very similar to what you suggested to get things working in a fork. I also created an identity for ActiveRecord so that I could use Patient.identify(params[:id])

[dblock]

Looking forward to some pull requests!

[danielkummer]

👍 on this...

[ericgross]

Would love to see a PR for this 👍