Issue #9 - Fixed code standards.

This commit is contained in:
Daniel Thee Roperto
2016-09-06 14:21:11 +10:00
parent d0c4a1c0be
commit 4cccb518f3
4 changed files with 7 additions and 6 deletions

View File

@@ -53,7 +53,7 @@ class auth_plugin_outage extends auth_plugin_base
/**
* Login page hook.
*/
function loginpage_hook() {
public function loginpage_hook() {
\auth_outage\outagelib::initialize();
}
}

View File

@@ -86,8 +86,8 @@ class outage {
outagelib::data2object($data, $this);
// FIXME types are wrong. Is this behaving as expected?
foreach (['createdby', 'id', 'lastmodified', 'modifiedby', 'starttime', 'stoptime', 'warningduration']
as $f) {
$fields = ['createdby', 'id', 'lastmodified', 'modifiedby', 'starttime', 'stoptime', 'warningduration'];
foreach ($fields as $f) {
$this->$f = ($this->$f === null) ? null : (int)$this->$f;
}

View File

@@ -164,8 +164,7 @@ final class outagedb {
FROM {auth_outage}
WHERE (starttime - warningduration <= :now1 AND stoptime >= :now2)
ORDER BY starttime ASC, stoptime DESC, title ASC
LIMIT 1
',
LIMIT 1',
['now1' => $now, 'now2' => $now]
);

View File

@@ -64,7 +64,9 @@ class outagelib {
}
self::$initialized = true;
if (($active = outagedb::getactive()) == null) return;
if (($active = outagedb::getactive()) == null) {
return;
}
$CFG->additionalhtmltopofbody .= self::get_renderer()->renderbar($active);
}