Use NoCoinsEligibleToMix instead of AllCoinsPrivate if all candidates are private

[turbolay]

Fixes #11072

As always, with this part of the codebase, not sure if this breaks something else.

The idea is that if we're L206, we passed L191. Hence, the NoCoinsEligibleToMix message is factually correct at this point (we have some non-private coins, but they are not eligible for coinjoin), but not AllCoinsPrivate (because we have some non-private coins).

[soosr]

FTR @nopara73 and I will fix every scenario including this here #11083

[turbolay]

FTR @nopara73 and I will fix every scenario including this here #11083

AFAICS #11083 doesn't include this fix.
Also, how come both things are assigned the same priority? This PR is a no-brainer bugfix, #11083 is an improvement that might take time if wanted to do correctly.

[soosr]

FTR @nopara73 and I will fix every scenario including this here #11083

AFAICS #11083 doesn't include this fix. Also, how come both things are assigned the same priority? This PR is a no-brainer bugfix, #11083 is an improvement that might take time if wanted to do correctly.

The PR is nowhere ready atm, we are waiting for @Kruwed and @MarnixCroes to gather every case, and all of them will be fixed and included in the release.

Just be clear, I am not nACKing your PR, I am just saying that there is a high chance that we will modify your solution either way to have an accurate message.

[turbolay]

Just be clear, I am not nACKing your PR, I am just saying that there is a high chance that we will modify your solution either way to have an accurate message.

Thank you for the clarification, sounds perfect to me!