";
try {
@@ -172,10 +194,10 @@ Klc5I28bGbvxIV5pnL6ZSjHEDp2WreM8HB0XFJwU+Q==
$s .= $json . "\n----\n";
$decoded = \json_decode($json, false);
$s .= "Read as object:\n----\n";
- $s .= print_r($decoded, true) ."\n----\n";
- $status = self::premiumStatus();
+ $s .= \json_encode($decoded, \JSON_PRETTY_PRINT) ."\n----\n";
+ $status = self::premiumstatus();
$s .= "Parsed premium status block:\n----\n";
- $s .= print_r($status, true) ."\n----\n";
+ $s .= \json_encode($status, \JSON_PRETTY_PRINT) ."\n----\n";
$s .= "Message: " . self::statusdescription() . "\n";
} else {
$s .= "Premium key empty";
@@ -188,14 +210,15 @@ Klc5I28bGbvxIV5pnL6ZSjHEDp2WreM8HB0XFJwU+Q==
}
}
- $s.="";
+ $s .= "";
return $s;
}
+
/**
* Determine, cache and retrieve premium status
* @return object
*/
- protected static function premiumStatus() {
+ protected static function premiumstatus() {
if (!isset(self::$cachedpremiumstatus)) {
// Initialize default object.
@@ -217,9 +240,9 @@ Klc5I28bGbvxIV5pnL6ZSjHEDp2WreM8HB0XFJwU+Q==
if (is_object($decoded)) {
- // Copy basic data/
+ // Copy basic data/.
$keys = ["name", "website", "expires", "issued"];
- foreach ( $keys as $k) {
+ foreach ($keys as $k) {
if (isset($decoded->$k)) {
$o->$k = $decoded->$k;
}
@@ -229,24 +252,26 @@ Klc5I28bGbvxIV5pnL6ZSjHEDp2WreM8HB0XFJwU+Q==
$o->intents = explode(", ", $decoded->intent);
}
- // Convert dates to DateTime for
+ // Convert dates to DateTime for.
$now = new \DateTime();
$issuedate = new \DateTime($o->issued);
- $expirydate = new \DateTime(); // Default to now
if ($o->expires == 'never') {
- // If expiry date == never
+ // If expiry date == never.
+ $expirydate = new \DateTime(); // Default to now and add a year.
$expirydate->add(new \DateInterval("P1Y"));
} else {
try {
$expirydate = new \DateTime($o->expires);
- } catch (\Exception $x) {}
+ } catch (\Exception $x) {
+ $expirydate = new \DateTime(); // Default to now.
+ }
}
- if ( \in_array('treestudyplan', $o->intents)
+ if (\in_array('treestudyplan', $o->intents)
&& !empty($o->issued)
&& self::website_match($o->website)
) {
- if ($expirydate > $now ) {
+ if ($expirydate > $now) {
$o->enabled = true;
$o->expired = false;
} else {
@@ -263,7 +288,7 @@ Klc5I28bGbvxIV5pnL6ZSjHEDp2WreM8HB0XFJwU+Q==
}
}
}
- } catch (\ValueError $x) {
+ } catch (\ParseError $x) {
$o->status = \get_string("premium:invalidactivationcontent", "local_treestudyplan");
}
self::$cachedpremiumstatus = $o;
@@ -281,7 +306,7 @@ Klc5I28bGbvxIV5pnL6ZSjHEDp2WreM8HB0XFJwU+Q==
// Add double slashes to key and site if no scheme is set.
// Basically: if no double slashes present before any dots, slashes or @s.
- if (!\preg_match_all('#^[^./@]*?//#', $key )) {
+ if (!\preg_match_all('#^[^./@]*?//#', $key)) {
$key = "//".$key;
}
if (!\preg_match_all('#^[^./@]*?//#', $site)) {
@@ -291,13 +316,13 @@ Klc5I28bGbvxIV5pnL6ZSjHEDp2WreM8HB0XFJwU+Q==
$keyurl = (object)\parse_url($key);
$siteurl = (object)\parse_url($site);
- // No match if host is empty on key or site
+ // No match if host is empty on key or site.
if (empty($keyurl->host) || empty($siteurl->host)) {
return false;
}
if ($keyurl->host == "*") {
- // * matches all
+ // Value * matches all.
return true;
}
@@ -306,37 +331,41 @@ Klc5I28bGbvxIV5pnL6ZSjHEDp2WreM8HB0XFJwU+Q==
$siteparts = \array_reverse(\explode(".", $siteurl->host));
// Trim starting www from both parts, since site.domain and www.site.domain should be treated as the same.
- if (($x = \array_pop($keyparts)) != "www") {\array_push($keyparts, $x);}
- if (($x = \array_pop($siteparts)) != "www") {\array_push($siteparts, $x);}
+ if (($x = \array_pop($keyparts)) != "www") {
+ \array_push($keyparts, $x);
+ }
+ if (($x = \array_pop($siteparts)) != "www") {
+ \array_push($siteparts, $x);
+ }
for ($i = 0; $i < count($keyparts); $i++) {
- // No match if the site does not have a part, but the key does. Unless the key part is *
- if (!isset($siteparts[$i]) ) {
+ // No match if the site does not have a part, but the key does. Unless the key part is .
+ if (!isset($siteparts[$i])) {
if ($keyparts[$i] != "*") {
return false;
} else {
- $i++; //increment $i by one before break, to make sure the comparison following this loop holds.
+ $i++; // Increment $i by one before break, to make sure the comparison following this loop holds.
break; // Stop comparison. Host part matches.
}
}
// Now do a proper case insensitive check for matching.
// Uses fnmatch to easily handle shell type wildcards.
- if ( ! \fnmatch($keyparts[$i], $siteparts[$i], \FNM_CASEFOLD)) {
+ if (! \fnmatch($keyparts[$i], $siteparts[$i], \FNM_CASEFOLD)) {
return false;
}
}
- // Fail if the site has a deeper subdomain than the key, unless the deepest key subdomain is *
- if ($keyparts[$i-1] != '*' && count($siteparts) > ($i)) {
+ // Fail if the site has a deeper subdomain than the key, unless the deepest key subdomain is .
+ if ($keyparts[$i - 1] != '*' && count($siteparts) > ($i)) {
return false;
}
// If we made it here then the host part matches. Now check the path.
- // If path is /*, matches all subpaths including /
+ // If path is /*, matches all subpaths including /.
$keypath = empty($keyurl->path) ? "/" : $keyurl->path;
$sitepath = empty($siteurl->path) ? "/" : $siteurl->path;
- // Trim trailing / from both paths before comparison
+ // Trim trailing / from both paths before comparison.
if (\strlen($sitepath) > 1) {
$sitepath = \rtrim($sitepath, "/");
}
@@ -365,7 +394,9 @@ Klc5I28bGbvxIV5pnL6ZSjHEDp2WreM8HB0XFJwU+Q==
"name" => new \external_value(PARAM_TEXT, 'premium registration name'),
"expires" => new \external_value(PARAM_TEXT, 'premium registration expiry date'),
"expired" => new \external_value(PARAM_BOOL, 'premium status expired'),
- "intents" => new \external_multiple_structure(new \external_value(PARAM_TEXT), 'additional intents included in the license '),
+ "intents" => new \external_multiple_structure(
+ new \external_value(PARAM_TEXT), 'additional intents included in the license'
+ ),
]);
}
@@ -375,7 +406,7 @@ Klc5I28bGbvxIV5pnL6ZSjHEDp2WreM8HB0XFJwU+Q==
*/
public static function get_premiumstatus() {
if (self::$premiumsupported) {
- $status = self::premiumStatus();
+ $status = self::premiumstatus();
$keys = [
"enabled",
"website",
@@ -386,7 +417,7 @@ Klc5I28bGbvxIV5pnL6ZSjHEDp2WreM8HB0XFJwU+Q==
];
$result = [];
- foreach ( $keys as $param) {
+ foreach ($keys as $param) {
$result[$param] = $status->$param;
}
return $result;
@@ -407,7 +438,7 @@ Klc5I28bGbvxIV5pnL6ZSjHEDp2WreM8HB0XFJwU+Q==
* @return string HTML description of premium status.
*/
public static function statusdescription() {
- $status = self::premiumStatus();
+ $status = self::premiumstatus();
$msg = new \stdClass;
$msg->name = $status->name;
@@ -439,4 +470,4 @@ Klc5I28bGbvxIV5pnL6ZSjHEDp2WreM8HB0XFJwU+Q==
}
}
-}
\ No newline at end of file
+}
diff --git a/classes/reportservice.php b/classes/reportservice.php
index d4d73a7..786b2fb 100644
--- a/classes/reportservice.php
+++ b/classes/reportservice.php
@@ -40,6 +40,9 @@ class reportservice extends \external_api {
*/
const CAP_VIEW = "local/treestudyplan:viewuserreports";
+ /**
+ * Parameter description for webservice function
+ */
public static function get_report_structure_parameters(): \external_function_parameters {
return new \external_function_parameters([
"pageid" => new \external_value(PARAM_INT, 'id of studyplan page'),
@@ -83,7 +86,7 @@ class reportservice extends \external_api {
$firstperiod = 1;
}
if (empty($lastperiod)) {
- // Index for periods starts at 1, so the period count is same as length
+ // Index for periods starts at 1, so the period count is same as length.
$lastperiod = $page->periods();
}
@@ -95,9 +98,9 @@ class reportservice extends \external_api {
"periods" => [],
];
- // Find all relevant parts in this order
+ // Find all relevant parts in this order.
- for($i = $firstperiod; $i <= $lastperiod; $i++) {
+ for ($i = $firstperiod; $i <= $lastperiod; $i++) {
$period = period::find($page, $i);
$pmodel = [
'period' => $period->model(),
@@ -127,13 +130,15 @@ class reportservice extends \external_api {
return $model;
}
+ /**
+ * Parameter description for webservice function
+ */
public static function get_report_data_parameters(): \external_function_parameters {
return new \external_function_parameters([
"pageid" => new \external_value(PARAM_INT, 'id of studyplan page'),
"userid" => new \external_value(PARAM_INT, 'id of user'),
"firstperiod" => new \external_value(PARAM_INT, 'first period to include in report', VALUE_DEFAULT),
"lastperiod" => new \external_value(PARAM_INT, 'last period to include in report', VALUE_DEFAULT),
-
]);
}
@@ -144,6 +149,13 @@ class reportservice extends \external_api {
return new \external_multiple_structure(studyitem::user_structure(), "Information per studyitem");
}
+ /**
+ * Webservice function get_report data
+ * @param int $pageid
+ * @param int $userid
+ * @param int|null $firsteperiod
+ * @param int|null $lastperiod
+ */
public static function get_report_data($pageid, $userid, $firstperiod=null, $lastperiod=null) {
$page = studyplanpage::find_by_id($pageid);
webservicehelper::require_capabilities(self::CAP_VIEW, $page->studyplan()->context());
@@ -156,15 +168,12 @@ class reportservice extends \external_api {
$firstperiod = 1;
}
if (empty($lastperiod)) {
- // Index for periods starts at 1, so the period count is same as length
+ // Index for periods starts at 1, so the period count is same as length.
$lastperiod = $page->periods();
}
$model = [];
-
- //TODO: Build report structure
-
- for($i = $firstperiod; $i <= $lastperiod; $i++) {
+ for ($i = $firstperiod; $i <= $lastperiod; $i++) {
$lines = studyline::find_page_children($page);
foreach ($lines as $line) {
$items = studyitem::search_studyline_children($line, $i, studyitem::COURSE);
@@ -175,11 +184,12 @@ class reportservice extends \external_api {
}
}
}
-
return $model;
-
}
+ /**
+ * Parameter description for webservice function
+ */
public static function get_report_details_parameters(): \external_function_parameters {
return new \external_function_parameters([
"itemid" => new \external_value(PARAM_INT, 'id of studyitem'),
diff --git a/classes/studentstudyplanservice.php b/classes/studentstudyplanservice.php
index 054bb4d..c84fa5f 100644
--- a/classes/studentstudyplanservice.php
+++ b/classes/studentstudyplanservice.php
@@ -127,7 +127,7 @@ class studentstudyplanservice extends \external_api {
if ($studyplan->exist_for_user($userid)) {
- $model = $studyplan->user_model($userid);
+ $model = $studyplan->user_model($userid);
return $model;
} else {
return null;
@@ -222,7 +222,7 @@ class studentstudyplanservice extends \external_api {
return [];
}
- // Do not validate the context in this instance to avoid requiring logins when this is not wanted
+ // Do not validate the context in this instance to avoid requiring logins when this is not wanted.
$userid = $invite->user_id;
@@ -279,7 +279,7 @@ class studentstudyplanservice extends \external_api {
return [];
}
- // Do not validate the context in this instance to avoid requiring logins when this is not wanted
+ // Do not validate the context in this instance to avoid requiring logins when this is not wanted.
$studyplan = studyplan::find_by_id($studyplanid);
$userid = $invite->user_id;
@@ -341,7 +341,7 @@ class studentstudyplanservice extends \external_api {
return [];
}
- // Do not validate the context in this instance to avoid requiring logins when this is not wanted
+ // Do not validate the context in this instance to avoid requiring logins when this is not wanted.
$page = studyplanpage::find_by_id($pageid);
$studyplan = $page->studyplan();
@@ -430,8 +430,8 @@ class studentstudyplanservice extends \external_api {
$studyplan = studyplan::find_by_id($studyplanid);
if ($studyplan->exist_for_user($userid) && !$studyplan->suspended()) {
- $model = $studyplan->user_model($userid);
- return $model;
+ $model = $studyplan->user_model($userid);
+ return $model;
} else {
throw new \moodle_exception("You do not have access to this studyplan");
}
@@ -556,8 +556,8 @@ class studentstudyplanservice extends \external_api {
\external_api::validate_context(\context_system::instance());
- // Return empty list if coach role not enabled
- if ( ! (premium::enabled() && \get_config("local_treestudyplan", "enablecoach")) ) {
+ // Return empty list if coach role not enabled.
+ if (! (premium::enabled() && \get_config("local_treestudyplan", "enablecoach"))) {
return [];
}
@@ -570,7 +570,9 @@ class studentstudyplanservice extends \external_api {
if (has_capability(self::CAP_COACH, $studyplan->context(), $USER)) {
$list[] = $studyplan->simple_model_coach();
}
- } catch(\Exception $x) {}
+ } catch (\Exception $x) {
+ $studyplan = null;
+ }
}
return $list;
}
@@ -694,7 +696,7 @@ class studentstudyplanservice extends \external_api {
$o->enrol($userid);
}
- // Trigger immediate cohort synchronization for this line only
+ // Trigger immediate cohort synchronization for this line only.
autocohortsync::syncline($o);
return $o->enrol_info_model($userid);
diff --git a/classes/studyitem.php b/classes/studyitem.php
index 985eea5..629d52b 100644
--- a/classes/studyitem.php
+++ b/classes/studyitem.php
@@ -422,7 +422,7 @@ class studyitem {
foreach ($resequence as $sq) {
// Only change line_id if new line is within the same studyplan page.
- if ( self::find_by_id($sq['id'])->studyline()->page()->id() ==
+ if (self::find_by_id($sq['id'])->studyline()->page()->id() ==
studyline::find_by_id($sq['line_id'])->page()->id() ) {
$DB->update_record(self::TABLE, [
'id' => $sq['id'],
@@ -646,7 +646,7 @@ class studyitem {
break;
}
}
- return ($notexpired) ? completion : :COMPLETED : completion::INCOMPLETE;
+ return ($notexpired) ? completion::COMPLETED : completion::INCOMPLETE;
} else {
return completion::COMPLETED;
}
diff --git a/classes/studyitemconnection.php b/classes/studyitemconnection.php
index 1c7df50..4d13f9d 100644
--- a/classes/studyitemconnection.php
+++ b/classes/studyitemconnection.php
@@ -96,7 +96,7 @@ class studyitemconnection {
/**
* Get target studyitem id for this connection
*/
- public function to_id() :int {
+ public function to_id(): int {
return $this->r->to_id;
}
@@ -167,9 +167,9 @@ class studyitemconnection {
'to_id' => $toid,
]);
- return success::success(['msg'=>'Items Disconnected']);
+ return success::success(['msg' => 'Items Disconnected']);
} else {
- return success::success(['msg'=>'Connection does not exist']);
+ return success::success(['msg' => 'Connection does not exist']);
}
}
diff --git a/classes/studyline.php b/classes/studyline.php
index f504666..9db0896 100644
--- a/classes/studyline.php
+++ b/classes/studyline.php
@@ -179,14 +179,14 @@ class studyline {
* Whether this line is enrollable by the student
*/
public function self_enrollable(): bool {
- return ($this->r->enrollable == self::ENROLLABLE_SELF || $this->r->enrollable == self::ENROLLABLE_SELF_ROLE);
+ return ($this->r->enrollable == self::ENROLLABLE_SELF || $this->r->enrollable == self::ENROLLABLE_SELF_ROLE);
}
/**
* Whether this line is enrollable by a role
*/
public function role_enrollable(): bool {
- return ($this->r->enrollable == self::ENROLLABLE_ROLE || $this->r->enrollable == self::ENROLLABLE_SELF_ROLE);
+ return ($this->r->enrollable == self::ENROLLABLE_ROLE || $this->r->enrollable == self::ENROLLABLE_SELF_ROLE);
}
/**
@@ -198,13 +198,13 @@ class studyline {
$plan = $this->studyplan();
if (!empty($userid)) {
- if ( $plan->has_linked_user(intval($userid))) {
- if ( $this->self_enrollable() && $userid == $USER->id ) {
+ if ($plan->has_linked_user(intval($userid))) {
+ if ($this->self_enrollable() && $userid == $USER->id) {
return true;
}
- if ( $this->role_enrollable()) {
+ if ($this->role_enrollable()) {
$context = $plan->context();
- foreach ( $this->enrol_roleids() as $rid) {
+ foreach ($this->enrol_roleids() as $rid) {
if (\user_has_role_assignment($USER->id, $rid, $context->id)) {
return true;
}
@@ -212,9 +212,9 @@ class studyline {
}
}
} else {
- if ( $this->role_enrollable()) {
+ if ($this->role_enrollable()) {
$context = $plan->context();
- foreach ( $this->enrol_roleids() as $rid) {
+ foreach ($this->enrol_roleids() as $rid) {
if (\user_has_role_assignment($USER->id, $rid, $context->id)) {
return true;
}
@@ -239,13 +239,15 @@ class studyline {
global $DB;
$list = [];
$roles = explode(", ", $this->r->enrolrole);
- foreach($roles as $r) {
+ foreach ($roles as $r) {
$roleid = intval($r);
if ($roleid > 0) {
try {
$role = $DB->get_record('role', ["id" => $roleid], "*", MUST_EXIST);
$list[] = $role;
- } catch(\Exception $x) {}
+ } catch (\Exception $x) {
+ $role = null;
+ }
}
}
@@ -259,7 +261,7 @@ class studyline {
global $DB;
$list = [];
$roles = explode(", ", $this->r->enrolrole);
- foreach($roles as $r) {
+ foreach ($roles as $r) {
$roleid = intval($r);
if ($roleid > 0) {
$list[] = $roleid;
@@ -274,7 +276,7 @@ class studyline {
*/
public function enrol_roles_model(): array {
$list = [];
- foreach($this->enrol_roles() as $role) {
+ foreach ($this->enrol_roles() as $role) {
$name = role_get_name($role, $this->context()); // Get localized role name.
$list[] = [
"id" => $role->id,
@@ -304,7 +306,7 @@ class studyline {
self::SLOTSET_FILTER => new \external_multiple_structure(
studyitem::editor_structure(), 'filter items'),
])
- )
+ ),
], "Study line editor structure", $value);
}
@@ -552,7 +554,7 @@ class studyline {
self::SLOTSET_FILTER => new \external_multiple_structure(
studyitem::user_structure(), 'filter items'),
])
- )
+ ),
], 'Studyline with user info', $value);
}
@@ -571,7 +573,7 @@ class studyline {
];
if (!empty($userid)) {
- $r = $DB->get_record('local_treestudyplan_lineuser', [
+ $r = $DB->get_record('local_treestudyplan_lineuser', [
'line_id' => $this->id(),
'user_id' => $userid,
]);
@@ -606,15 +608,16 @@ class studyline {
return $model;
}
- /** Get the users enrolled in this studyline if relevant
- * @return array of Userids linked
+ /**
+ * Get the users enrolled in this studyline if relevant
+ * @return array of Userids linked
*/
public function get_enrolled_userids() {
$userids = $this->studyplan()->find_linked_userids();
if ($this->enrollable()) {
$list = [];
- foreach($userids as $uid) {
- if ( $this->isenrolled($uid)) {
+ foreach ($userids as $uid) {
+ if ($this->isenrolled($uid)) {
$list[] = $uid;
}
}
@@ -632,7 +635,7 @@ class studyline {
public function isenrolled($userid) {
global $DB;
if ($this->r->enrollable == self::ENROLLABLE_NONE) {
- return true; // If student cannot enrol, the student always is enrolled
+ return true; // If student cannot enrol, the student always is enrolled.
} else {
$r = $DB->get_record('local_treestudyplan_lineuser', [
'line_id' => $this->id(),
@@ -721,7 +724,7 @@ class studyline {
// Registration already exists, check if enrolled is true and update accordingly.
if (boolval($r->enrolled)) {
$r->enrolled = 0;
- $r->timeenrolled = time(); // Regi
+ $r->timeenrolled = time(); // Regi.
$r->enrolledby = $USER->id;
$DB->update_record("local_treestudyplan_lineuser", $r);
diff --git a/classes/studyplan.php b/classes/studyplan.php
index bb6e881..925360d 100644
--- a/classes/studyplan.php
+++ b/classes/studyplan.php
@@ -146,7 +146,7 @@ class studyplan {
*/
public function startdate() {
$date = null;
- foreach($this->pages() as $p) {
+ foreach ($this->pages() as $p) {
if (!isset($date) || $p->startdate() < $date) {
$date = $p->startdate();
}
@@ -154,13 +154,17 @@ class studyplan {
return $date;
}
+ /**
+ * Determine studyplan icon
+ * @return string Url of icon
+ */
private function icon() {
global $CFG;
$fs = \get_file_storage();
// Returns an array of `stored_file` instances.
$files = $fs->get_area_files(\context_system::instance()->id, 'local_treestudyplan', 'icon', $this->id);
- if (count($files) > 0 ) {
+ if (count($files) > 0) {
$file = array_shift($files);
if ($file->get_filename() == ".") {
// Get next file if the first is the directory itself.
@@ -183,8 +187,14 @@ class studyplan {
// Fall back to the standard (ugly) default image.
$url = new \moodle_url($CFG->wwwroot . "/local/treestudyplan/pix/default_icon.png");
} else {
- $url = \moodle_url::make_pluginfile_url(\context_system::instance()->id, 'local_treestudyplan', 'defaulticon', 0, "/",
- $defaulticon);
+ $url = \moodle_url::make_pluginfile_url(
+ \context_system::instance()->id,
+ 'local_treestudyplan',
+ 'defaulticon',
+ 0,
+ "/",
+ $defaulticon
+ );
}
}
@@ -201,9 +211,9 @@ class studyplan {
// The pluginfile URL which will serve the request.
'pluginfile.php',
- // The combination of contextid / component / filearea / itemid
- // form the virtual bucket that file are stored in.
- \context_system::instance()->id, // System instance is always used for this
+ // The combination of contextid / component / filearea / itemid.
+ // Form the virtual bucket that file are stored in.
+ \context_system::instance()->id, // System instance is always used for this.
'local_treestudyplan',
'studyplan',
$this->id
@@ -216,7 +226,7 @@ class studyplan {
* @return studyplanpage[]
*/
public function pages($refresh=false): array {
- if ( ((bool)$refresh) || empty($this->pagecache)) {
+ if (((bool)$refresh) || empty($this->pagecache)) {
$this->pagecache = studyplanpage::find_studyplan_children($this);
}
return $this->pagecache;
@@ -266,7 +276,11 @@ class studyplan {
"aggregation_info" => aggregator::basic_structure(),
"pages" => new \external_multiple_structure(studyplanpage::simple_structure(), 'pages'),
"progress" => new \external_value(PARAM_FLOAT, "fraction of completed modules", VALUE_OPTIONAL),
- "amteaching" => new \external_value(PARAM_BOOL, "Current user is teaching one or more courses in this studyplan", VALUE_OPTIONAL),
+ "amteaching" => new \external_value(
+ PARAM_BOOL,
+ "Current user is teaching one or more courses in this studyplan",
+ VALUE_OPTIONAL
+ ),
"suspended" => new \external_value(PARAM_BOOL, 'if studyplan is suspended', VALUE_OPTIONAL),
], 'Basic studyplan info', $value);
}
@@ -313,7 +327,7 @@ class studyplan {
$model = $this->simple_model();
$users = $this->find_linked_userids();
$sum = 0;
- foreach ( $users as $uid ) {
+ foreach ($users as $uid) {
$sum += $this->scanuserprogress($uid);
}
@@ -408,7 +422,8 @@ class studyplan {
public static function add($fields, $bare = false): self {
global $CFG, $DB;
- $addable = ['name', 'shortname', 'description', 'descriptionformat', 'idnumber', 'context_id', 'aggregation', 'aggregation_config'];
+ $addable = ['name', 'shortname', 'description', 'descriptionformat',
+ 'idnumber', 'context_id', 'aggregation', 'aggregation_config'];
$info = ['enddate' => null, "template" => 0, "suspended" => 0];
foreach ($addable as $f) {
if (array_key_exists($f, $fields)) {
@@ -418,8 +433,8 @@ class studyplan {
$id = $DB->insert_record(self::TABLE, $info);
$plan = self::find_by_id($id); // Make sure the new studyplan is immediately cached.
- // Add a single page and initialize it with placeholder data
- // This makes it easier to create a new study plan
+ // Add a single page and initialize it with placeholder data.
+ // This makes it easier to create a new study plan.
// On import, adding an empty page messes things up , so we have an option to skip this....
if (!$bare) {
@@ -459,7 +474,7 @@ class studyplan {
'aggregation',
'aggregation_config',
'suspended',
- 'template'
+ 'template',
];
$info = ['id' => $this->id ];
foreach ($editable as $f) {
@@ -501,7 +516,7 @@ class studyplan {
$DB->delete_records("local_treestudyplan_coach", ["studyplan_id" => $this->id]);
$DB->delete_records("local_treestudyplan_cohort", ["studyplan_id" => $this->id]);
$DB->delete_records("local_treestudyplan_user", ["studyplan_id" => $this->id]);
- //
+
$DB->delete_records('local_treestudyplan', ['id' => $this->id]);
return success::success();
}
@@ -768,7 +783,8 @@ class studyplan {
return $this->linkeduserids;
}
- /** Check if this studyplan is linked to a particular user
+ /**
+ * Check if this studyplan is linked to a particular user
* @param bool|stdClass $user The userid or user record of the user
*/
public function has_linked_user($user) {
@@ -785,13 +801,14 @@ class studyplan {
}
}
- /** Check if this studyplan is linked to a particular user
+ /**
+ * Check if this studyplan is linked to a particular user
* @param bool|stdClass|null $user The userid or user record of the user Leave empty to check current user.
*/
public function is_coach($user=null) {
- global $DB, $USER
- ;
- if ( ! (premium::enabled() && \get_config("local_treestudyplan", "enablecoach")) ) {
+ global $DB, $USER;
+
+ if (! (premium::enabled() && \get_config("local_treestudyplan", "enablecoach"))) {
// If coach role is not available, return false immediately.
return false;
}
@@ -803,7 +820,7 @@ class studyplan {
} else {
$userid = $user->id;
}
- $r = $DB->get_record(self::TABLE_COACH, ["studyplan_id" => $this->id, "user_id"=> $userid]);
+ $r = $DB->get_record(self::TABLE_COACH, ["studyplan_id" => $this->id, "user_id" => $userid]);
if ($r && has_capability(associationservice::CAP_COACH, $this->context(), $userid)) {
return true;
@@ -845,8 +862,8 @@ class studyplan {
$prg = $p->scanuserprogress($userid);
$progress += $prg;
}
- // Now average it out over the amount of pages
- if (count($pages) > 0 ) {
+ // Now average it out over the amount of pages.
+ if (count($pages) > 0) {
return $progress / count($pages);
} else {
return 0;
@@ -912,7 +929,7 @@ class studyplan {
'description' => $this->r->description,
'descriptionformat' => $this->r->descriptionformat,
'aggregation' => $this->r->aggregation,
- 'aggregation_config' => $this->r->aggregation_config
+ 'aggregation_config' => $this->r->aggregation_config,
], true);
// Copy any files related to this studyplan.
@@ -981,7 +998,7 @@ class studyplan {
$json = json_encode([
"type" => "studyplan",
"version" => 2.0,
- "studyplan" => $model
+ "studyplan" => $model,
], \JSON_PRETTY_PRINT);
return [ "format" => "application/json", "content" => $json];
}
@@ -1042,10 +1059,10 @@ class studyplan {
*/
public function import_files($importfiles, $area) {
$fs = get_file_storage();
- foreach($importfiles as $file) {
+ foreach ($importfiles as $file) {
if ($file['name'] != ".") {
$fileinfo = [
- 'contextid' =>\context_system::instance()->id, // ID of the system context.
+ 'contextid' => \context_system::instance()->id, // ID of the system context.
'component' => 'local_treestudyplan', // Your component name.
'filearea' => $area, // Usually = table name.
'itemid' => $this->id, // ID of the studyplanpage.
@@ -1094,11 +1111,11 @@ class studyplan {
// Create a new plan, based on the given parameters - this is the import studyplan part.
$plan = self::add( $planmodel, true);
- // Import the files
+ // Import the files.
if (isset( $planmodel['files']) && is_array( $planmodel['files'])) {
$plan->import_files( $planmodel['files'], "studyplan");
}
- // Import the icon
+ // Import the icon.
if (isset( $planmodel['iconfiles']) && is_array( $planmodel['iconfiles'])) {
$plan->import_files( $planmodel['iconfiles'], "icon");
}
@@ -1215,7 +1232,7 @@ class studyplan {
$fields = $DB->get_fieldset_sql($sql, ["studyplan_id" => $this->id]);
$list = [];
- foreach($fields as $id) {
+ foreach ($fields as $id) {
$list[] = studyline::find_by_id($id);
}
return $list;
diff --git a/classes/studyplanpage.php b/classes/studyplanpage.php
index 1de3e5f..9b700cf 100644
--- a/classes/studyplanpage.php
+++ b/classes/studyplanpage.php
@@ -154,9 +154,9 @@ class studyplanpage {
// The pluginfile URL which will serve the request.
'pluginfile.php',
- // The combination of contextid / component / filearea / itemid
- // form the virtual bucket that file are stored in.
- \context_system::instance()->id, // System instance is always used for this
+ // The combination of contextid / component / filearea / itemid.
+ // Form the virtual bucket that file are stored in.
+ \context_system::instance()->id, // System instance is always used for this.
'local_treestudyplan',
'studyplanpage',
$this->id
@@ -164,6 +164,10 @@ class studyplanpage {
return $text;
}
+ /**
+ * Determine studyplan timing
+ * @return string
+ */
public function timing() {
$now = new \DateTime();
if ($now > $this->startdate()) {
@@ -292,7 +296,7 @@ class studyplanpage {
}
$id = $DB->insert_record(self::TABLE, $info);
- // Refresh the page cache for the studyplan
+ // Refresh the page cache for the studyplan.
$plan->pages(true);
$page = self::find_by_id($id); // Make sure the new page is immediately cached.
@@ -301,7 +305,7 @@ class studyplanpage {
if (get_config("local_treestudyplan", "copystudylinesnewpage")) {
$templatepage = null;
$templatelines = [];
- // find the latest a page with lines in the plan
+ // Find the latest a page with lines in the plan.
$pages = $plan->pages(true);
foreach ($pages as $p) {
if ($p->id() != $id) {
@@ -322,7 +326,7 @@ class studyplanpage {
}
if (count(studyline::find_page_children($page)) == 0) {
- // Add an empty study line if there are currently no study lines
+ // Add an empty study line if there are currently no study lines.
$lineinfo = ['page_id' => $id,
'name' => get_string("default_line_name", "local_treestudyplan"),
'shortname' => get_string("default_line_shortname", "local_treestudyplan"),
@@ -450,8 +454,8 @@ class studyplanpage {
}
}
}
- if ( $courses > 0 ) {
- return ($completed/$courses);
+ if ($courses > 0) {
+ return ($completed / $courses);
} else {
return 0;
}
@@ -502,7 +506,7 @@ class studyplanpage {
'description' => $this->r->description,
'pages' => $this->r->pages,
'startdate' => $this->startdate()->add($timeoffset)->format("Y-m-d"),
- 'enddate' => empty($this->r->enddate) ? null : ($this->enddate()->add($timeoffset)->format("Y-m-d")),
+ 'enddate' => empty($this->r->enddate) ? null : ($this->enddate()->add($timeoffset)->format("Y-m-d")),
]);
// Copy any files related to this page.
@@ -585,7 +589,7 @@ class studyplanpage {
$json = json_encode([
"type" => "studyplanpage",
"version" => 2.0,
- "page" => $model
+ "page" => $model,
], \JSON_PRETTY_PRINT);
return [ "format" => "application/json", "content" => $json];
}
@@ -752,10 +756,10 @@ class studyplanpage {
*/
public function import_files($importfiles, $area) {
$fs = get_file_storage();
- foreach($importfiles as $file) {
+ foreach ($importfiles as $file) {
if ($file['name'] != ".") {
$fileinfo = [
- 'contextid' =>\context_system::instance()->id, // ID of the system context.
+ 'contextid' => \context_system::instance()->id, // ID of the system context.
'component' => 'local_treestudyplan', // Your component name.
'filearea' => $area, // Usually = table name.
'itemid' => $this->id, // ID of the studyplanpage.
diff --git a/classes/studyplanservice.php b/classes/studyplanservice.php
index 081d798..7c6c19e 100644
--- a/classes/studyplanservice.php
+++ b/classes/studyplanservice.php
@@ -133,7 +133,7 @@ class studyplanservice extends \external_api {
webservicehelper::require_capabilities([self::CAP_EDIT, self::CAP_VIEW], $studyplan->context());
}
- // If suspended, only allow the mapping if the user has edit rights
+ // If suspended, only allow the mapping if the user has edit rights.
if (!has_capability(self::CAP_EDIT, $studyplan->context()) && $studyplan->suspended()) {
return null;
} else {
@@ -237,7 +237,7 @@ class studyplanservice extends \external_api {
'description' => $description,
'periods' => $periods,
'startdate' => $startdate,
- 'enddate' => empty($enddate) ? null : $enddate,
+ 'enddate' => empty($enddate) ? null : $enddate,
'aggregation' => $aggregation,
'aggregation_config' => $aggregationconfig,
'context_id' => $contextid,
@@ -650,10 +650,10 @@ class studyplanservice extends \external_api {
'type' => $type,
'slot' => $slot,
'layer' => $layer,
- 'competency_id' => isset($details['competency_id']) ? $details['competency_id'] : null,
- 'course_id' => isset($details['course_id']) ? $details['course_id'] : null,
- 'badge_id' => isset($details['badge_id']) ? $details['badge_id'] : null,
- 'continuation_id' => isset($details['continuation_id']) ? $details['continuation_id'] : null,
+ 'competency_id' => isset($details['competency_id']) ? $details['competency_id'] : null,
+ 'course_id' => isset($details['course_id']) ? $details['course_id'] : null,
+ 'badge_id' => isset($details['badge_id']) ? $details['badge_id'] : null,
+ 'continuation_id' => isset($details['continuation_id']) ? $details['continuation_id'] : null,
]);
return $o->editor_model();
}
@@ -902,7 +902,6 @@ class studyplanservice extends \external_api {
*/
public static function list_badges_parameters(): \external_function_parameters {
return new \external_function_parameters( [
-
] );
}
@@ -922,12 +921,9 @@ class studyplanservice extends \external_api {
$result = [];
$badges = badges_get_badges(BADGE_TYPE_SITE, "timemodified");
+
foreach ($badges as $badge) {
- // TODO: Add config option to list only active badges.
$result[] = (new badgeinfo($badge))->editor_model();
-
- // TODO: Include course badges somehow... Just site badges is not enough.
-
}
return $result;
@@ -962,7 +958,7 @@ class studyplanservice extends \external_api {
*/
public static function search_badges($search, $active=false) {
$systemcontext = webservicehelper::system_context();
- // Check system permission to
+ // Check system permission to.
webservicehelper::require_capabilities("moodle/badges:viewbadges", $systemcontext);
$results = [];
@@ -1115,8 +1111,9 @@ class studyplanservice extends \external_api {
$studyplan = $item->studyline()->studyplan();
if (($studyplan->is_coach() && !$studyplan->suspended())
|| has_capability('local/treestudyplan:editstudyplan', $studyplan->context())
- || ($coursecontext && is_enrolled($coursecontext, $USER, 'local/treestudyplan:selectowngradables'))) {
- return coursecompetencyinfo::require_competency($competencyid, $itemid, $required)->model();
+ || ($coursecontext && is_enrolled($coursecontext, $USER, 'local/treestudyplan:selectowngradables'))
+ ) {
+ return coursecompetencyinfo::require_competency($competencyid, $itemid, $required)->model();
} else {
return success::fail("Access denied")->model();
}
@@ -1518,7 +1515,7 @@ class studyplanservice extends \external_api {
// Validate import context.
webservicehelper::require_capabilities(self::CAP_EDIT, $plan->context());
$result = $plan->import_pages($content, $format);
- return ($result ? success : :success(): success::fail())->model();
+ return ($result ? success::success() : success::fail())->model();
} catch (\webservice_access_exception $x) {
return success::fail("Access denied")->model();
}
@@ -1556,7 +1553,7 @@ class studyplanservice extends \external_api {
webservicehelper::require_capabilities(self::CAP_EDIT, $page->studyplan()->context());
$result = $page->import_studylines($content, $format);
- return ($result ? success : :success(): success::fail())->model();
+ return ($result ? success::success() : success::fail())->model();
} catch (\webservice_access_exception $x) {
return success::fail("Access denied")->model();
}
@@ -1622,7 +1619,7 @@ class studyplanservice extends \external_api {
foreach (explode("&", $formdata) as $pair) {
$p = explode("=", $pair, 2);
$k = urldecode($p[0]);
- $v = (count($p) > 1) ? urldecode($p[1]): "";
+ $v = (count($p) > 1) ? urldecode($p[1]) : "";
if (strpos($k, "[") > 0 && strpos($k, "]") == strlen($k) - 1) {
// Its a bracketet field, like filename[text] which should be separated and put into a named array.
@@ -1892,7 +1889,7 @@ class studyplanservice extends \external_api {
$studyitems = studyitem::find_studyline_children($line);
foreach ($studyitems as $item) {
if ($item->type() == studyitem::COURSE) {
- if ( $item->slot() <= $page->periods() ) {
+ if ($item->slot() <= $page->periods()) {
$period = $periods[$item->slot()];
try {
self::course_period_timing($period->id(), $item->courseid(), $item->span());
@@ -1991,11 +1988,11 @@ class studyplanservice extends \external_api {
$contextlevels = [CONTEXT_SYSTEM, CONTEXT_COURSECAT];
$list = [];
$roles = \get_all_roles();
- foreach($roles as $role) {
+ foreach ($roles as $role) {
$name = \role_get_name($role, $p->context()); // Get localized role name.
$rctxtlevels = \get_role_contextlevels($role->id);
$intersect = array_intersect($rctxtlevels, $contextlevels);
- if ( count($intersect) > 0 ) {
+ if (count($intersect) > 0) {
$list[] = [
'id' => $role->id,
'name' => $name,
@@ -2081,7 +2078,7 @@ class studyplanservice extends \external_api {
}
}
- // Trigger immediate cohort synchronization for this line only
+ // Trigger immediate cohort synchronization for this line only.
autocohortsync::syncline($o);
return $list;
}
@@ -2132,7 +2129,7 @@ class studyplanservice extends \external_api {
$list[] = self::student_enrol_status_model($userid, $o);
}
- // Trigger immediate cohort synchronization for this line only
+ // Trigger immediate cohort synchronization for this line only.
autocohortsync::syncline($o);
return $list;
}
@@ -2174,7 +2171,7 @@ class studyplanservice extends \external_api {
$list = [];
$p = $o->studyplan();
- foreach( $p->find_linked_userids() as $userid) {
+ foreach ($p->find_linked_userids() as $userid) {
$list[] = self::student_enrol_status_model($userid, $o);
}
@@ -2195,7 +2192,6 @@ class studyplanservice extends \external_api {
*/
public static function count_templates_parameters(): \external_function_parameters {
return new \external_function_parameters( [
-
] );
}
diff --git a/classes/success.php b/classes/success.php
index 9581d96..9f2c72b 100644
--- a/classes/success.php
+++ b/classes/success.php
@@ -63,7 +63,7 @@ class success {
* @param array|object $data Custom data to pass to receiver
*/
public function __construct($success, $msg, $data=[]) {
- $this->success = ($success) ? true : false;
+ $this->success = ($success) ? true : false;
$this->msg = $msg;
$this->data = json_encode($data);
}
diff --git a/classes/task/autocohortsync.php b/classes/task/autocohortsync.php
index eefb49d..adc342f 100644
--- a/classes/task/autocohortsync.php
+++ b/classes/task/autocohortsync.php
@@ -98,7 +98,7 @@ class autocohortsync extends \core\task\scheduled_task {
$userenroller = new cascadeusersync($plan);
$userenroller->sync($line);
}
- // Leave the csync required flag for the next auto update
+ // Leave the csync required flag for the next auto update.
}
}
diff --git a/classes/teachingfinder.php b/classes/teachingfinder.php
index bcf1329..8e0fe34 100644
--- a/classes/teachingfinder.php
+++ b/classes/teachingfinder.php
@@ -64,7 +64,7 @@ class teachingfinder {
public static function is_teaching_studyplan(studyplan $plan, $userid) {
global $DB;
$count = $DB->count_records(self::TABLE, ['teacher_id' => $userid, "studyplan_id" => $plan->id()]);
- return ($count > 0)?true:false;
+ return ($count > 0) ? true : false;
}
/**
@@ -76,7 +76,7 @@ class teachingfinder {
public static function is_teaching($userid) {
global $DB;
$count = $DB->count_records(self::TABLE, ['teacher_id' => $userid]);
- return ($count > 0)?true:false;
+ return ($count > 0) ? true : false;
}
/**
@@ -167,7 +167,6 @@ class teachingfinder {
$now = time();
foreach ($list as $pageid) {
// Retrieve the studyplan id from the page.
- // TODO: Change this when page management is implemented to return the page instead of the plan.
$planid = $DB->get_field("local_treestudyplan_page", "studyplan_id", ["id" => $pageid]);
$DB->insert_record(self::TABLE, ["teacher_id" => $userid, "studyplan_id" => $planid, "update_time" => $now]);
}
diff --git a/classes/utilityservice.php b/classes/utilityservice.php
index 8fb5b49..1ed463c 100644
--- a/classes/utilityservice.php
+++ b/classes/utilityservice.php
@@ -45,6 +45,12 @@ class utilityservice extends \external_api {
*/
const CAP_VIEW = "local/treestudyplan:viewuserreports";
+ /**
+ * Load moodle form
+ * @param string $formname
+ * @param string $params
+ * @param string $ajaxformdata
+ */
protected static function load_mform($formname, $params, $ajaxformdata = null) {
global $CFG;
/* We don't need to load the form php file (class autoloading will handle that)
@@ -58,7 +64,7 @@ class utilityservice extends \external_api {
$mformclassname = "\\local_treestudyplan\\form\\{$formname}";
// Check if the form is a subclass of formbase.
- if ( is_a($mformclassname, "\\local_treestudyplan\\form\\formbase", true)) {
+ if (is_a($mformclassname, "\\local_treestudyplan\\form\\formbase", true)) {
$jsonparams = json_decode($params);
$mform = new $mformclassname( $jsonparams, $ajaxformdata);
return $mform;
@@ -121,7 +127,7 @@ class utilityservice extends \external_api {
return new \external_single_structure(
[
'html' => new \external_value(PARAM_RAW, 'HTML fragment.'),
- 'javascript' => new \external_value(PARAM_RAW, 'JavaScript fragment')
+ 'javascript' => new \external_value(PARAM_RAW, 'JavaScript fragment'),
]
);
}
@@ -157,7 +163,7 @@ class utilityservice extends \external_api {
$ajaxformdata = [];
parse_str($formdata, $ajaxformdata);
- // Load the form, provide submitted form data and perform security checks
+ // Load the form, provide submitted form data and perform security checks.
$mform = self::load_mform($formname, $params, $ajaxformdata);
$return = $mform->process_submission();
@@ -165,7 +171,7 @@ class utilityservice extends \external_api {
return success::success($return)->model();
}
- /**
+ /**
* Parameter description for webservice function submit_cm_editform
*/
public static function getsettings_parameters(): \external_function_parameters {
@@ -193,7 +199,7 @@ class utilityservice extends \external_api {
return [
"hivizdropslots" => get_config("local_treestudyplan", "hivizdropslots"),
- "toolboxleft" => get_config("local_treestudyplan", "toolboxleft")
+ "toolboxleft" => get_config("local_treestudyplan", "toolboxleft"),
];
}
diff --git a/cli/generate_badges.php b/cli/generate_badges.php
index 1f0f65f..501376d 100644
--- a/cli/generate_badges.php
+++ b/cli/generate_badges.php
@@ -41,7 +41,7 @@ Options:
list($options, $unrecognised) = cli_get_params([
'help' => false,
- 'amount' => null
+ 'amount' => null,
], [
'h' => 'help',
'a' => 'amount',
@@ -56,7 +56,7 @@ if ($options['help']) {
cli_writeln($usage);
exit(2);
}
-print_r($options);
+
if (empty($options['amount']) || !is_numeric($options['amount'])) {
cli_error('Missing mandatory integer argument "amount"', 2);
}
@@ -76,7 +76,7 @@ if (!$user) {
cli_error("Unable to find admin user in DB.");
}
-$auth = empty($user->auth) ? 'manual' : $user->auth;
+$auth = empty($user->auth) ? 'manual' : $user->auth;
if ($auth == 'nologin' || !is_enabled_auth($auth)) {
cli_error(sprintf("User authentication is either 'nologin' or disabled. Check Moodle authentication method for '%s'",
$user->username));
@@ -87,8 +87,8 @@ $authplugin->sync_roles($user);
login_attempt_valid($user);
complete_user_login($user);
-for ($i=0; $i<$amount; $i++) {
- $fortune = shell_exec("{$fortunepath} -n 160 ");
+for ($i = 0; $i < $amount; $i++) {
+ $fortune = shell_exec("{$fortunepath} -n 160 ");
$pieces = explode(" ", $fortune);
$name = implode(" ", array_splice($pieces, 0, 4));
@@ -118,7 +118,7 @@ for ($i=0; $i<$amount; $i++) {
$fordb->issuercontact = $CFG->badges_defaultissuercontact;
$fordb->expiredate = null;
- $fordb->expireperiod = null;
+ $fordb->expireperiod = null;
$fordb->type = \BADGE_TYPE_SITE; // Site badge.
$fordb->courseid = null;
$fordb->messagesubject = get_string('messagesubject', 'badges');
diff --git a/cli/prime_students.php b/cli/prime_students.php
index 98b0604..bcf2749 100644
--- a/cli/prime_students.php
+++ b/cli/prime_students.php
@@ -43,7 +43,7 @@ list($options, $unrecognised) = cli_get_params([
'help' => false,
'studyplan' => null,
'all' => false,
- 'file' => "/tmp/generategrades.json"
+ 'file' => "/tmp/generategrades.json",
], [
'h' => 'help',
's' => 'studyplan',
diff --git a/cli/randomize_grades.php b/cli/randomize_grades.php
index 1e2f07b..85ac580 100644
--- a/cli/randomize_grades.php
+++ b/cli/randomize_grades.php
@@ -47,13 +47,13 @@ list($options, $unrecognised) = cli_get_params([
'dryrun' => false,
'studyplan' => null,
'all' => false,
- 'file' => "/tmp/generategrades.json"
+ 'file' => "/tmp/generategrades.json",
], [
'h' => 'help',
's' => 'studyplan',
'a' => 'all',
'f' => 'file',
- 'd' => 'dryrun'
+ 'd' => 'dryrun',
]);
if ($unrecognised) {
@@ -73,7 +73,7 @@ if (!$user) {
cli_error("Unable to find admin user in DB.");
}
-$auth = empty($user->auth) ? 'manual' : $user->auth;
+$auth = empty($user->auth) ? 'manual' : $user->auth;
if ($auth == 'nologin' || !is_enabled_auth($auth)) {
cli_error(sprintf("User authentication is either 'nologin' or disabled. Check Moodle authentication method for '%s'",
$user->username));
@@ -150,7 +150,10 @@ foreach ($plans as $plan) {
$ug = $a->get_user_grade($u->id, true);
$ug->grade = grade_floatval($gg->grade);
$ug->grader = $USER->id;
- $ug->feedbacktext = nl2br( htmlspecialchars($gg->fb));
+ $ug->feedbacktext = nl2br(htmlspecialchars(
+ $gg->fb,
+ ENT_QUOTES | ENT_SUBSTITUTE | ENT_HTML401
+ ));
$ug->feedbackformat = FORMAT_HTML;
if (!$options["dryrun"]) {
diff --git a/coach.php b/coach.php
index 603cdcb..d7d5bd8 100644
--- a/coach.php
+++ b/coach.php
@@ -14,7 +14,7 @@
// You should have received a copy of the GNU General Public License
// along with Moodle. If not, see {$text->studyplan_noneselected}
{$text->studyplan_noneselected}
{$text->studyplan_noneselected}