Removes param = self.param to use self.param directly

[medha-14]

Description

Used regex accompanied with some manual checking and have removed param = self.param and have modified the code to use self.param directly.
Fixes #4484

Type of change

• New feature (non-breaking change which adds functionality)
• Optimization (back-end change that speeds up the code)
• Bug fix (non-breaking change which fixes an issue)

Key checklist:

• No style issues: $ pre-commit run (or $ nox -s pre-commit) (see CONTRIBUTING.md for how to set this up to run automatically when committing locally, in just two lines of code)
• All tests pass: $ python run-tests.py --all (or $ nox -s tests)
• The documentation builds: $ python run-tests.py --doctest (or $ nox -s doctests)

You can run integration tests, unit tests, and doctests together at once, using $ python run-tests.py --quick (or $ nox -s quick).

Further checks:

• Code is commented, particularly in hard-to-understand areas
• Tests added that prove fix is effective or that feature works

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.42%. Comparing base (e4eb82a) to head (795b8dd).
Report is 182 commits behind head on develop.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #4494      +/-   ##
===========================================
- Coverage    99.42%   99.42%   -0.01%     
===========================================
  Files          299      299              
  Lines        22739    22679      -60     
===========================================
- Hits         22608    22548      -60     
  Misses         131      131              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[valentinsulzer]

I think this change makes the code harder to read

[brosaplanella]

I think this change makes the code harder to read

For reference, see discussion in #4484

[kratman]

Couple mistakes

[kratman]

Looks good now.

To address the readability concerns from this change, I think the best approach is that we should move towards breaking up the long functions into much smaller ones. That way we can be clear that we are using class memory while also making the code easier to read. This is outside the scope of this PR though