From 9c5bc7a58037799817495ab4f1dc558da548b3ce Mon Sep 17 00:00:00 2001 From: Andreas Grabs Date: Fri, 28 Oct 2011 00:30:42 +0200 Subject: [PATCH] MDL-29804 - Clean up Feedback modules coding style --- mod/feedback/README.txt | 3 +- mod/feedback/analysis.php | 75 +- mod/feedback/analysis_course.php | 107 +- mod/feedback/analysis_to_excel.php | 256 ++--- mod/feedback/backup/moodle1/lib.php | 1 - .../backup_feedback_activity_task.class.php | 3 +- .../moodle2/backup_feedback_settingslib.php | 1 - .../moodle2/backup_feedback_stepslib.php | 3 - .../restore_feedback_activity_task.class.php | 1 - .../moodle2/restore_feedback_stepslib.php | 1 - mod/feedback/choose_group_form.php | 87 +- mod/feedback/complete.php | 1016 +++++++++-------- mod/feedback/complete_guest.php | 861 +++++++------- mod/feedback/db/access.php | 15 + mod/feedback/db/install.php | 16 +- mod/feedback/db/install.xml | 2 +- mod/feedback/db/log.php | 1 - mod/feedback/db/messages.php | 1 - mod/feedback/db/upgrade.php | 136 ++- mod/feedback/delete_completed.php | 179 +-- mod/feedback/delete_completed_form.php | 39 +- mod/feedback/delete_item.php | 133 ++- mod/feedback/delete_item_form.php | 39 +- mod/feedback/delete_template.php | 68 +- mod/feedback/delete_template_form.php | 40 +- mod/feedback/edit.php | 411 ++++--- mod/feedback/edit_form.php | 128 ++- mod/feedback/edit_item.php | 41 +- mod/feedback/export.php | 306 ++--- mod/feedback/import.php | 527 ++++----- mod/feedback/import_form.php | 52 +- mod/feedback/index.php | 33 +- mod/feedback/item/captcha/captcha_form.php | 24 +- mod/feedback/item/captcha/lib.php | 150 ++- mod/feedback/item/captcha/print_captcha.php | 46 +- mod/feedback/item/feedback_item_class.php | 154 ++- .../item/feedback_item_form_class.php | 62 +- mod/feedback/item/info/info_form.php | 27 +- mod/feedback/item/info/lib.php | 235 ++-- mod/feedback/item/label/label_form.php | 29 +- mod/feedback/item/label/lib.php | 160 ++- mod/feedback/item/multichoice/lib.php | 526 +++++---- .../item/multichoice/multichoice_form.php | 70 +- mod/feedback/item/multichoicerated/lib.php | 401 ++++--- .../multichoicerated_form.php | 74 +- mod/feedback/item/numeric/lib.php | 468 +++++--- mod/feedback/item/numeric/numeric_form.php | 45 +- mod/feedback/item/textarea/lib.php | 247 ++-- mod/feedback/item/textarea/textarea_form.php | 30 +- mod/feedback/item/textfield/lib.php | 240 ++-- .../item/textfield/textfield_form.php | 30 +- mod/feedback/lang/en/feedback.php | 11 +- mod/feedback/lib.php | 847 ++++++++------ mod/feedback/mapcourse.php | 38 +- mod/feedback/mod_form.php | 143 ++- mod/feedback/print.php | 65 +- mod/feedback/settings.php | 21 +- mod/feedback/show_entries.php | 161 ++- mod/feedback/show_entries_anonym.php | 109 +- mod/feedback/show_nonrespondents.php | 513 +++++---- mod/feedback/styles.css | 2 +- mod/feedback/tabs.php | 143 ++- mod/feedback/unmapcourse.php | 14 + mod/feedback/use_templ.php | 61 +- mod/feedback/use_templ_form.php | 31 +- mod/feedback/version.php | 38 +- mod/feedback/view.php | 165 ++- 67 files changed, 5875 insertions(+), 4087 deletions(-) diff --git a/mod/feedback/README.txt b/mod/feedback/README.txt index 9a0c603be7e..c88bf751dcc 100644 --- a/mod/feedback/README.txt +++ b/mod/feedback/README.txt @@ -135,7 +135,8 @@ Now it only runs on Moodle 1.8 or later - roles are full implemented - now radiobuttons and checkboxes can be aligned horizontally or vertically - now you can insert pagebreakes -- a feedback what is switch to the next page is saved temporary. the user can cancel the completion and later continue at the last filled page. +- a feedback what is switch to the next page is saved temporary. + the user can cancel the completion and later continue at the last filled page. - now you can ex-/import feedbacks into/from a xml-file - course-reseting is supported diff --git a/mod/feedback/analysis.php b/mod/feedback/analysis.php index 78f46d52b39..da86ecf7f4a 100644 --- a/mod/feedback/analysis.php +++ b/mod/feedback/analysis.php @@ -1,4 +1,18 @@ . /** * shows an analysed view of feedback @@ -11,7 +25,6 @@ require_once("../../config.php"); require_once("lib.php"); -// $SESSION->feedback->current_tab = 'analysis'; $current_tab = 'analysis'; $id = required_param('id', PARAM_INT); //the POST dominated the GET @@ -39,23 +52,23 @@ if (!$context = get_context_instance(CONTEXT_MODULE, $cm->id)) { print_error('badcontext'); } -if($course->id == SITEID) { +if ($course->id == SITEID) { require_login($course->id, true); -}else{ +} else { require_login($course->id, true, $cm); } //check whether the given courseid exists -if($courseid AND $courseid != SITEID) { - if($course2 = $DB->get_record('course', array('id'=>$courseid))){ +if ($courseid AND $courseid != SITEID) { + if ($course2 = $DB->get_record('course', array('id'=>$courseid))) { require_course_login($course2); //this overwrites the object $course :-( $course = $DB->get_record("course", array("id"=>$cm->course)); // the workaround - }else { + } else { print_error('invalidcourseid'); } } -if( !( ((intval($feedback->publish_stats) == 1) AND +if ( !( ((intval($feedback->publish_stats) == 1) AND has_capability('mod/feedback:viewanalysepage', $context)) OR has_capability('mod/feedback:viewreports', $context))) { print_error('error'); @@ -65,30 +78,29 @@ if( !( ((intval($feedback->publish_stats) == 1) AND $strfeedbacks = get_string("modulenameplural", "feedback"); $strfeedback = get_string("modulename", "feedback"); -$PAGE->navbar->add(get_string('analysis','feedback')); +$PAGE->navbar->add(get_string('analysis', 'feedback')); $PAGE->set_heading(format_string($course->fullname)); $PAGE->set_title(format_string($feedback->name)); echo $OUTPUT->header(); /// print the tabs -include('tabs.php'); +require('tabs.php'); //print analysed items echo $OUTPUT->box_start('generalbox boxaligncenter boxwidthwide'); //get the groupid -$groupselect = groups_print_activity_menu($cm, $CFG->wwwroot . '/mod/feedback/analysis.php?id=' . $cm->id.'&do_show=analysis', true); +$myurl = $CFG->wwwroot.'/mod/feedback/analysis.php?id='.$cm->id.'&do_show=analysis'; +$groupselect = groups_print_activity_menu($cm, $myurl, true); $mygroupid = groups_get_activity_group($cm); -if( has_capability('mod/feedback:viewreports', $context) ) { +if ( has_capability('mod/feedback:viewreports', $context) ) { echo isset($groupselect) ? $groupselect : ''; echo '
'; //button "export to excel" - //echo '
'; - // echo '