Fixing styling issues

physionet-django/project/cloud/s3.py

@@ -1121,7 +1121,7 @@ def create_s3_access_point(project, access_point_name, bucket_name, account_id):
     """
     s3 = create_s3_control_client()
     try:
-        response = s3.create_access_point(
+        s3.create_access_point(
             AccountId=account_id,
             Bucket=bucket_name,
             Name=access_point_name,
@@ -1200,7 +1200,7 @@ def upload_project_to_S3(project):
         raise Exception(f"A bucket named {bucket_name} already exists.")
     except s3.exceptions.BucketAlreadyOwnedByYou:
         bucket_created = False
-    
+
     # Set the bucket policy only if the bucket was newly created and has controlled access
     if bucket_created and project.access_policy == AccessPolicy.CREDENTIALED:
         controlled_policy = create_controlled_bucket_policy(bucket_name)

physionet-django/project/modelcomponents/middleware.py

@@ -2,10 +2,12 @@
 
 _request_local = threading.local()
 
+
 def get_current_request():
     """Retrieve the current request from thread-local storage."""
     return getattr(_request_local, 'request', None)
 
+
 class CurrentRequestMiddleware:
     """Middleware to store the current request in thread-local storage."""
     def __init__(self, get_response):

physionet-django/project/modelcomponents/storage.py

@@ -63,7 +63,7 @@ class AWS(models.Model):
 
     class Meta:
         default_permissions = ()
-    
+
     def s3_uri(self):
         """
         Construct the S3 URI for the project.
@@ -81,7 +81,10 @@ def s3_uri(self):
             # Fetch access point name
             access_point_name = get_access_point_name_for_user_and_project(current_user, self)
             if access_point_name and "No " not in access_point_name:
-                return f's3://arn:aws:s3:us-east-1:{settings.AWS_ACCOUNT_ID}:accesspoint/{access_point_name}/{self.project.slug}/{self.project.version}/'
+                return (
+                    f's3://arn:aws:s3:us-east-1:{settings.AWS_ACCOUNT_ID}:accesspoint/'
+                    f'{access_point_name}/{self.project.slug}/{self.project.version}/'
+                )
             else:
                 print(f"Error: {access_point_name}")
                 return None
@@ -105,12 +108,12 @@ class AWSAccessPoint(models.Model):
 
 class AWSAccessPointUser(models.Model):
     access_point = models.ForeignKey(
-        AWSAccessPoint, 
+        AWSAccessPoint,
         related_name='linked_users',
         on_delete=models.CASCADE
     )
     user = models.ForeignKey(
-        'user.User', 
+        'user.User',
         related_name='aws_access_point_users',
         on_delete=models.CASCADE
     )
@@ -120,4 +123,3 @@ class Meta:
 
     def __str__(self):
         return f"User: {self.user}, Access Point: {self.access_point}"
-

physionet-django/project/views.py

@@ -1938,7 +1938,7 @@ def published_project(request, project_slug, version, subdir=''):
             s3_uri = project.aws.s3_uri()
         else:
             s3_uri = '--no-sign-request ' + project.aws.s3_uri()
-    
+
     context = {
         'project': project,
         'authors': authors,
@@ -2048,7 +2048,11 @@ def sign_dua(request, project_slug, version):
 
     if request.method == 'POST' and 'agree' in request.POST:
         DUASignature.objects.create(user=user, project=project)
-        if has_s3_credentials() and files_sent_to_S3(project) is not None and s3_bucket_has_credentialed_users(project):
+        if (
+            has_s3_credentials() 
+            and files_sent_to_S3(project) is not None 
+            and s3_bucket_has_credentialed_users(project)
+        ):
             update_data_access_point_policy(project)
         return render(request, 'project/sign_dua_complete.html', {
             'project':project})