refactor: Refactor `CalcCbTxMerkleRootQuorums` (#4954)
refactor: Refactor `CalcCbTxMerkleRootQuorums` (#4954)

* wip

* trivial cleanups

* more refactoring/adjustments

* fix: drop incorrect assert

Ofc it's not 0 when cache is used... not sure what I was thinking about.

Co-authored-by: pasta <46139b0b16d7696468a818ef81ab82a93691f7d5@dashboost.org>
dashpay/dash,UdjinM6/dash,dashpay/dash,dashpay/dash,UdjinM6/dash,thelazier/dash,thelazier/dash,thelazier/dash,dashpay/dash,dashpay/dash,thelazier/dash,UdjinM6/dash,UdjinM6/dash,UdjinM6/dash,thelazier/dash