spree/spree — pull request #1707 by sbeam (2012-06-22 02:02:25+00:00)
Title: implement re-ordering of option values in admin UI

fixes #1700

this uses a new ajax method called `update_values_positions` on Admin/OptionTypesController. Technically this should be on a OptionValuesController, but since that doesn't exist (and it would have only the one method, everything else dealing with OptionValue is managed as nested attributes on OptionType), I didn't want to be so aggressive as to create a new controller and refactor. But, it's not maybe ideal.

RE testing, I made a stab but it seems testing sortable lists of this nature is a known problem. Even @jnicklas has given up on it, and if he can't do it my chances are slim https://groups.google.com/forum/?fromgroups#!topic/ruby-capybara/ZGLA1FAW30A

thirdly, this is against 1-1-stable because I'm unable to get master to run due to auth issues at the moment, so could not sanity check. But this should install cleanly on master at any rate.