camelcase check for properties seems broken

[Means88]

Tell us about your environment

• ESLint Version: 4.13.1
• Node Version: 6.9.1
• npm Version: 3.10.8

What parser (default, Babel-ESLint, etc.) are you using?
default

Configuration

Written in command line directly as below.

const foo = { a_b: '' };
const { a_b } = foo;

node_modules/.bin/eslint --rule 'camelcase: [2, { "properties": "never" }]' --parser-options=ecmaVersion:6 1.js

What did you expect to happen?
exit with 0

What actually happened? Please include the actual, raw output from ESLint.

2:9  error  Identifier 'a_b' is not in camel case  camelcase

screenshots

It will not throw an error before @4.13

[platinumazure]

This is working as intended (as of the bugfix for issue #9468).

This line:

const { a_b } = foo;

Is equivalent to this line:

const a_b = foo.a_b;

The right-hand side should not be reported with properties: never, but the left-hand side should be (because you're declaring a new variable, not an object property).

You can still get at properties in an object through destructuring-- they just need to be reassigned to a camelcase variable:

const { a_b: aB } = foo;

[Means88]

@platinumazure 👍 Yes, I think you are right.