search

ericmerrill / Banner-Luminis-Message-Broker-for-Moodle

A plugin for Moodle to import Banner/Luminis Message Broker data
13 stars 8 forks source link

syntax error in lmb_assign_role_log #6

Closed ghost closed 11 years ago

ghost commented 11 years ago

In "master" branch from lib.php:

public function lmb_assign_role_log($roleid, $courseid, $userid, &$logline, $restrictstart = 0; $restrictend = 0) {

Error results in using semicolon instead of comma.

emerrill commented 11 years ago

Thanks for pointing this out - I would note the master should be considered unstable, I use it for staging between releases.

-eric On Jul 17, 2013 12:23 PM, "timgus" notifications@github.com wrote:

In "master" branch from lib.php:

public function lmb_assign_role_log($roleid, $courseid, $userid, &$logline, $restrictstart = 0; $restrictend = 0) {

Error results in using semicolon instead of comma.

— Reply to this email directly or view it on GitHubhttps://github.com/merrill-oakland/Banner-Luminis-Message-Broker-for-Moodle/issues/6 .

ericmerrill commented 11 years ago

This has been fixed in master.