Made owner deed list show owner's collection's deeds #3264
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Resolves #3261
Previously, the "show more" (deed list) page for a collection owner was showing all of the FTP deeds instead of just deeds related to that owner.
This is because only
options[:user_id]
was being passed to the deed list page, which wasnil
, asoptions[:owner]
is what carried the owner's id. Nowoptions[:owner]
is passed ifoptions[:user_id]
isnil
.In the deed list, it's checked if
@user
is an owner or not, and if it is, the deeds to display will be deeds on that owner's collections, just like on the shortened deed list.