Indent rule doesn’t validate multiline expressions

[lo1tuma]

The following example has no error:

if (true) {
    a = (
1 +
2);
}

I would expect the following errors:

3:0  error  Expected indentation of 8 characters  indent
4:0  error  Expected indentation of 8 characters  indent

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

[message]

Also think about error messages. When i have [2, "tab"] rule enabled, error message "Expected indentation of 1 characters" doesn't trigger any clues in my mind about tabs.

[remitbri]

Likewise, in a jsx file, in the render method, no error message when writing

  render() {
    return <div>
<h2>My app </h2>
  <p>{this.state.foo}</p>
          <p>{this.state.bar}</p>
    </div>
  }

[nzakas]

This is a known issue with the current implementation. Patches welcome!

[czjvic]

Some news? It's very painful issue.

[BYK]

I tried to work on this but it deemed quite hard. I'll try another approach but I don't think I can resolve this soon.

[nzakas]

Yeah, I'd imagine this will be open for quite some time. It's pretty difficult to do correctly, but patches always welcome.

[czjvic]

If there is some progress by BYK, will you create a branch?

[BYK]

@czjvic - nothing to share at the moment unfortunately :(

[ilyavolodin]

Here's a small repro case:

Original code:

/* eslint indent: [2, 4] */
var a = b.extend(
{
    test: function() {
        console.log("test");
    }
});

Will report the following:

3:1 error Expected indentation of 4 space characters but found 0 indent
4:2 error Expected indentation of 8 space characters but found 0 indent
5:3 error Expected indentation of 4 space characters but found 0 indent
7:1 error Expected indentation of 4 space characters but found 0 indent
First attempt at autofixing will result in no error messages and the following code:

/* eslint indent: [2, 4] */
var a = b.extend(
    {
            test: function() {
            console.log("test");
    }
    });

If you run eslint again on the generated code you will get:

4:10 error Expected indentation of 8 space characters but found 0 indent
5:7 error Expected indentation of 4 space characters but found 0 indent
Running autofix again will result in completely screwed up code:

/* eslint indent: [2, 4] */
var a = b.extend(
    {
                    test: function() {
                console.log("test");
    }
    });

Every consecutive run from there on will only make things worse.

Using ESLint v2.0.0-beta.1

[benderTheCrime]

Any indication as to when we can expect the fix associated with this will be available?