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

Leased and Unpublished item found in search result of the user #1686

Closed
Karthikeya212 opened this issue Mar 3, 2017 · 9 comments
Closed

Leased and Unpublished item found in search result of the user #1686

Karthikeya212 opened this issue Mar 3, 2017 · 9 comments

Comments

@Karthikeya212
Copy link

@Karthikeya212 Karthikeya212 commented Mar 3, 2017

Descriptive summary

Server: spruce.dlib.indiana.edu
Testing: Regression
Browser: Chrome(56.0.2924.87 (64-bit))
Operating System: Windows10
Issue: An item that is leased to a user until a specified date, and is not published by the manager yet is found in the browse list and search result of the user.

Expected behavior

Actual behavior

Steps to reproduce the behavior

  1. Open two windows with spruce system, and login with a Manager and a avalon User.

  2. Using Manager window, add lease for the user for an Item.
    lease list

  3. Check that the video is unpublished.
    unpublished item

  4. Go to the User window and search for the item.
    lease_unpub item found

Related work

Link to related tickets or prior related work here.

@cjcolvar
Copy link
Member

@cjcolvar cjcolvar commented Mar 9, 2017

I couldn't reproduce this on spruce.

@Karthikeya212
Copy link
Author

@Karthikeya212 Karthikeya212 commented Jun 20, 2017

Issue found again on sodium server.

access controls

item preview

search result

@cjcolvar
Copy link
Member

@cjcolvar cjcolvar commented Jun 20, 2017

@Karthikeya212 It doesn't look like the item is "hidden" from your screenshots. Could you click the "Hide this item from search results" checkbox and see if it is still showing up?

@Karthikeya212
Copy link
Author

@Karthikeya212 Karthikeya212 commented Jun 20, 2017

@cjcolvar yes the item is not hidden, the purpose here is to show that an Item that is unpublished is visible to the user. Once we provide lease to a user, the item should be visible to him only after it is published right?

@cjcolvar
Copy link
Member

@cjcolvar cjcolvar commented Jun 20, 2017

You're right. Sorry, I didn't read over the whole ticket.

@cjcolvar
Copy link
Member

@cjcolvar cjcolvar commented Jun 21, 2017

I was not able to reproduce this issue of unpublished leases causing a user to be able to see it in search results. I was however able to reproduce the issue of published leased hidden items being visible in search results.

@cjcolvar
Copy link
Member

@cjcolvar cjcolvar commented Jun 21, 2017

@cjcolvar
Copy link
Member

@cjcolvar cjcolvar commented Jun 21, 2017

Sorry, I wasn't following the full instructions and this is reproducible and somewhat known. See comment above.

@joncameron joncameron added this to the Backlog milestone Jun 22, 2017
@joncameron joncameron removed this from the Sprint 122 (March 6-March 17) milestone Jun 22, 2017
@joncameron joncameron added this to the Someday milestone Aug 16, 2017
@joncameron joncameron removed this from the Backlog milestone Aug 16, 2017
@joncameron joncameron removed the ready label Sep 27, 2017
@davidschober
Copy link

@davidschober davidschober commented May 17, 2019

edge case, won't fix.

@joncameron joncameron removed this from the Icebox/Someday milestone Apr 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants