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

Delete File: Can still search on files from versions more than 1 pub version back. #762

Closed
kcondon opened this issue Jul 23, 2014 · 4 comments

Comments

@kcondon
Copy link
Contributor

kcondon commented Jul 23, 2014

Tested on 7/23
Not entirely sure of the rules here but since search result card logic only shows at most 2 versions of a study: published and a draft, it makes sense that searching on a deleted file would at most show up the recent published version if it was deleted in draft and once that draft is published then no results. Confirmed with Phil.

It currently behaves correctly if 1 pub and 1 draft but once the draft is published with file deleted, the prior version containing the file is still searchable by filename.

@esotiri
Copy link
Contributor

esotiri commented Dec 19, 2014

testing scenario (ok=found):
create dataset > add file > search file -> ok
publish dataset > search file -> ok
draft dataset change metadata > search file -> on on draft & ok on published
draft dataset delete file > search file -> ok on published
publish dataset -> search file -> ok on published . This is the issue the file should not be found.
@kcondon and @scolapasta confirm that file should not be found on the last action.

@scolapasta scolapasta removed this from the TEMP milestone Jan 23, 2015
@scolapasta scolapasta modified the milestones: Beta 12 - Dataverse 4.0, In Review - Dataverse 4.0, Beta 13 - Dataverse 4.0 Feb 6, 2015
@pdurbin
Copy link
Member

pdurbin commented Feb 7, 2015

I agree this is a bug. Here's how I reproduced it:

  • create dataset (unpublished)
  • upload file (dataset still unpublished)
  • publish dataset (as v1)
  • delete file (dataset enters draft)
  • re-publish dataset (as v2, with no files)
  • determine if the published file card is still around while browsing (it is and I agree it shouldn't be)

The next step, of course, is to fix the bug. :)

@pdurbin
Copy link
Member

pdurbin commented Feb 26, 2015

As of dde9997 the bug in the scenario above is no longer reproducible. Passing to QA.

Unrelated: @kcondon if you use my scenario, you might want to look at something somewhat related. If you go to V1 of the dataset and look at the Files tab, the thumbnail image is broken (this is more evident in Chrome than Firefox). I mentioned this to @landreev and he said there might be a ticket for this.

@pdurbin pdurbin removed their assignment Feb 26, 2015
@kcondon
Copy link
Contributor Author

kcondon commented Mar 3, 2015

Works, closing. I'll test any unrelated issues in their respective unrelated tickets.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants