Fix nested condition

[bopm]

To be updated

[eqdrs]

In my scenario, this method returns true:

[30] pry(#<Ransack::Nodes::Condition>)> parent_table.class
=> Arel::Table
[31] pry(#<Ransack::Nodes::Condition>)> parent_table.name
=> "tag_values"
[32] pry(#<Ransack::Nodes::Condition>)> attribute.name
=> "tag_values_uid"

And then, when build_correlated_subquery is called, it crashes with:

undefined method 'eq' for an instance of Array

For context, here's the attribute:

Attribute <tag_values_uid>

And the parent_table:

#<Arel::Table:0x0000000158b97680
 @klass=RequestTag(id: integer, uid: string, ...),
 @name="tag_values",
 @table_alias=nil,
 @type_caster=#<ActiveRecord::TypeCaster::Map:0x000000014d011960 @klass=RequestTag(...)>>

[bopm]

Interesting, any chance for additional details on data structure and reproduction?

[eqdrs]

In the build_correlated_subquery method, correlated_key is an array of Arel::Nodes::Equality elements:

[1] pry(#<Ransack::Adapters::ActiveRecord::Context>)> correlated_key.class
=> Array
[2] pry(#<Ransack::Adapters::ActiveRecord::Context>)> correlated_key.map(&:class)
=> [Arel::Nodes::Equality, Arel::Nodes::Equality]
[3] pry(#<Ransack::Adapters::ActiveRecord::Context>)> correlated_key.map(&:to_sql)
=> ["\"user_relationships\".\"kind\" = $1", "\"user_relationships\".\"source_uid\" = \"tasks\".\"uid\""]

[bopm]

I will take a look into that at the end of my working day today.

[bopm]

The way you can help me is by providing parts from schema.rb.related to tables included into query and ransack query itself.

[eqdrs]

Here's another example, with the data structure:

We are trying to filter the tasks by users_uid_not_in: ["uid_example"].

class Task < ApplicationRecord
  has_many :follows, primary_key: :uid, inverse_of: :followed, foreign_key: :followed_uid
  has_many :users, through: :follows, source: :follower, source_type: User.name
end

# join table
class Follow < ApplicationRecord
  belongs_to :follower, polymorphic: true, foreign_key: :follower_uid, primary_key: :uid
  belongs_to :followed, polymorphic: true, foreign_key: :followed_uid, primary_key: :uid
end

class User < ApplicationRecord
  has_many :follows, primary_key: :uid, inverse_of: :follower, foreign_key: :follower_uid
  has_many :tasks, through: :follows, source: :followed, source_type: Task.name
end

# schema.rb
create_table "follows", force: :cascade do |t|
  t.string "followed_uid", null: false
  t.string "followed_type", null: false
  t.string "follower_uid", null: false
  t.string "follower_type", null: false
  t.datetime "created_at", null: false
  t.datetime "updated_at", null: false
  t.index ["followed_uid", "followed_type"], name: "index_follows_on_followed_uid_and_followed_type"
  t.index ["follower_uid", "follower_type"], name: "index_follows_on_follower_uid_and_follower_type"
end

# Ransack query
relation = Task.all
search = relation.ransack({ users_uid_not_in: ['uid_example'] })
search.result