rails_admin: crash on querying models with json columns

Here is my workaround, in an initializer:

RailsAdmin::Config::Fields::Types::Json.inspect # Load before override.
class RailsAdmin::Config::Fields::Types::Json
  def queryable?
    false
  end
end

Alternatively the following code works too, and does not crash with a 500 error if you enter bad JSON data in a RailsAdmin form:

# Don't blow up RailsAdmin on JSON searches or invalid JSON formats
class RailsAdmin::Config::Fields::Types::Json
  register_instance_option :formatted_value do
    if value.is_a?(Hash) || value.is_a?(Array)
      JSON.pretty_generate(value)
    else
      value
    end
  end

  def parse_value(value)
    value.present? ? JSON.parse(value) : nil
  rescue JSON::ParserError
    value
  end
end

I improved on @sbull’s solution by making it work with the current rails_admin version (1.4.2) at the time of writing:

# Prevent json fields from being queried, since RailsAdmin cannot handle them
RailsAdmin::Config::Fields::Types::Json.inspect
class RailsAdmin::Config::Fields::Types::Json
  register_instance_option :queryable? do
    false
  end
end

The advantage of this technique is that it does not force you to eager load your app.

Note for Rails 5 (or any situation where you have an abstract model defined like ApplicationRecord), you’ll need to skip abstract classes:

  ActiveRecord::Base.descendants.each do |model|
    next if model.abstract_class?
    config.model(model.name.to_s) do
      fields_of_type(:json) { queryable false }
    end
  end

Also don’t forget you’ll probably need to eager load your models beforehand (Rails.application.eager_load!), otherwise they won’t be found as descentendants of ActiveRecord::Base when that code runs

It’s not that it should be non-queryable, but rather it should be coerced into text.

I think I found a way of hacking the search function to work with JSONB fields. It casts the entire JSONB field as text, which means your search will also include key names.

list do
  field :field_name, :string do
    queryable true
    searchable_columns [{ column: "field_name::TEXT", type: :string }]
  end
end

Hope this helps! Maybe someone else can figure out how to eliminate the key names? And maybe this could also serve as the foundation for a more general solution that can be integrated into the core project?

Edit: I don’t mean to pass the buck on this. I’ve just spent half a day trying to figure this out and I’m zonked right now.

This seems to prevent an error, but even when I put in a valid JSON value to query by, the query by doesn’t actually search the JSON field (it just returns no records), or if I filter specifically by the JSON field (an array in my case), it returns all records, regardless of a match.

@MatthiasRMS solution worked for us with small tweaks:

Rails.application.eager_load!
ActiveRecord::Base.descendants.each do |model|
  next if model.abstract_class?
  config.model(model.name.to_s) do
    exclude_fields_if do
      [:json, :jsonb].include?(type)
    end
  end
end

@jtoy I ended up making those json attrs non-queryable. Give this a try:

 ActiveRecord::Base.descendants.each do |model|
    config.model(model.name.to_s) do
      fields_of_type(:json) { queryable false }
    end
  end

None of the above worked for me. This fixed it:

RailsAdmin.config do |config|
  ApplicationRecord.descendants.each do |model|
    next if model.abstract_class?
    config.model(model.name.to_s) do
      exclude_fields_if do
        type == :json
      end

      exclude_fields_if do
        type == :jsonb
      end
    end
  end
end

@MatthiasRMS Did you eager load your models beforehand as @fny suggested?

Rails.application.eager_load!
ActiveRecord::Base.descendants.each do |model|
  next if model.abstract_class? # comment out this line if you are on Rails < 5
  config.model(model.name.to_s) do
    fields_of_type(:json) { queryable false }
  end
end

this works, thank you!

On Sep 22, 2016, at 3:11 AM, Mario Pérez notifications@github.com wrote:

@jtoy https://github.com/jtoy I ended up making those json attrs non-queryable. Give this a try:

ActiveRecord::Base.descendants.each do |model| config.model(model.name.to_s) do fields_of_type(:json) { queryable false } end end — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/sferik/rails_admin/issues/2502#issuecomment-248829117, or mute the thread https://github.com/notifications/unsubscribe-auth/AAA5v55JCy-bEDljou-OqThN7uqNXNudks5qsioXgaJpZM4GyFQ_.

Until this is fixed you could use this workaround in your configuration:

ActiveRecord::Base.descendants.each do |model|
  config.model "#{model.name}" do
    exclude_fields_if { type == :json }
  end
end