From 30daf68a691bddc34fed6957d840147b4b20246a Mon Sep 17 00:00:00 2001
From: Nick Freear <nfreear@yahoo.co.uk>
Date: Sun, 4 Mar 2018 22:26:12 +0000
Subject: [PATCH] Bug #8, add `tesla_consent::agree()` PHP class & method
 [iet:10310858]

 * [iet:10310859]
---
 CHANGELOG.md                    |  8 +++--
 classes/local/tesla_consent.php | 61 +++++++++++++++++++++++++++++++++
 composer.json                   |  1 +
 survey-end/index.php            |  5 ++-
 4 files changed, 71 insertions(+), 4 deletions(-)
 create mode 100644 classes/local/tesla_consent.php

diff --git a/CHANGELOG.md b/CHANGELOG.md
index e2d0686..7dbee36 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -7,9 +7,11 @@
 
  * _Date:  ~ 19 February ~ March 2018_
  * _Tag:  v1.5.0-beta_
- * Add functionality to reveal a TeSLA activity after the pre-activity survey is completed — `conditional_embedded_survey` PHP class.
+ * Add functionality to reveal a TeSLA activity after the pre-activity
+   survey is completed — `conditional_embedded_survey` PHP class (Bug #8)
    (That is, to "Complete" a survey embedded via `<IFRAME>` in a Moodle `mod/assign` conditional activity.)
- * Add functionality to "consent" to the TeSLA student-agreement in a survey embedded in an `<IFRAME>`. `tesla_consent` PHP class;
+ * Add functionality to "consent" to the TeSLA student-agreement in a
+   survey embedded in an `<IFRAME>`. `tesla_consent` PHP class (Bug #8)
  * Automated accessibility testing — [pa11y-ci][];
 
 ## Version 1.4.2 (Beta) - Pilot 3-A
@@ -26,7 +28,7 @@
  * _Date:  ~ 22 August - 6 December 2017_ (Deployed at start of TeSLA pilot 3-A)
  * _Tag:  v1.4.0-beta_
  * _Build:  "2017-12-05T13:25:56Z (MCD267768)"_
- * Javascript now built using Browserify,
+ * Javascript now built using Browserify (Bug #7)
  * Add 'poem server' — convert long texts (poems), into images of text, (Bug #6)
  * Add `fix_enrollment_callibrate_page()` Javascript,
  * Add long texts, with Chris E.
diff --git a/classes/local/tesla_consent.php b/classes/local/tesla_consent.php
new file mode 100644
index 0000000..66796bb
--- /dev/null
+++ b/classes/local/tesla_consent.php
@@ -0,0 +1,61 @@
+<?php
+/**
+ * Allow a student to "consent" to the TeSLA pilot study, via an embedded survey.
+ *
+ * @package auth_ouopenid
+ * @author  Nick Freear, 02-March-2018.
+ * @copyright © 2018 The Open University.
+ *
+ * @link  https://docs.moodle.org/dev/Data_manipulation_API#Inserting_Records
+ */
+
+namespace auth_ouopenid\local;
+
+class tesla_consent {
+
+  const DB_TABLE = 'local_tesla_agreement';
+  const AGREE_VERSION = '2.0';
+
+  // const CONFIG_KEY = 'auth_ouopenid_tesla_consent';
+
+  /** Has the user already consented?
+   * @param int $userid
+   */
+  public static function has_agreed( $userid = null ) {
+    global $DB, $USER; // Moodle global.
+
+    $userid = $userid ? $userid : $USER->id;
+
+    return $DB->count_records( self::DB_TABLE, [ 'userid' => $userid ]);
+  }
+
+  /** The user consents to the agreement.
+   * @param int $userid
+   * @param string $version  Agreement version.
+   * @return int  The inserted agreement ID.
+   */
+  public static function agree( $userid = null, $version = self::AGREE_VERSION ) {
+    global $DB, $USER; // Moodle global.
+
+    $userid = $userid ? $userid : $USER->id;
+
+    $lastinsertid = $DB->insert_record(self::DB_TABLE, (object) [
+      'userid' => $userid,
+      'timesubmitted' => time(), // UNIX_TIMESTAMP()
+      'version' => $version,
+      'accepted' => true,
+    ], false);
+
+    // self::_debug([ __METHOD__, $lastinsertid ]);
+    return $lastinsertid;
+  }
+
+  /* [1241] => stdClass Object (
+      [id] => 1241
+      [userid] => 5
+      [timesubmitted] => 1519734197
+      [version] => 2.0
+      [accepted] => 1
+      [timesubmitted_iso] => 2018-02-27T12:23:17+00:00
+  ) */
+}
diff --git a/composer.json b/composer.json
index d14b3ab..3b07b52 100644
--- a/composer.json
+++ b/composer.json
@@ -17,6 +17,7 @@
   "require": {
     "php": ">=5.5.9",
     "dariuszp/cli-progress-bar": "^1.0.2",
+    "goodby/csv": "^1.3",
     "vlucas/phpdotenv": "^2.4",
     "composer/installers": "^1.2"
   },
diff --git a/survey-end/index.php b/survey-end/index.php
index bd83bb8..d9a76da 100644
--- a/survey-end/index.php
+++ b/survey-end/index.php
@@ -10,8 +10,9 @@
 // For 'print_string()' language support!
 require_once __DIR__ . '/../../../config.php';
 require_once __DIR__ . '/../classes/local/conditional_embedded_survey.php';
+require_once __DIR__ . '/../classes/local/tesla_consent.php';
 
-use auth_ouopenid\local\conditional_embedded_survey;
+use auth_ouopenid\local\tesla_consent;
 
 define( 'OUOP_STRING', 'auth_ouopenid' );
 
@@ -54,6 +55,7 @@ public static function complete_conditional() {
 Ou_Open_Id_Survey_End::checkMaintenanceMode();
 
 $conditional_completed = Ou_Open_Id_Survey_End::complete_conditional();
+$agreement_id = tesla_consent::agree();
 
 header('Content-Language: en');
 header('X-Frame-Options: sameorigin');
@@ -122,6 +124,7 @@ public static function complete_conditional() {
   echo json_encode([
     'course_code' => Ou_Open_Id_Survey_End::getReturnCode(),
     'cond_completed' => $conditional_completed,
+    'agreement_id' => $agreement_id,
     'redirects' => $CFG->auth_ouopenid_redirects,
     'hash' => '#section-3',
     'timeout' => 3000,