Is there anything wrong with this command?

execute if entity @a[advancements=one_mace:one_mace/mace_used] run datapack enable "file/one_mace_block.zip"
Normally I would just run it in a command block or something but I don’t have access to Minecraft right now