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

Adding topic URI to dataset context and velocity template #66

Merged
merged 1 commit into from May 22, 2013
Merged

Conversation

escowles
Copy link
Contributor

Uses the Dataset Context object to store the topic URI. The Context object hold key-value pairs (with boolean or Object values), so it could also be used to pass a base URL, or other values the display code needs to fulfill the request.

HtmlProvider now uses the topic URI to choose the correct type and corresponding velocity template.

cbeer added a commit that referenced this pull request May 22, 2013
Adding topic URI to dataset context and velocity template
@cbeer cbeer merged commit dd8a54f into master May 22, 2013
new HttpGraphSubjects(FedoraNodes.class, uriInfo);
Dataset dataset = resource.getGraphStore(subjects).toDataset();
String uri = null;
try {
Copy link
Contributor

Choose a reason for hiding this comment

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

I wonder if we can push this context logic into getGraphStore, but it's a property of Dataset..

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