search

Panopto / Moodle-2.0-plugin-for-Panopto

Panopto's integration with the Moodle LMS.
http://www.panopto.com
GNU General Public License v3.0
18 stars 38 forks source link

Blank shortname #73

Closed elearningatbath closed 7 years ago

elearningatbath commented 7 years ago

https://github.com/Panopto/Moodle-2.0-plugin-for-Panopto/blob/master/lib/panopto_data.php#L236

There could be courses on a Moodle site with no shortnames. For such courses, Panopto would provision them as ":Course Long name" as folder names in Panopto. There should be an if empty check for $provisioninginfo->shortname and be replaced with a suitable value when pushing to the Panopto server.

A possible suggestion could be using the initial letters of the long name in cases where no short name has been found.

Something like :

if(empty($provisioninginfo->shortname ){
$provisioninginfo->shortname  = substr($provisioninginfo->longname,0, 5); 
}

Regards, Hittesh

jmalmsten-panopto commented 7 years ago

Hi,

I went ahead and implemented this fix into our future release, keep an eye out for it in the near future.