Command block teleportation not working

I type in "/tp @s 120 50 30" in the command block and it is correct but if I use it it does not work. It uses Redstone connected to a button and the command block is set to needs Redstone.