Fix for compilation errors when HA_AUTODISCOVERY is commented out

[greenoem]

What

This PR fixes MQTT build errors when autodiscovery is disabled and mqtt is enabled as per issue #831
Needs proper testing from someone who has HA/MQTT enabled

Why

Build fails when #define HA_AUTODISCOVERY is commented out (disabled) in USER_SETTINGS.h

How

ifdefs changed to incorporate missing function definitions

[amarofarinha]

LGTM.
the pre-commit check i failing.. please read https://github.com/dalathegreat/Battery-Emulator/blob/205605dc4588ed47c3353c9106f13359241dbd93/CONTRIBUTING.md for running locally the pre-commit first

[amarofarinha]

LGTM

[greenoem]

LGTM. the pre-commit check i failing.. please read https://github.com/dalathegreat/Battery-Emulator/blob/205605dc4588ed47c3353c9106f13359241dbd93/CONTRIBUTING.md for running locally the pre-commit first

Thanks, sorry I missed the pre-commit requirement - do you want me to delete the PR, run pre-commit and resubmit?

[amarofarinha]

the PR, run pre-commit and resubmit?

you just have to run the pre commiit and commit the changes

[greenoem]

the PR, run pre-commit and resubmit?

you just have to run the pre commiit and commit the changes

Thanks, still getting used to VSCode and built in source control - think I'll go back to CLI in future! Have now run pre-commit and committed changes.

[dalathegreat]

The pre-commit still fails. Try:

If you are seeing this message in CI, reproduce locally with: pre-commit run --all-files.

[greenoem]

The pre-commit still fails. Try:

If you are seeing this message in CI, reproduce locally with: pre-commit run --all-files.

pre-commit now all set up locally, runs with result "clang-format ............ Passed" (first time it ran it made corrections). However simply can't get VSCode (or now manual git cli) to amend the commit with the updated file. Any pointers?