This is correct, no need to fix. Moreover, the fix relies on secondary to be None but the original purpose was not to rely on defaults