Skip to content

Commit

Permalink
Merge pull request #728 from isb-cgc/isb-cgc-prod-sp
Browse files Browse the repository at this point in the history
Hotfix for multiple user whitelist access in IGV
  • Loading branch information
s-paquette authored Aug 22, 2018
2 parents 3a24001 + 4f071f2 commit f57263f
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
2 changes: 1 addition & 1 deletion cohorts/file_helpers.py
Original file line number Diff line number Diff line change
Expand Up @@ -291,7 +291,7 @@ def cohort_files(cohort_id, inc_filters=None, user=None, limit=25, page=1, offse
whitelist_found = False
# If this is a controlled-access entry, check for the user's access to it
if item['access'] == 'controlled' and access:
whitelists = item['acl'].split(',')
whitelists = item['acl'].split(';')
for whitelist in whitelists:
if whitelist in access:
whitelist_found = True
Expand Down
1 change: 1 addition & 0 deletions cohorts/views.py
Original file line number Diff line number Diff line change
Expand Up @@ -2185,6 +2185,7 @@ def export_data(request, cohort_id=0, export_type=None, export_sub_type=None):
# Some files only have case barcodes, but some have sample barcodes. We have to make sure
# to export any files linked to a case if any sample from that case is in the cohort, but
# if files are linked to a sample, we only export them if the specific sample is in the cohort.

if export_type == 'file_manifest':
query_string_base = """
SELECT md.sample_barcode, md.case_barcode, md.file_name_key as cloud_storage_location,
Expand Down

0 comments on commit f57263f

Please sign in to comment.