Style/WordArray cop would be more inteligent in case of multidimensional array

[deepj]

I have some cases of multidimensional array where (from my perspective) default Style/WordArray cop makes the code less readable and confusing. See this example:

CONTENT_TYPE = [
    ['TSR', 'Teaser'],
    ['PRO', 'Promo'],
    ['RTG', 'Rating Tag'],
    ['RTG-F', 'Rating Tag for Feature'],
]

Rubocop forces me to change it to:

CONTENT_TYPE = [
    %w[TSR Teaser],
    %w[PRO Promo],
    ['RTG', 'Rating Tag'],
    ['RTG-F', 'Rating Tag for Feature'],
]

I don't think this helps to better readability. I would wish if this would be valid and fine for default Style/WordArray cop

CONTENT_TYPE = [
    ['TSR', 'Teaser'],
    ['PRO', 'Promo'],
    ['RTG', 'Rating Tag'],
    ['RTG-F', 'Rating Tag for Feature'],
]

Please, consider this as an example ^^.

[tejasbubane]

I would argue that it looks confusing because your arrays happen to have some strings with spaces (which cannot be converted to WordArray). Had it been as follows, it is not confusing anymore:

CONTENT_TYPE = [
    %w[TSR Teaser],
    %w[PRO Promo]
]

However one solution could be to add a configuration flag which disable cop for nested arrays:

# EnabledForNestedArrays: false
# good
CONTENT_TYPE = [
    ['TSR', 'Teaser'],
    ['PRO', 'Promo']
]

# EnabledForNestedArrays: true
# bad
CONTENT_TYPE = [
    ['TSR', 'Teaser'],
    ['PRO', 'Promo']
]

Although I don't prefer adding more complexity with flags, I can put together a PR if maintainers are willing to accept this.

[deepj]

@tejasbubane I appreciate your comment but the arrays cannot be changed.

CONTENT_TYPE = [
    ['TSR', 'Teaser'],
    ['PRO', 'Promo'],
    ['RTG', 'Rating Tag'],
    ['RTG-F', 'Rating Tag for Feature'],
]

What I need in this case the cop ignores this kind of arrays. I think your suggestion to turn off it for them is one of the solution.

@koic What do you think?

[deepj]

@fatkodima Thank you!