I'm not sure I like that this method now does two things. At the least it should be renamed to reflect it's new dual purpose. I think I'd prefer it if this method remained unchanged and the group ordering was modified and saved explicitly in the controller as a separate step.