-
Notifications
You must be signed in to change notification settings - Fork 71
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
User other than admin not able to add Basic Collection or Basic Image content successfully #78
Comments
I suspect https://github.com/Islandora-Labs/islandora/blob/7.x-2.x/camel/commands/src/Services/Collection/NodeToSparql.php#L139-L144 is the culprit. I was aware of it enough to make that comment, lol. |
@daniel-dgi, is this related to how Drupal manages UID or for entities in general?. So if enabling UUID for files, users, other nodes, will get us to the same problem. I think we should handle this before serialising to JSON on the drupal side by getting all entity types that have an UUID representation an make them full URI's. Camel should get clean data…will look for the "how". |
@DiegoPino Agreed. We should clean it up before leaving drupal land, since we'll have access to everything we need without having to create a drupal service to expose that functionality. This actually touches on another issue where maybe we should be doing the node -> sparql update conversion in Drupal land, but that's out of scope for this ticket. Let's see what happens with this, first. |
@daniel-dgi, i agree. Right now i'm looking at "where" drupal does the rdfa displaying using this triples, to understand if the full uri is available inside some structure of it's dynamically generated on display. Will find this out first. |
Linking this discussion in: https://groups.google.com/forum/#!topic/islandora-dev/dr-1cwofVRk Looks like unless there's any objections, we should be doing this on the drupal side. |
Closing d7 related issues since we're moving to 8. |
If a new Drupal user is created:
/opt/apache-karaf-3.0.4/etc/islandora.cfg
is updated to reflect this userbundle:restart
The text was updated successfully, but these errors were encountered: