Hi,

I cannot change the last pull request, so here is my suggested change.
I force all to strings

-          # I assume that field_names contains strings
-          # and that name_attribute returns a symbol
-          field_names.delete(klass.name_attribute)
+          # force to work only with strings
+         field_names.map!(&:to_s)
+          field_names.delete(klass.name_attribute.to_s)


ALex.





>> @@ -83,7 +83,10 @@ and your controller like this
>>            if can_edit?(this.first, field_names[0])
>>              field_names << "actions"
>>            end
>> -          field_names[0] = "this"
>> +          # I assume that field_names contains strings
>> +          # and that name_attribute returns a symbol
>> +          field_names.delete(klass.name_attribute)
>
> As far as symbols vs strings, Ruby 1.8.7 & Ruby 1.9.3 sometimes return
> different things so we should either check it in both or force it one way
> or another.   A manual check is fine; the only tests we have for
> hobo_bootstrap are those in hobo/integration_tests/agility_bootstrap.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Hobo/hobo_bootstrap/pull/31/files#r3216162
>