As far as I understand, the problem was that using coverArt was requesting cover for each song, because each song had different coverArt, see #1636, so this will actually make it impossible to use album ID if coverArt exists which will re-introduce the issue again.
CC: @Leohige

I see. It is a bit unfortunate that different Subsonic servers behave differently. So #1636 was introduced to make gonic and subsonic to behave sanely, while this PR would make lms to work.
OpenSubsonic (https://opensubsonic.netlify.app/docs/) is establishing itself as the new open standard, and gonic is a participant of it, and so is lms, so at least it would be nice if both behave in a compatible way. I cannot say for subsonic itself since it is not open source.
One possible solution is that the option usealbumidforalbumcovers uses the album "coverArt" field to get the cover id, rather than calling getCoverArt with the album id. I will try to see whether that change would work. @Leohige : do you think that would work in your case?
Note that OpenSubsonic documentation clarifies the semantics of getCoverArt: opensubsonic/open-subsonic-api#140

I have now modified the PR to ask for the coverArt field of the reponse to getAlbum if the "Use album id for album covers" option is enabled. That should give the same benefit as #1636 while being compatible with the clarification of opensubsonic on the coverId for getCoverArt.
I have tested that this gives correct results with gonic, ampache and lms. @Leohige : if you could also test this patch would be great. Thanks!

I have also tested now that it works with navidrome as well.

Unfortunately, I can't test this right now. I'm traveling, and my computer, which I use to compile the project, is at home.