feat: false negative with onlyDeclarations + properties in id-match

[snitin315]

Prerequisites checklist

• I have read the contributing guidelines.

What is the purpose of this pull request? (put an "X" next to an item)

[ ] Documentation update
[X] Bug fix (template)
[ ] New rule (template)
[ ] Changes an existing rule (template)
[ ] Add autofix to a rule
[ ] Add a CLI option
[ ] Add something to the core
[ ] Other, please explain:

What changes did you make? (Give an overview)

Fixes #15123

Report invalid property names for ObjectExpressions when property: true, regardless the value of onlyDeclaration option.

Before - Online Demo

After

/* eslint "id-match": [
      "error",
      "^[^_]+$",
      {
            "properties": true,
            "onlyDeclarations": true
      }
] */

const foo = {
                foo_one: 1, // error
                bar_one: 2, // error
                fooBar: 3 // ok
     };

Is there anything you'd like reviewers to focus on?

No

[mdjermanovic]

I changed the title to feat: because this change produces more warnings, and then also removed a pair of backticks to fit the maximum of 72 characters.

[mdjermanovic]

This should be invalid, but that's another bug. It might be best to just remove this test case for now, and revisit this rule at some point. In particular, L219-L249 looks broken.

[snitin315]

We should create a separate issue to track it.

[mdjermanovic]

LGTM, thanks!