Rails/RedundantActiveRecordAllMethod has many many false positives

[john-h-k]

Expected behavior

Rails/RedundantActiveRecordAllMethod ignores .all when it is called with args, e.g Foo.all(blah)

Actual behavior

Rails/RedundantActiveRecordAllMethod removes .all when it is called with args, e.g Foo.all(blah), breaking the syntax.

It transforms:

Foo.baz(class: 'hello').all(type: 'div').first

into

Foo.baz(class: 'hello').(type: 'div')first

which is completely invalid syntax

Steps to reproduce the problem

See code above

RuboCop version

1.56.3 (using Parser 3.2.2.3, rubocop-ast 1.29.0, running on ruby 3.2.2) [aarch64-linux]
  - rubocop-rails 2.21.0
  - rubocop-rspec 2.23.2

[koic]

This issue is a dup with #1110 and fixed by #1114. It will be released soon (Maybe tomorrow). Please wait a moment.