1.Ну дело не в "on" и "one", но да, это криво)
2. Я понимаю, но если к каждому изображению присваивать свои id, то нужно так же и скрипт повторять с новым id? А если сделать классом, то при нажатии на кнопку все изображения с этим классом изменят путь.
Ну это вроде галерей, на странице отображаются например 30 изображений светильников, и под ними нужно сделать кнопки изменения цвета светильника, но не в карточке, а на самой странице, где все 30 изображений.