Merge pull request #91 from joybrusich/bugfixes

Fixes #90 $instance not initiated

lang/en/enrol_apply.php

@@ -92,6 +92,7 @@ $string['notification'] = '<b>Enrolment application successfully sent</b>. <br/>
 $string['mailtoteacher_suject'] = 'New Enrolment request!';
 $string['editdescription'] = 'Textarea description';
 $string['comment'] = 'Comment';
+$string['applycomment'] = 'Comment';
 $string['applymanage'] = 'Manage enrolment applications';
 
 $string['status_desc'] = 'Allow course access of internally enrolled users.';

lang/en_us/en_us_enrol_apply.php

@@ -57,6 +57,7 @@ $string['sendmailtomanager'] = 'Send email notification to managers';
 $string['mailtoteacher_suject'] = 'New enrollment request!';
 $string['editdescription'] = 'Textarea description';
 $string['comment'] = 'Comment';
+$string['applycomment'] = 'Comment';
 $string['applymanage'] = 'Manage enrollment applications';
 
 $string['status_desc'] = 'Allow course access of internally enrolled users.';

manage.php

@@ -38,6 +38,7 @@ if ($id == null) {
     $context = context_system::instance();
     require_capability('enrol/apply:manageapplications', $context);
     $pageheading = get_string('confirmusers', 'enrol_apply');
+    $instance = null;
 } else {
     $instance = $DB->get_record('enrol', array('id' => $id, 'enrol' => 'apply'), '*', MUST_EXIST);
     require_course_login($instance->courseid);

renderer.php

@@ -91,7 +91,8 @@ class enrol_apply_renderer extends plugin_renderer_base {
             'fullname', // Magic happens here: The column heading will automatically be set due to column name 'fullname'.
             get_string('email'),
             get_string('applydate', 'enrol_apply'),
-            $instance->customtext2);
+            get_string('applycomment', 'enrol_apply'),
+        );
         $table->define_columns($columns);
         $table->define_headers($headers);