New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Updating LDP membership triples #290
Conversation
public static final Property INLINED_RESOURCE = | ||
createProperty(LDP_NAMESPACE + "inlinedResource"); | ||
|
||
public static final Set<Property> ldpProperties = of(PAGE, PAGE_OF, | ||
FIRST_PAGE, NEXT_PAGE, MEMBERS_INLINED, CONTAINER, | ||
MEMBERSHIP_SUBJECT, MEMBERSHIP_PREDICATE, MEMBERSHIP_OBJECT, | ||
MEMBER_SUBJECT, INLINED_RESOURCE); | ||
MEMBERSHIP_RESOURCE, HAS_MEMBER_RELATION, INLINED_RESOURCE); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should DIRECT_CONTAINER be added to the "ldpProperties" set?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Container and DirectContainer should probably be Resources instead of Properties -- I'll make a new Set to hold them, which we may want to use when filtering out statements like "<> rdf:type ldp:Container" from updates.
… using RdfLexicon instead of literals
MEMBER_SUBJECT, INLINED_RESOURCE); | ||
FIRST_PAGE, NEXT_PAGE, MEMBERS_INLINED, MEMBERSHIP_RESOURCE, | ||
HAS_MEMBER_RELATION, INLINED_RESOURCE); | ||
public static final Set<Resource> ldpResources = of(CONTAINER, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It does not look like "ldpResources" is actually used anywhere else in the code base. What purpose does it serve here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I expect to use it to filter out rdf:type assertions that should not be persisted -- see https://www.pivotaltracker.com/story/show/66168308
Resolved with: 8068263 |
Fixes https://www.pivotaltracker.com/s/projects/684825/stories/69469312