diff options
author | Valentin Popov <info@valentineus.link> | 2017-11-18 05:31:25 +0300 |
---|---|---|
committer | Valentin Popov <info@valentineus.link> | 2017-11-18 05:31:25 +0300 |
commit | 97140dccf76f878991853d827449ca608eb91864 (patch) | |
tree | 7017e6d948a6aaa0bd36a10921987cc614132322 /managerservice.php | |
parent | bf2cef3ea035f4487e4b41e3033c60b1f1f0ead3 (diff) | |
download | local_webhooks-97140dccf76f878991853d827449ca608eb91864.tar.xz local_webhooks-97140dccf76f878991853d827449ca608eb91864.zip |
Aesthetics in the code
Diffstat (limited to 'managerservice.php')
-rw-r--r-- | managerservice.php | 25 |
1 files changed, 13 insertions, 12 deletions
diff --git a/managerservice.php b/managerservice.php index 48b14a9..066dc99 100644 --- a/managerservice.php +++ b/managerservice.php @@ -27,13 +27,14 @@ require_once($CFG->libdir . "/tablelib.php"); require_once($CFG->libdir . "/adminlib.php"); admin_externalpage_setup("pluginsoverview"); - -$backupservices = optional_param("backup", 0, PARAM_BOOL); -$deleteid = optional_param("deleteid", 0, PARAM_INT); -$hideshowid = optional_param("hideshowid", 0, PARAM_INT); - require_login(); +/* Optional parameters */ +$backupservices = optional_param("getbackup", 0, PARAM_BOOL); +$deleteid = optional_param("deleteid", 0, PARAM_INT); +$hideshowid = optional_param("hideshowid", 0, PARAM_INT); + +/* Used references */ $editservice = "/local/webhooks/editservice.php"; $managerservice = "/local/webhooks/managerservice.php"; $restorebackup = "/local/webhooks/restorebackup.php"; @@ -74,7 +75,7 @@ if (boolval($hideshowid) && confirm_sesskey()) { } /* Page template */ -$titlepage = new lang_string("externalservices", "webservice"); +$titlepage = new lang_string("pluginname", "local_webhooks"); $PAGE->set_pagelayout("admin"); $PAGE->set_title($titlepage); $PAGE->set_heading($titlepage); @@ -95,13 +96,13 @@ $table->setup(); foreach ($callbacks as $callback) { /* Filling of information columns */ $titlecallback = html_writer::div($callback->title, "title"); - $urlcallback = html_writer::div($callback->url, "url"); + $urlcallback = html_writer::div($callback->url, "url"); /* Defining service status */ - $hideshowicon = "t/show"; + $hideshowicon = "t/show"; $hideshowstring = new lang_string("enable", "moodle"); if (boolval($callback->enable)) { - $hideshowicon = "t/hide"; + $hideshowicon = "t/hide"; $hideshowstring = new lang_string("disable", "moodle"); } @@ -125,11 +126,11 @@ foreach ($callbacks as $callback) { $table->print_html(); /* Add service button */ -$addurl = new moodle_url($editservice); -echo $OUTPUT->single_button($addurl, new lang_string("addaservice", "webservice"), "get"); +$addserviceurl = new moodle_url($editservice); +echo $OUTPUT->single_button($addserviceurl, new lang_string("addaservice", "webservice"), "get"); /* Button to get a backup */ -$backupurl = new moodle_url($managerservice, array("backup" => true)); +$backupurl = new moodle_url($managerservice, array("getbackup" => true)); echo $OUTPUT->single_button($backupurl, new lang_string("backup", "moodle"), "get"); /* Button for restoring settings */ |