So you're using teleop-periodic. This means that the teleop loop gets called over and over. You can get what you want out of the joystick buttons with this concept.
Code:
if (getButton)
shoot.start()
else
shoot.stop()
This will stop the motors if the button isn't pressed. You don't need another button to stop