Refactor code with phpstan warning #248

[beesaferoot]

- Updated existing code using phpstan suggestions
- added missing code where needed
- deleted unused code/comments

Brief summary of the change made

Makes progress on: #248

This PR fixes some PHPStan warnings/errors. However, there are still about 18 such errors left.

Remaining errors are more tricky due to references to missing models or table fields.

Are there any other side effects of this change that we should be aware of?

Describe how you tested your changes?

Pull Request checklist

Please confirm you have completed any of the necessary steps below.

• Meaningful Pull Request title and description
• Changes tested as described above
• Added appropriate documentation for the change.
• Created GitHub issues for any relevant followup/future enhancements if appropriate.

[dmohns]

Nice. Thanks a lot!

Yeah, for the remaining ones, we need to go case-by-case to understand where they are coming from. They probably relate to some in-proper refactoring that was done in the past.