CTP-2812 Hide submission log information

ucl-isd · Jan 5, 2024 · d43c1a1 · d43c1a1
1 parent 823d291
commit d43c1a1
Show file tree

Hide file tree

Showing 6 changed files with 28 additions and 14 deletions.
diff --git a/classes/output/renderer.php b/classes/output/renderer.php
@@ -71,7 +71,7 @@ public function __construct(moodle_page $page, $target) {
      * @return string Rendered HTML
      * @throws \moodle_exception
      */
-    public function render_button(string $id, string $name, string $disabled = '', $class = '') : string {
+    public function render_button(string $id, string $name, string $disabled = '', string $class = '') : string {
         return $this->output->render_from_template(
             'local_sitsgradepush/button',
             ['id' => $id, 'name' => $name, 'disabled' => $disabled, 'class' => $class]
@@ -95,10 +95,11 @@ public function render_link(string $id, string $name, string $url) : string {
      * Render the assessment push status table.
      *
      * @param \stdClass $mapping Assessment mapping
+     * @param int $courseid Course ID
      * @return string Rendered HTML
      * @throws \moodle_exception
      */
-    public function render_assessment_push_status_table(\stdClass $mapping) : string {
+    public function render_assessment_push_status_table(\stdClass $mapping, int $courseid) : string {
         $students = null;
         // Modify the timestamp format and add the label for the last push result.
         if (!empty($mapping->students)) {
@@ -129,13 +130,17 @@ public function render_assessment_push_status_table(\stdClass $mapping) : string
         // Check if there is any task info to display.
         $additionalinfo = $lasttasktext || $hastaskinprogress;
 
+        // Check if the user has the capability to see the submission log column.
+        $showsublogcolumn = has_capability('local/sitsgradepush:showsubmissionlogcolumn', \context_course::instance($courseid));
+
         // Render the table.
         return $this->output->render_from_template('local_sitsgradepush/assessmentgrades', [
             'mappingid' => $mappingid,
             'tabletitle' => $mapping->formattedname,
             'students' => $students,
             'lasttask' => $lasttasktext,
             'additionalinfo' => $additionalinfo,
+            'showsublogcolumn' => $showsublogcolumn,
         ]);
     }
 

diff --git a/db/access.php b/db/access.php
@@ -38,4 +38,9 @@
         'contextlevel' => CONTEXT_COURSE,
         'archetypes' => [],
     ],
+    'local/sitsgradepush:showsubmissionlogcolumn' => [
+        'captype' => 'read',
+        'contextlevel' => CONTEXT_COURSE,
+        'archetypes' => [],
+    ],
 ];
diff --git a/index.php b/index.php
@@ -129,12 +129,12 @@
     // Display grade push records for each mapping.
     foreach ($content['mappings'] as $mapping) {
         $mappingids[] = $mapping->id;
-        echo $renderer->render_assessment_push_status_table($mapping);
+        echo $renderer->render_assessment_push_status_table($mapping, $coursemodule->course);
     }
 
     // Display invalid students.
     if (!empty($content['invalidstudents']->students)) {
-        echo $renderer->render_assessment_push_status_table($content['invalidstudents']);
+        echo $renderer->render_assessment_push_status_table($content['invalidstudents'], $coursemodule->course);
     }
 } else {
     echo '<p class="alert alert-info">' . get_string('error:assessmentisnotmapped', 'local_sitsgradepush') . '</p>';

diff --git a/lang/en/local_sitsgradepush.php b/lang/en/local_sitsgradepush.php
@@ -141,6 +141,7 @@
 // Capability strings.
 $string['sitsgradepush:mapassessment'] = 'Map assessment component to Moodle activity';
 $string['sitsgradepush:pushgrade'] = 'Transfer Marks to SITS';
+$string['sitsgradepush:showsubmissionlogcolumn'] = 'Show Submission Log column';
 
 // Task strings.
 $string['task:pushtask:name'] = 'Schedule Transfer Task';

diff --git a/templates/assessmentgrades.mustache b/templates/assessmentgrades.mustache
@@ -32,7 +32,7 @@
     Example context (json):
     {
         "tabletitle":"PHAY0063-2022-T1-A7P-001 Coursework 4000 word written case studies (50%)",
-        "additionalinfo":true,
+        "additionalinfo":false,
         "students":[{
                 "firstname":"Test",
                 "lastname":"User",
@@ -62,12 +62,14 @@
         </tr>
     {{/additionalinfo}}
     <tr>
-        <th scope="col">{{#str}} index:student, local_sitsgradepush {{/str}}</th>
-        <th scope="col">{{#str}} index:porticonumber, local_sitsgradepush {{/str}}</th>
-        <th scope="col">{{#str}} index:grade, local_sitsgradepush {{/str}}</th>
-        <th scope="col">{{#str}} index:submissiondate, local_sitsgradepush {{/str}}</th>
-        <th scope="col">{{#str}} index:lastmarktransfer, local_sitsgradepush {{/str}}</th>
-        <th scope="col">{{#str}} index:lastsublogtransfer, local_sitsgradepush {{/str}}</th>
+        <th scope="col">Student</th>
+        <th scope="col">Portico number</th>
+        <th scope="col">Grade</th>
+        <th scope="col">Submission date</th>
+        <th scope="col">Last grade push</th>
+        {{#showsublogcolumn}}
+            <th scope="col">Last submission log push</th>
+        {{/showsublogcolumn}}
     </tr>
     </thead>
     <tbody>
@@ -78,8 +80,9 @@
             <td>{{marks}}</td>
             <td>{{handindatetime}}</td>
             <td>{{{lastgradepushresultlabel}}}{{lastgradepushtime}}</td>
-            <td>{{{lastsublogpushresultlabel}}}{{lastsublogpushtime}}
-            </td>
+            {{#showsublogcolumn}}
+                <td>{{{lastsublogpushresultlabel}}}{{lastsublogpushtime}}</td>
+            {{/showsublogcolumn}}
         </tr>
     {{/students}}
     </tbody>

diff --git a/version.php b/version.php
@@ -27,7 +27,7 @@
 
 $plugin->component = 'local_sitsgradepush';
 $plugin->release = '0.1.0';
-$plugin->version = 2023121900;
+$plugin->version = 2023122200;
 $plugin->requires = 2021051708;
 $plugin->maturity = MATURITY_ALPHA;
 $plugin->dependencies = [