Interpolated variables not enclosed in braces are not noticed

[rubocop]

Interpolated variables not enclosed in braces are not noticed

[bbatsov]

This certainly is a bug, that we'll address soon. On a related note - interpolating variables like this is considered bad style(it's a remnant of the ancient Ruby history), so we should add a check for this as well.

[bbatsov]

@jonas054 Here's where the problem with the existing check lies:

[3] pry(main)> Ripper.lex('puts "test#@@top"')
=> [[[1, 0], :on_ident, "puts"],
 [[1, 4], :on_sp, " "],
 [[1, 5], :on_tstring_beg, "\""],
 [[1, 6], :on_tstring_content, "test"],
 [[1, 10], :on_embvar, "#"],
 [[1, 11], :on_cvar, "@@top"],
 [[1, 16], :on_tstring_end, "\""]]
[4] pry(main)> Ripper.lex('puts "test#{@@top}"')
=> [[[1, 0], :on_ident, "puts"],
 [[1, 4], :on_sp, " "],
 [[1, 5], :on_tstring_beg, "\""],
 [[1, 6], :on_tstring_content, "test"],
 [[1, 10], :on_embexpr_beg, "\#{"],
 [[1, 12], :on_cvar, "@@top"],
 [[1, 17], :on_embexpr_end, "}"],
 [[1, 18], :on_tstring_end, "\""]]

The code in StringLiterals currently checks only for :on_embexpr_beg.

[jonas054]

Thanks, I'll try to solve it tonight.

[bbatsov]

That won't be needed - I got a bit of an inspiration and fixed it myself :-) I've also implemented a cop to warn against the use of interpolated variables.

[jonas054]

Great!