Why is `empty($unique_holdings)` needed here? It seems unnecessary.