@jordan-brough We thought about that but we kept it because the Spree ResourceController that we extend from provides update_position logic for free. Furthermore, it allowed us to re-use the templates a lot easier than trying to get rid of it. In this particular case, it feels like this is harmless.