fix excel-import redesign test grad table

bkd-mba-fbi · Jul 30, 2024 · daf81c9 · daf81c9
1 parent fa6d2e2
commit daf81c9
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 18 deletions.
diff --git a/public/apps/MasterExcel/X.js b/public/apps/MasterExcel/X.js
@@ -269,13 +269,13 @@ var X = {
             var tests = X.collectTestNames();
             index = tests.indexOf(aTest);
             if (index == -1 && tests[aTest]) {
-                index = aTest;
+                index = aTest-1; // Es wurde eine leer Spalte eingefugt deswegen darf diese nicht gezählt werden
             }
             if (index == -1) {
                 // ungültiger Wert für aTest
                 return;
             }
-            var kursname = tests[index];
+            var kursname = tests[index+1]; // Die leer Spalte am anfang muss übersprungen werden.
             lines = X.strings[X.lang].views[aView].default_lines.join("\n").replace("%s", kursname) +
                 "\n" + X.collectNames(aView, index).join("\n") + "\n";
         }
@@ -528,8 +528,8 @@ var X = {
                 // * ein "Ungültiger Wert" Fehler angezeigt, wenn die eingegebene Note in der
                 //   Auswahlliste nicht vorkam
                 // * der Wert übertragen und kein Fehler angezeigt
-                $("bkd-test-edit-grades table tbody tr:not(:last-child)").each(function() {
-                    var name = X.trimName($("td.name span:first-child", this).text());
+                $("bkd-test-edit-grades div table tbody tr:not(:last-child)").each(function() {
+                    var name = X.trimName($("td.student-name a div:first-child", this).text());
                     var error = [null, null];
 
                     var cell = $("td.name, td:not(.sticky)", this).get(aTest);
@@ -598,16 +598,15 @@ var X = {
     collectNames: function(aView, aIncData) {
         var values = [];
 
-        var nameCell = aView == 4 ? "bkd-test-edit-grades tbody td.name span:first-child" :
+        var nameCell = aView == 4 ? "bkd-test-edit-grades div table tbody td.student-name a div:first-child" :
             aView >= 2 ? "td.validationColumn + td" :
             aView == 0 ? ".tablelabel + .content1" :
             "td.tablelabel:first-child, table.WebPart-Adaptive td:first-child";
-
         $(nameCell).each(function() {
             var name = X.trimName($(this).text());
             if (name) {
                 var data = "";
-                if (aIncData) {
+                if (aIncData >= 0) {
                     switch (aView) {
                         case 0:
                             var select = $(this).parent().find("select");
@@ -642,7 +641,7 @@ var X = {
                             break;
                         case 4:
                             // Noten/Punkte des gewünschten Tests
-                            var cell = $(this).parents("tr").find("td.name, td:not(.sticky)").get(aIncData);
+                            var cell = $(this).parents("tr").find("td.test-grade, td:not(.sticky)").get(aIncData);
                             var number = $("input[type=number]", cell);
                             if (number.length == 1) {
                                 data = number.val();
@@ -703,8 +702,8 @@ var X = {
      */
     getFirstInput: function(aView, aTest) {
         if (aView == 4 && aTest) {
-            var rows = $("bkd-test-edit-grades table tbody tr");
-            var cell = $("td.name, td:not(.sticky)", rows.get(0)).get(aTest);
+            var rows = $("bkd-test-edit-grades div table tbody tr");
+            var cell = $("td.test-grade, td:not(.sticky)", rows.get(0)).get(aTest);
             return $("input[type=number], select", cell).get(0);
         }
 

diff --git a/public/apps/MasterExcel/main.js b/public/apps/MasterExcel/main.js
@@ -74,18 +74,19 @@ function insertButtonsTest() {
 
   var tests = X.collectTestNames();
   var dropdownItems = '';
-
-  var i = 1
-  tests.forEach(test => {
+  var atworkTests = 0;
+  for (let index = 0; index < tests.length; index++) {
+    const test = tests[index];
+
     if (dropdownItems.indexOf(test) === -1 && tests.indexOf(test) > 0 && test.length > 0) {
-      i++
-      dropdownItems = dropdownItems + '<a onclick="X.showOverlay(4,'+ i +')";>' + test + '</a>';
+      dropdownItems = dropdownItems + '<a onclick="X.showOverlay(4,'+ index +')";>' + test + '</a>';
+      atworkTests++;
+    }
   }
-  });
 
   // Falls sie vorhanden sind neu einfügen. Damit kann der useCase gelöst werden, falls ein Test publiziert wird das dieser nicht mehr dargestellt wird und umgekehrt.
-  var testslinks = $('#overlay-toggle-embedded-test div').first().children().length+1;
-  if (testslinks != i) {
+  var testslinks = $('#overlay-toggle-embedded-test div').first().children().length;
+  if (testslinks != atworkTests ) {
     $('#excel-import').empty();   
   } else {
     return;