Merge pull request #229 from abhihari010/abhi/dumpfix

Author: awhicks

app/assets/javascripts/odsa_tools.js

@@ -5,6 +5,86 @@ $(function () {
     storeName: storeName,
   });
 
+  /* ---------------- CSV Download for Module Scores Table ---------------- */
+
+  // Use both delegated and direct handlers for maximum compatibility
+  $(document).on("click", "#btn-module-csv", function (e) {
+    e.preventDefault();
+    e.stopPropagation();
+    downloadModuleCSV();
+  });
+
+  // Also set up a direct handler once the button appears
+  function setupDirectHandler() {
+    const btn = document.getElementById("btn-module-csv");
+    if (btn) {
+      btn.addEventListener("click", function (e) {
+        e.preventDefault();
+        e.stopPropagation();
+        downloadModuleCSV();
+      });
+    } else {
+      setTimeout(setupDirectHandler, 1000);
+    }
+  }
+  setTimeout(setupDirectHandler, 100);
+
+  function downloadModuleCSV() {
+    const table = $("#module-scores-table");
+    const headers = [];
+    const rows = [];
+    
+    // Get module name from the selectize dropdown
+    const selectize = $("#select-for-modules")[0]?.selectize;
+    const moduleName = selectize ? selectize.getItem(selectize.getValue())?.text() : "module";
+    const sanitizedModuleName = moduleName.replace(/[^a-z0-9]/gi, '_').replace(/_+/g, '_');
+
+    // Get headers
+    $("#mst-header-row th").each(function () {
+      headers.push(
+        $(this)
+          .text()
+          .trim()
+          .replace(/\s*\(\?\)\s*/g, ""),
+      );
+    });
+
+    console.log("Headers:", headers);
+
+    // Get data rows
+    $("#mst-body tr").each(function () {
+      const row = [];
+      $(this)
+        .find("td")
+        .each(function () {
+          const text = $(this).text().trim();
+          // Escape commas and quotes in CSV
+          const escaped =
+            text.includes(",") || text.includes('"')
+              ? `"${text.replace(/"/g, '""')}"`
+              : text;
+          row.push(escaped);
+        });
+      rows.push(row.join(","));
+    });
+
+    // Create CSV content
+    const csvContent = [headers.join(","), ...rows].join("\n");
+
+    // Create download link
+    const blob = new Blob([csvContent], { type: "text/csv;charset=utf-8;" });
+    const link = document.createElement("a");
+    const url = URL.createObjectURL(blob);
+
+    link.setAttribute("href", url);
+    link.setAttribute("download", `MS-Overview-${sanitizedModuleName}.csv`);
+    link.style.visibility = "hidden";
+
+    document.body.appendChild(link);
+    link.click();
+    document.body.removeChild(link);
+    URL.revokeObjectURL(url);
+  }
   // Returns weeks start and end dates
   function getWeeksDates(start, end) {
     var sDate;

app/controllers/odsa_exercise_attempts_controller.rb

@@ -589,21 +589,28 @@ def export_all_attempts_csv
       return
     end
 
-    course_offering =
-      CourseOffering.find(params[:course_offering_id]) if params[:course_offering_id].present?
+    course_offering = CourseOffering.find(params[:course_offering_id]) if params[:course_offering_id].present?
 
-    unless current_user.global_role.is_admin? ||
-          (course_offering && course_offering.is_instructor?(current_user))
+    unless current_user.global_role.is_admin? || (course_offering && course_offering.is_instructor?(current_user))
       render json: { message: "You are not authorized to view this data." },
             status: :forbidden and return
     end
 
+    # Get the inst_book_id from the course offering
+    inst_book_id =
+      if course_offering&.odsa_books&.first
+        course_offering.odsa_books.first.id
+      else
+        nil
+      end
+
     filename = "opendsa_attempts_#{Time.zone.now.strftime('%Y%m%d_%H%M%S')}.csv"
+    conn = ActiveRecord::Base.connection
 
     sql = <<~SQL
       SELECT
         a.user_id AS "User ID",
-        im.name   AS "Module Name",      -- from inst_modules
+        im.name   AS "Module Name",
         ie.name   AS "Exercise Name",
         a.question_name AS "Question name",
         a.correct       AS "Correct",
@@ -615,13 +622,15 @@ def export_all_attempts_csv
       JOIN inst_exercises ie               ON ie.id  = ibse.inst_exercise_id
       JOIN inst_sections s                 ON s.id   = ibse.inst_section_id
       JOIN inst_chapter_modules icm        ON icm.id = s.inst_chapter_module_id
-      JOIN inst_modules im                 ON im.id  = icm.inst_module_id   -- <== if your FK is module_id, change to icm.module_id
+      JOIN inst_chapters ich               ON ich.id = icm.inst_chapter_id
+      JOIN inst_modules im                 ON im.id  = icm.inst_module_id
       JOIN users u                         ON u.id   = a.user_id
-      WHERE u.email != #{ActiveRecord::Base.connection.quote(OpenDSA::STUDENT_VIEW_EMAIL)}
+      WHERE u.email != #{conn.quote(OpenDSA::STUDENT_VIEW_EMAIL)}
+        #{inst_book_id ? "AND ich.inst_book_id = #{conn.quote(inst_book_id)}" : ""}
       ORDER BY a.user_id, im.name, a.time_done
     SQL
 
-    rows = ActiveRecord::Base.connection.exec_query(sql)
+    rows = conn.exec_query(sql)
 
     header = [
       "User ID",