Rename variables that start with opt_

This is a good first issue for people who want to start contributing to this repository.

Category

Plugins

Component

All plugins

Describe the bug

The existing code uses opt_ as a prefix for optional parameter names. This is based on old recommendations in Google’s style guide.

The style guide now says that parameter names should all be written in lowerCamelCase.

You can see all of the places where the wrong naming convention is used by running npm run lint from the root of the repository.

Steps to fix

1. Run npm run lint from the root of the repository.
2. Choose a file with lint errors due to camelcase issues.
3. Replace all names: opt_someParameterName should become optSomeParameterName.
4. Make a pull request.

[lazy-geek]

@rachel-fenichel ok Also just to clarify By

Category Plugins

You mean that these changes only need to be made in Plugins folder right?

[rachel-fenichel]

If the names are conflicting, pick a new name for either the parameter or the variable name in the function–hopefully most of these are straightforward.

For lint not detecting every instance: Yes, manually using find and replace is fine as well.

Cheers, Rachel