Merge branch 'binary-ids-in-sortable-tables' of https://github.com/tsutsu/ex_admin

Author: smpallen99

web/controllers/admin_association_controller.ex

@@ -48,7 +48,11 @@ defmodule ExAdmin.AdminAssociationController do
 
     selected_ids
     |> Enum.each(fn(assoc_id) ->
-      assoc_id = String.to_integer(assoc_id)
+      assoc_id = if String.match?(assoc_id, ~r/^\d+$/) do
+        String.to_integer(assoc_id)
+      else
+        assoc_id
+      end
       Ecto.build_assoc(resource, through_assoc, %{resource_key => resource_id, assoc_key => assoc_id})
       |> repo().insert!
     end)

web/static/vendor/active_admin/lib/sortable_associations.js

@@ -18,7 +18,7 @@ $(document).ready(function() {
           //$("#progress").show();
           var positions = [];
           $.each($('table.sortable tbody tr'), function(position, obj){
-            var reg = /(\w+_?)+_(\d+)/;
+            var reg = /^(\w+_?)_([-0-9a-f]+)$/i;
             var parts = reg.exec($(obj).prop('id'));
             if (parts) {
               positions = positions.concat({'id': parts[2], 'position': position});