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

object and datastream straw-man services #5

Merged
merged 1 commit into from Feb 12, 2013
Merged

object and datastream straw-man services #5

merged 1 commit into from Feb 12, 2013

Conversation

cbeer
Copy link
Contributor

@cbeer cbeer commented Feb 11, 2013

Is this how we should extract business logic from the JAX-RS endpoints? Is there a better way? Are there better names for these things?

@ajs6f
Copy link
Contributor

ajs6f commented Feb 11, 2013

Looks good to me. We're still using the JCR API as our model, but I think that's a fair thing to do, at least for now.

cbeer added a commit that referenced this pull request Feb 12, 2013
object and datastream creation services
@cbeer cbeer merged commit 41ae108 into master Feb 12, 2013
@cbeer cbeer deleted the model-services branch February 12, 2013 00:40
birkland pushed a commit to birkland/fcrepo4 that referenced this pull request Jul 22, 2016
updated license header with correct year
awoods pushed a commit that referenced this pull request Sep 12, 2018
* Integration tests for acl:Append.

Test append to container and append to binary.

Fixes: https://jira.duraspace.org/browse/FCREPO-2845

* Adjust Delete check and only count deletes with quads. (#5)

* Correct check for whether a resource is a binary or not.

* Must check the resource types rather than doing an instanceof check.
* Do not cache the returned resource from the node service as this was causing failures when multiple requests were being handled by the same instance of the filter.
* Added correct mock stub return values to the unit test.

Fixes: https://jira.duraspace.org/browse/FCREPO-2858
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