Browse Source

couple of coding guideline changes.

MOODLE_36_STABLE
Dan Marsden 6 years ago
parent
commit
85aa66d99e
  1. 10
      classes/output/mobile.php

10
classes/output/mobile.php

@ -33,12 +33,12 @@ defined('MOODLE_INTERNAL') || die();
*/ */
class mobile { class mobile {
/* /**
* Subnet warning. - constants used to prevent warnings from showing multiple times. * Subnet warning. - constants used to prevent warnings from showing multiple times.
*/ */
const MESSAGE_SUBNET = 10; const MESSAGE_SUBNET = 10;
/* /**
* Prevent shared warning. used to prevent warnings from showing multiple times. * Prevent shared warning. used to prevent warnings from showing multiple times.
*/ */
const MESSAGE_PREVENTSHARED = 30; const MESSAGE_PREVENTSHARED = 30;
@ -153,7 +153,8 @@ class mobile {
list($canmark, $reason) = attendance_can_student_mark($sess); list($canmark, $reason) = attendance_can_student_mark($sess);
if (!$isteacher && $reason == 'preventsharederror') { if (!$isteacher && $reason == 'preventsharederror') {
$data['showmessage'] = true; $data['showmessage'] = true;
$data['messages'][self::MESSAGE_PREVENTSHARED]['string'] = 'preventsharederror'; // Lang string to show as a message. // Lang string to show as a message.
$data['messages'][self::MESSAGE_PREVENTSHARED]['string'] = 'preventsharederror';
} }
if ($isteacher || $canmark) { if ($isteacher || $canmark) {
@ -174,7 +175,8 @@ class mobile {
if (!$isteacher) { if (!$isteacher) {
if (!empty($sess->subnet) && !address_in_subnet(getremoteaddr(), $sess->subnet)) { if (!empty($sess->subnet) && !address_in_subnet(getremoteaddr(), $sess->subnet)) {
$data['showmessage'] = true; $data['showmessage'] = true;
$data['messages'][self::MESSAGE_SUBNET]['string'] = 'subnetwrong'; // Lang string to show as a message. // Lang string to show as a message.
$data['messages'][self::MESSAGE_SUBNET]['string'] = 'subnetwrong';
$html['sessid'] = null; // Unset sessid as we cannot record session on this ip. $html['sessid'] = null; // Unset sessid as we cannot record session on this ip.
} else if ($sess->autoassignstatus && empty($sess->studentpassword)) { } else if ($sess->autoassignstatus && empty($sess->studentpassword)) {
$statusid = attendance_session_get_highest_status($att, $sess); $statusid = attendance_session_get_highest_status($att, $sess);

Loading…
Cancel
Save