forked from alxp/islandora
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #574 from DiegoPino/7.x
breadcrumbs xacml compliant
- Loading branch information
Showing
1 changed file
with
39 additions
and
26 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
6cdfb43
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.
Missing
isPartOf
no?6cdfb43
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.
Uh-oh!
@DiegoPino, did we miss porting
or <info:fedora/' . $pid . '> <fedora-rels-ext:isPartOf> $parentObject
to the SPARQL query?6cdfb43
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.
6cdfb43
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.
Adding isConstituentOf probably would be a sound idea, not sure what other folks might say as a lot of people are still on holidays. I believe isPartOf is for legacy reasons iirc.
6cdfb43
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.
adding isConstituentOf
👍
6cdfb43
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.
Ok, doing so. But having no particular result order will give as result always the parent Collection first(first added predicate to the triple-store). the "order by $title DESC" of the previous itql should/could also be preserved? order by make things always slower. If no php sorting or precedence is being done, then maybe "Limit 1" is faster?Just thoughts. Thanks.
6cdfb43
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.
added #575