Closed kjetilk closed 9 years ago
Happy to merge this if you change from_attean
to check does()
on the Attean::API::*
roles, instead of checking isa()
on the concrete classes.
Sure!
BTW, does Nil exist in Attean?
No. Nil was needed to cover up a bad design decision in Trine: the need to represent the default graph. In Attean, models are full quad/graph stores without any notion of a default (unnamed) graph. It's the query planning process that converts the concept of a default graph into a query plan that references explicit graph names in the model.
OK, cool. I removed that, then. :-)
I don't know if this makes sense at all, but it fixed my immediate problem. :-)
It should probably be a coercion, or be done in some other way entirely.