Powerlimiter calc method /set method refactor

[MalteSchm]

I started to work on the PowerLimiter code a bit. Creating a PR with the following changes:

calcPowerLimit

• refactored to reduce the nesting level of if/else statements. This makes the code easier to read in my mind and I hope you agree
• Make sure that the check for old values respects the DTU poll interval. I assumed that the old value 15 was meant to be 3x 5 seconds (with 5s being the default DTU value)

setNewPowerLimit

• refactored to reduce the nesting level of if/else statements. This makes the code easier to read in my mind and I hope you agree
• This could be a bug: Subsequent calls to this method with the same power limit would not re-trigger starting / stopping the inverter. Depending on the requested power limit this case will be triggered in cases where the user turns the inverter off / on manually.

Webapp lint fails for same reason as development branch. Tested on my setup

[MalteSchm]

any feedback on this change?
I have the fix for the webapp linting error ready but this is on a branch that contains the refactoring of these two functions. A PR will have these included so I did not submit this yet

[github-actions]

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new discussion or issue for related concerns.