IQSS / dataverse

Open source research data repository software
http://dataverse.org
Other
882 stars 494 forks source link

Dataset UI - Terms Tab - Restricted Files section is displayed even when there is no data present #10879

Closed ofahimIQSS closed 1 month ago

ofahimIQSS commented 1 month ago

What steps does it take to reproduce the issue? Steps:

  1. Go to the dataverse site (https://dataverse-internal.iq.harvard.edu/)
  2. Login using valid credentials | if you are not registered, please create an account
  3. Create a New Dataset
  4. Under Description, enter test
  5. Subject: Pick any subject
  6. Save the data set.
  7. Navigate to the Terms Tab and look under "Restricted Files+Terms of Access"

Which version of Dataverse are you using? Dataverse Prod and Internal

Any related open or closed issues to this bug report? N/A

Screenshots: image

ofahimIQSS commented 1 month ago

Reproduced in production (v. 6.3 build 8c99a74+10614+10720+10808+10797+10820-iqss) image v. 6.3 build 8c99a74+10614+10720+10808+10797+10820-iqss Reproduced in Demo 6.4 Release Candidate image

sekmiller commented 1 month ago

I'm not seeing in on Demo which has the 6.4 release candidate, I think

pdurbin commented 1 month ago

I can reproduce it on demo (yes, it's running a 6.4 release candidate).

I created the dataset in the root collection but I doubt that matters.

Here's a screenshot:

Screenshot 2024-09-27 at 4 25 57 PM

The bug is in JSF, which we're throwing away 🗑️ in favor of React so I'm not sure if this is worth fixing. /me ducks

sekmiller commented 1 month ago

My working theory is that this is due to the "Request Access to Restricted Files" being true by default. The reason I wasn't seeing it in my prior datasets was because I was using a template on create where "Request Access...." was false.

sekmiller commented 1 month ago

Mentioned this in standup on Monday - the fix was simple enough that there's no need for further discussion. (My original thought was that it could be complex or require changing a default dataset setting. fortunately that is not the case)