Backend: dynamically generate CSV filenames based on challenge and phase details for submission download (#4811)

* Enhance submission download functionality: dynamically generate CSV filenames based on challenge and phase details.

* Refactor loop to use 'let' for block scoping and add initial state for page and phases in challenge controller tests

Author: RishabhJain2018

apps/challenges/views.py

@@ -2300,7 +2300,7 @@ def download_all_submissions(
 
                 response = HttpResponse(content_type="text/csv")
                 response["Content-Disposition"] = (
-                    f"attachment; filename={filename}"
+                    f'attachment; filename="{filename}"'
                 )
                 writer = csv.writer(response)
                 writer.writerow(
@@ -2447,9 +2447,15 @@ def download_all_submissions(
                 submissions = ChallengeSubmissionManagementSerializer(
                     submissions, many=True, context={"request": request}
                 )
+
+                # Use helper function to generate filename
+                filename = get_submissions_csv_filename(
+                    challenge, challenge_phase
+                )
+
                 response = HttpResponse(content_type="text/csv")
                 response["Content-Disposition"] = (
-                    "attachment; filename=all_submissions.csv"
+                    f'attachment; filename="{filename}"'
                 )
                 writer = csv.writer(response)
                 writer.writerow(
@@ -2539,7 +2545,7 @@ def download_all_submissions(
 
                 response = HttpResponse(content_type="text/csv")
                 response["Content-Disposition"] = (
-                    f"attachment; filename={filename}"
+                    f'attachment; filename="{filename}"'
                 )
                 writer = csv.writer(response)
                 fields = [fields_to_export[field] for field in request.data]

frontend/src/js/controllers/challengeCtrl.js

@@ -2173,6 +2173,17 @@
 
         vm.downloadChallengeSubmissions = function() {
             if (vm.phaseId) {
+                // Generate dynamic filename based on challenge and phase info
+                var challengeName = vm.page.title.replace(/\s+/g, '_').replace(/\//g, '_');
+                var phaseName = '';
+                for (var i = 0; i < vm.phases.results.length; i++) {
+                    if (vm.phases.results[i].id == vm.phaseId) {
+                        phaseName = vm.phases.results[i].name.replace(/\s+/g, '_').replace(/\//g, '_');
+                        break;
+                    }
+                }
+                var filename = 'all_submissions_' + challengeName + '_' + vm.challengeId + '_' + phaseName + '_' + vm.phaseId + '.csv';
+                
                 parameters.url = "challenges/" + vm.challengeId + "/phase/" + vm.phaseId + "/download_all_submissions/" + vm.fileSelected + "/";
                 if (vm.fieldsToGet === undefined || vm.fieldsToGet.length === 0) {
                     parameters.method = "GET";
@@ -2182,7 +2193,7 @@
                             var anchor = angular.element('<a/>');
                             anchor.attr({
                                 href: 'data:attachment/csv;charset=utf-8,' + encodeURI(details),
-                                download: 'all_submissions.csv'
+                                download: filename
                             })[0].click();
                         },
                         onError: function(response) {
@@ -2195,7 +2206,7 @@
                 else {
                     parameters.method = "POST";
                     var fieldsExport = [];
-                    for(var i = 0 ; i < vm.fields.length ; i++) {
+                    for(let i = 0 ; i < vm.fields.length ; i++) {
                         if (vm.fieldsToGet.includes(vm.fields[i].id)) {
                             fieldsExport.push(vm.fields[i].id);
                         }
@@ -2207,7 +2218,7 @@
                             var anchor = angular.element('<a/>');
                             anchor.attr({
                                 href: 'data:attachment/csv;charset=utf-8,' + encodeURI(details),
-                                download: 'all_submissions.csv'
+                                download: filename
                             })[0].click();
                         },
                         onError: function(response) {

frontend/tests/controllers-test/challengeCtrl.test.js

@@ -3516,6 +3516,12 @@ describe('Unit tests for challenge controller', function () {
                 { id: 'participant_team' },
                 { id: 'status' }
             ];
+            vm.page = { title: 'Test Challenge' };
+            vm.phases = {
+                results: [
+                    { id: 2, name: 'Test Phase' }
+                ]
+            };
         });
 
         it('should send GET request and trigger download when fieldsToGet is undefined', function () {