Skip to content

Commit ee58b48

Browse files
committed
- fehlerhafte Gruppen-Befehle für Korrekturen wurden korrigiert
- Berechnungsfehler der Zulassungsseite beseitigt - zuordnung zwischen Korrekturen und Einsendungen berichtigt (für Seite des Einsendungsverlufs) close #177 close #178 close #179
1 parent 8aa106f commit ee58b48

File tree

3 files changed

+24
-23
lines changed

3 files changed

+24
-23
lines changed

DB/DBMarking/Sql/procedures/GetCourseUserGroupMarkings.sql

Lines changed: 11 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -42,21 +42,22 @@ select SQL_CACHE
4242
S.E_id as E_id2,
4343
S.ES_id as ES_id2
4444
from
45-
Marking M
46-
left join
47-
Submission S ON ('",sub,"'<>'nosubmission' and M.S_id = S.S_id)
45+
`Group` G
46+
join
47+
`Group` G2 ON (G.U_id_leader = '",userid,"'
48+
and G.U_id_member = G2.U_id_member
49+
and G.C_id = '",courseid,"'
50+
and G2.ES_id = G.ES_id)
51+
join
52+
Submission S ON (S.ES_id = G2.ES_id and G2.U_id_leader = S.U_id)
53+
join
54+
Marking M ON (S.S_id = M.S_id)
4855
left join
4956
SelectedSubmission SS ON (S.S_id = SS.S_id_selected)
50-
join
51-
`Group` G ON (G.ES_id = S.ES_id)
5257
left join
5358
File F ON (F.F_id = M.F_id_file)
5459
left join
55-
File F2 ON (F2.F_id = S.F_id_file)
56-
where
57-
G.C_id = '",courseid,"'
58-
and G.U_id_leader = '",userid,"'
59-
and G.U_id_member = S.U_id;");
60+
File F2 ON (F2.F_id = S.F_id_file)");
6061
PREPARE stmt1 FROM @s;
6162
EXECUTE stmt1;
6263
DEALLOCATE PREPARE stmt1;

DB/DBMarking/Sql/procedures/GetUserGroupMarkings.sql

Lines changed: 11 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -41,22 +41,23 @@ select SQL_CACHE
4141
S.S_hideFile as S_hideFile2,
4242
S.E_id as E_id2,
4343
S.ES_id as ES_id2
44-
from
45-
Marking M
44+
from
45+
`Group` G
4646
join
47-
Submission S ON ('",sub,"'<>'nosubmission' and M.S_id = S.S_id)
47+
`Group` G2 ON (G.U_id_leader = '",userid,"'
48+
and G.U_id_member = G2.U_id_member
49+
and G.ES_id = '",esid,"'
50+
and G2.ES_id = G.ES_id)
51+
join
52+
Submission S ON (S.ES_id = '",esid,"' and G2.U_id_leader = S.U_id)
53+
join
54+
Marking M ON (S.S_id = M.S_id)
4855
left join
4956
SelectedSubmission SS ON (S.S_id = SS.S_id_selected)
50-
join
51-
`Group` G ON (G.ES_id = S.ES_id)
5257
left join
5358
File F ON (F.F_id = M.F_id_file)
5459
left join
55-
File F2 ON (F2.F_id = S.F_id_file)
56-
where
57-
M.ES_id = '",esid,"'
58-
and G.U_id_leader = '",userid,"'
59-
and G.U_id_member = S.U_id;");
60+
File F2 ON (F2.F_id = S.F_id_file);");
6061
PREPARE stmt1 FROM @s;
6162
EXECUTE stmt1;
6263
DEALLOCATE PREPARE stmt1;

logic/LGetSite/LGetSite.php

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1025,15 +1025,14 @@ public function uploadHistory($userid, $courseid, $sheetid, $uploaduserid)
10251025
if (isset($submission['exerciseId'])){
10261026
if (!empty($answer2)){
10271027
foreach ($answer2 as $key => $marking){
1028-
if (isset($marking['submission']['id'])){
1028+
if (isset($marking['submission']['id']) && $marking['submission']['id'] == $submission['id']){
10291029
unset($marking['submission']);
10301030
$submission['marking'] = $marking;
10311031
unset($answer2[$key]);
10321032
break;
10331033
}
10341034
}
10351035
}
1036-
10371036
$submissions[$submission['exerciseId']][] = $submission;
10381037
}
10391038
}
@@ -1650,7 +1649,7 @@ function ($value, $exercise) {
16501649
if (!isset($studentMarkings[$studentID][$exerciseType]))
16511650
$studentMarkings[$studentID][$exerciseType] = 0;
16521651

1653-
$studentMarkings[$studentID][$exerciseType] += isset($marking['points']) ? $marking['points'] : 0;
1652+
$studentMarkings[$leaderID][$exerciseType] += isset($marking['points']) ? $marking['points'] : 0;
16541653

16551654
if (isset($allGroups[$sheetID][$leaderID])){
16561655
$group = $allGroups[$sheetID][$leaderID];

0 commit comments

Comments
 (0)