[BUG] lidarr_disable_sync / disable_sync in config doesn't seem to work/ be implemented

[RollForCode]

Describe the bug:

Upon editing the disable_sync feature to True in the config, import still seems to be attempted.

To Reproduce:

1. Adjust the config.ini file to set disable_sync to True, as per the config below
2. Run a download trigger from lidarr, and check the soularr logs.
3. The logs state the import has failed, and moved to failed_imports

Expected behavior:

I believe what's meant to happen is that the import shouldn't be attempted.

From what I can see in the code base, lidarr_disable_sync is assigned, but never used or implemented. (I could be wrong here). It seems there is no check of the setting for when process_completed_album() is called.

Screenshots / Log Files:

[Lidarr]
.....
disable_sync = True

System Information (please complete the following information):

• OS: Arch
• Using Docker?: Yes

[mrusse]

@insanemal Has there been any changes to this functionality in the recent updates? This was working previously.

@RollForCode We will investigate. Ty

[insanemal]

Ahhh, yes, I totally missed that in the refactoring. I'll get a patch asap

[mrusse]

Awesome ty!