diff options
author | Valentin Popov <info@valentineus.link> | 2018-11-01 23:36:09 +0300 |
---|---|---|
committer | Valentin Popov <info@valentineus.link> | 2018-11-02 00:07:14 +0300 |
commit | d8f3a9f3edf66fad0af84a703e39e45911f37d85 (patch) | |
tree | fb73b9abcb557b7f86d026a1294315c958e79432 /editservice.php | |
parent | f9772b1fb97b81468793388ed8af3c06f4f3e279 (diff) | |
download | local_webhooks-d8f3a9f3edf66fad0af84a703e39e45911f37d85.tar.xz local_webhooks-d8f3a9f3edf66fad0af84a703e39e45911f37d85.zip |
Formatting code and eliminating possible errors
Signed-off-by: Valentin Popov <info@valentineus.link>
Diffstat (limited to 'editservice.php')
-rw-r--r-- | editservice.php | 28 |
1 files changed, 14 insertions, 14 deletions
diff --git a/editservice.php b/editservice.php index 8a7d4f6..821e0cc 100644 --- a/editservice.php +++ b/editservice.php @@ -22,22 +22,22 @@ * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later */ -require_once(__DIR__ . "/../../config.php"); -require_once(__DIR__ . "/classes/service_form.php"); -require_once(__DIR__ . "/lib.php"); +require_once(__DIR__ . '/../../config.php'); +require_once(__DIR__ . '/classes/service_form.php'); +require_once(__DIR__ . '/lib.php'); -require_once($CFG->libdir . "/adminlib.php"); +require_once($CFG->libdir . '/adminlib.php'); /* Optional parameters */ -$serviceid = optional_param("serviceid", 0, PARAM_INT); +$serviceid = optional_param('serviceid', 0, PARAM_INT); /* Link generation */ -$urlparameters = array("serviceid" => $serviceid); -$baseurl = new moodle_url("/local/webhooks/editservice.php", $urlparameters); -$managerservice = new moodle_url("/local/webhooks/index.php"); +$urlparameters = array('serviceid' => $serviceid); +$baseurl = new moodle_url('/local/webhooks/editservice.php', $urlparameters); +$managerservice = new moodle_url('/local/webhooks/index.php'); /* Configure the context of the page */ -admin_externalpage_setup("local_webhooks", "", null, $baseurl, array()); +admin_externalpage_setup('local_webhooks', '', null, $baseurl); $context = context_system::instance(); /* Create an editing form */ @@ -50,7 +50,7 @@ if ($mform->is_cancelled()) { /* Getting the data */ $servicerecord = new stdClass(); -if ($editing = boolval($serviceid)) { +if ($editing = (bool) $serviceid) { $servicerecord = local_webhooks_get_record($serviceid); $mform->set_data($servicerecord); } @@ -60,15 +60,15 @@ if ($data = $mform->get_data()) { if ($editing) { $data->id = $serviceid; local_webhooks_update_record($data, false); - redirect($managerservice, new lang_string("eventwebserviceserviceupdated", "webservice")); + redirect($managerservice, new lang_string('eventwebserviceserviceupdated', 'webservice')); } else { - local_webhooks_update_record($data, true); - redirect($managerservice, new lang_string("eventwebserviceservicecreated", "webservice")); + local_webhooks_update_record($data); + redirect($managerservice, new lang_string('eventwebserviceservicecreated', 'webservice')); } } /* The page title */ -$titlepage = new lang_string("externalservice", "webservice"); +$titlepage = new lang_string('externalservice', 'webservice'); $PAGE->navbar->add($titlepage); $PAGE->set_heading($titlepage); $PAGE->set_title($titlepage); |