Style/UnneededCondition auto-correct yields invalid syntax on un-parenthesized multiline statements

[joeadcock]

The Style/UnneededCondition auto-correct can yield syntax errors when a single statement in an else branch spans more than one line without parenthesis.

Example:

def foo(bar = false)
  if bar
    bar
  else
    OpenStruct.new one: 1,
                   two: 2
  end
end

---

Expected behavior

This should auto-correct to

def foo(bar = false)
  bar || OpenStruct.new(one: 1,
                        two: 2)
end

or

def foo(bar = false)
  bar || OpenStruct.new one: 1, two: 2
end

Actual behavior

Rubocop auto-corrected to (invalid syntax):

def foo(bar = false)
  bar || OpenStruct.new one: 1,
                        two: 2
end

Steps to reproduce the problem

# test.rb
def foo(bar = false)
  bar || OpenStruct.new one: 1,
                        two: 2
end

rubocop --safe-auto-correct test.rb 

RuboCop version

$ rubocop -V
0.60.0 (using Parser 2.5.1.2, running on ruby 2.5.1 x86_64-darwin17)

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contribution and understanding!

[stale]

This issues been automatically closed due to lack of activity. Feel free to re-open it if you ever come back to it.