Skip to content
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

Removing JCR leakage #3

Closed
wants to merge 3 commits into from
Closed

Removing JCR leakage #3

wants to merge 3 commits into from

Conversation

escowles
Copy link
Contributor

@awoods
Copy link

awoods commented Apr 23, 2015

I assume this PR is dependent on: fcrepo/fcrepo#781

@escowles
Copy link
Contributor Author

Yes, this PR depends on fcrep4/fcrepo4#781

}

final HttpResourceConverter subjects = new HttpResourceConverter(session, UriBuilder.fromUri(baseURL));
auditResource.replaceProperties(subjects, m, new RdfStream(), containerService);
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This isn't working for some reason -- I'm digging around in the RDF machinery trying to figure out why, but no luck so far. I did fix the HttpResourceConvertor to use the audit resource URI for UriBuilder.forUri(), but that didn't fix it.

@escowles
Copy link
Contributor Author

I put this in the JIRA ticket, but thought it would be good to have here to: This PR depends on fcrepo/fcrepo#781 (adding FedoraResource.setURIProperty method) and fcrepo/fcrepo#782 (updating DefaultIdentifierTranslator)

@awoods
Copy link

awoods commented Apr 24, 2015

Resolved with: ace483a

@awoods awoods closed this Apr 24, 2015
@awoods awoods deleted the removeJCRleakage branch April 24, 2015 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants