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

Comprehensive drops don't work if CRNs are under 5 digits #32

Open ericmerrill opened 3 years ago

ericmerrill commented 3 years ago

Line https://github.com/merrill-oakland/Banner-Luminis-Message-Broker-for-Moodle/blob/fac423aa98765073a2822ca059ccc88c171cbfd8/lib.php#L2378

Should be changed to if (preg_match('{[0-9]+\.(.+?)$}is', $enrolment->coursesourcedid, $matches)) {