General developer forum

Hi guys,

I have a local plugin which handles a mandatory 'survey' for students enrolled into a specific course, among other things. Everything is working in this plugin except for when I try to use require_login($course) in one of the files.

Here is what I mean.

In lib.php, I have the following code:

defined('MOODLE_INTERNAL') || die;

function local_mbaentrepreneurship_extend_settings_navigation($settingsnav, $context) {
    global $CFG, $PAGE;
    // Only add this settings item on non-site course pages.
    if (!$PAGE->course or $PAGE->course->id == 1) {
        return;
    }
    // Only let users with the appropriate capability see this settings item.
    if (!has_capability('moodle/backup:backupcourse', context_course::instance($PAGE->course->id))) {
        // return;
    }
    if ($settingnode = $settingsnav->find('courseadmin', navigation_node::TYPE_COURSE)) {
        $mbaentrepreneurshipsettings = get_string('mbaentrepreneurshipsettings', 'local_mbaentrepreneurship');
        $url = new moodle_url('/local/mbaentrepreneurship/data.php', array('id' => $PAGE->course->id));
        if ($PAGE->course->id == 63) { // this is the module that I'm targeting
            $mbaentrepreneurship = navigation_node::create(
                            $mbaentrepreneurshipsettings, $url, navigation_node::NODETYPE_LEAF, 'mbaentrepreneurship', 'mbaentrepreneurship', new pix_icon('t/addcontact', $mbaentrepreneurshipsettings)
            );
            if ($PAGE->url->compare($url, URL_MATCH_BASE)) {
                $mbaentrepreneurshipsettings->make_active();
            }
            $settingnode->add_node($mbaentrepreneurship);
        }
    }
}

I require this file in /local/mbaentrepreneurship/index.php, and then do this:

$courseid = optional_param('courseid', 0, PARAM_INT);
    $userid = optional_param('userid', 0, PARAM_INT);
    if ($courseid !== 63) {
        // throw an exception here
        throw new moodle_exception(' Invalid course has been passed to the page. Please make sure that you\'re on an MBA'
        . ' Entrepreneurship page.');
    }
    $course = $DB->get_record('course', array('id' => $courseid), '*', MUST_EXIST);
    $student = $DB->get_record('user', array('id' => $userid));
    $context = context_course::instance($course->id, MUST_EXIST);
    require_login($course);
    if (!is_siteadmin()) {
        // well, not gonna bother with this!
    }
    $PAGE->set_context($context);
    $PAGE->set_url('/local/mbaentrepreneurship', array(
        'courseid' => $course->id,
        'userid' => $student->id
    ));
    $PAGE->set_heading($course->fullname);
    $PAGE->set_pagelayout('course');
    $PAGE->set_title('MBA Entrepreneurship: Choice');
    $PAGE->navbar->add(fullname($student) . ': Make Your Assessment Choice', new moodle_url('/local/mbaentrepreneurship', array(
        'courseid' => $course->id,
        'userid' => $student->id
    )));
    $PAGE->requires->js('/local/mbaentrepreneurship/mbaentrepreneurship.js');

This code in index.php works like a charm! But then I have another file called data.php, in the same plugin, and I do a call to require_login($course), and then it throws an error. Here is the code in data.php:

require_once('../../config.php');
require_once 'lib.php';
$id = optional_param('id', 0, PARAM_INT);
$course = $DB->get_record('course', array('id' => $id));
$context = context_course::instance($course->id, MUST_EXIST);
require_login($course); // THROWS AN ERROR HERE
$PAGE->set_context($context);
$PAGE->set_url('/local/mbaentrepreneurship/data.php', array(
    'id' => $course->id
));
$PAGE->set_pagetype('course-view-' . $course->format);
$PAGE->set_title($course->shortname);
$PAGE->set_pagelayout('course');
$PAGE->navbar->add('Programme', new moodle_url('/local/programme', array(
    'id' => 3
)));
$PAGE->navbar->add('MBA', new moodle_url('/course/index.php', array(
    'categoryid' => 3
)));
$PAGE->navbar->add($course->fullname, new moodle_url('/course/view.php', array(
    'id' => 63
)));
echo $OUTPUT->header();

The above code in data.php throws an error below, and I've noticed that if I removed the require_login($course) part, the error goes away. In fact, if I do require_login() alone, the page works. And its something to do with the navigation node.

Can you please point me in the right direction so that I can get this right and working well.

Exception - Call to a member function make_active() on string