ActiveRecord 5.2 Support

[bbonislawski]

Did someone succesfully run ransack with AR 5.2? Build is failing on rails master branch(actually it's failing everywhere). Is someone working on that?

[halo]

It's an underlying issue over here.

[pioz]

Also in lib/ransack/adapters/active_record/context.rb:282 pass 3 args to JoinDependency.new but with Rails 5.2.0.beta2 seems the constructor take 4.

[halo]

@pioz Haha, I just ended up in the same rabbit hole as you. You win :) Let me link to the lines:

3 arguments passed here:

ransack/lib/ransack/adapters/active_record/context.rb

Lines 282 to 284 in 724d020

	join_dependency = JoinDependency.new(
	relation.klass, association_joins, join_list
	)

4 required here:

https://github.com/rails/rails/blob/38b2904f2c9d89868e96edd71c5f3143ca235a88/activerecord/lib/active_record/associations/join_dependency.rb#L91

[halo]

Also, this override in ransack accepts an argument:

ransack/lib/ransack/helpers/form_builder.rb

Line 9 in 724d020

def value(object)

But the original in Rails no longer takes an argument:

https://github.com/rails/rails/blob/38b2904f2c9d89868e96edd71c5f3143ca235a88/actionview/lib/action_view/helpers/tags/base.rb#L38

[pioz]

Also the join_root method used in

ransack/lib/ransack/adapters/active_record/ransack/context.rb

Line 34 in 724d020

@base = @join_dependency.join_root

is become protected:

https://github.com/rails/rails/blob/c5ab6e51a7b9ee05a2d262a72c7130b9c1d1b0ce/activerecord/lib/active_record/associations/join_dependency.rb#L159

[bbonislawski]

So I have working version for me:
https://github.com/spark-solutions/polyamorous
and
https://github.com/spark-solutions/ransack

[scarroll32]

activerecord-hackery/polyamorous#28

[scarroll32]

Not sure if it is the same issue, but a branch of Ransack based on Polyamorous is available here for testing. #858

[scarroll32]

Closed as fixed by #868