Diff ESLint errors [#2490874]

As of Drupal 8 all JS needs to follow our JavaScript coding standards. For this we use ESLint to check most of our standards.

Here is the summary of running ESLint on this module JS:
✖ 49 problems (49 errors, 0 warnings)

Attached is the full list of errors found by ESLint.

Comment	File	Size	Author
#1	fix-eslint-errors-2490874.patch	5.02 KB	Bès
	diff-eslint-output.txt	3.24 KB	Bès

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

Bès CreditAttribution: Bès commented 18 May 2015 at 19:40

Issue summary:

View changes

File	Size
fix-eslint-errors-2490874.patch	5.02 KB

Here is a patch that fix the small errors. Lots of them are about bad indentation be sure to configure your editor using the JavaScript coding standards.

Comment #2

Bès CreditAttribution: Bès commented 18 May 2015 at 19:42

Status:

Active

» Needs review

Comment #3

21 May 2015 at 07:26

lhangea committed 4f62bd8 on 8.x-1.x authored by Bès
```
Issue #2490874 by Bès: Diff ESLint errors
```

Comment #4

lhangea CreditAttribution: lhangea commented 21 May 2015 at 07:27

Status:

Needs review

» Fixed

The test bot was stuck because of a fatal error with schema validation. About this one, I checked with eslint, the javascript is validated after applying the patch.

Thanks for your contribution!