Allow ActiveFedora 12 #425
Conversation
jcoyne
force-pushed
the
update_active_fedora
branch
from
7d47c13
to
56152a6
Compare
There was a problem hiding this comment.
This seems pretty straightforward, my only question is do we need any documentation on the change from Solrizer to AF::Indexing in the README?
@awead, what was your question on it?
|
@jenlindner I don't know that we need that documentation here. Solrizer merged into ActiveFedora...so Hydra-Head had to follow suit. |
| @@ -80,18 +80,18 @@ def build_agent(name, type) | |||
| end | |||
|
|
|||
| def build_agent_resource(prefix, name) | |||
| Agent.new(::RDF::URI.new("#{prefix}##{URI.encode(name)}")) | |||
| [Agent.new(::RDF::URI.new("#{prefix}##{URI.encode(name)}"))] | |||
There was a problem hiding this comment.
why are we returning arrays now?
There was a problem hiding this comment.
Because active triples no longer supports scalars. Right @no-reply?
There was a problem hiding this comment.
I think the reason for this is probably: samvera/active_fedora@3c21a7d
Previously, RDF::Term objects were incorrectly accepted as "multiple" values in ActiveFedora, and this code was likely taking advantage of that bug.
No description provided.