Added Hydra::AccessControls::Permission#to_hash for better compatibility...

[dchandekstark]

... with hash-based permissions

Added unit test module for Hydra::AccessControls::Permission

If merged, hydra-head version should be incremented to 6.5.0.

[jcoyne]

Better compatibility with what? Can you be more specific? It doesn't look like any behavior actually changed here, did it?

[dchandekstark]

@jcoyne in HH 6.4 the #permissions= setter (and #permissions_attributes=) does not work with the data that the #permissions getter returns -- i.e., an array of Permission objects. This could be patched in more than one way. Specifically the setter blows up now b/c tries to call #with_indifferent_access on each "permission" object in the collection. At least with this method you can do:

obj1.permissions_attributes = obj2.permissions.collect { |p| p.to_hash }

I brought this general issue up on hydra-tech a while back - the issue of the HH 6.4 permissions getter/setter mismatch - but it got no traction.

[jcoyne]

@dchandekstark that sounds good. Shouldn't you be testing to_hash though?

[dchandekstark]

@jcoyne I guess you're right. It gets exercised by using #[] in the tests, but probably should have an explicit test. I will update the PR.