search

smss-dev / moodle-local_artena

Moodle local plugin to support an interface with the Artena SMS
2 stars 3 forks source link

$course not defined #2

Open danmarsden opened 8 years ago

danmarsden commented 8 years ago

https://github.com/smss-dev/moodle-artena-interface/blob/master/local/artena/externallib.php#L1703

danmarsden commented 8 years ago

https://github.com/smss-dev/moodle-artena-interface/blob/master/local/artena/externallib.php#L611

danmarsden commented 8 years ago

https://github.com/smss-dev/moodle-artena-interface/blob/master/local/artena/externallib.php#L755

danmarsden commented 8 years ago

https://github.com/smss-dev/moodle-artena-interface/blob/master/local/artena/externallib.php#L1257

danmarsden commented 8 years ago

(possibly others too)

smss-dev commented 8 years ago

Merged manually, closing pull

danmarsden commented 8 years ago

did you mean to close this one? - I can't see any commits related to fixing these undefined var calls to $course.

smss-dev commented 8 years ago

did not mean to close, mixed it up with the other pull request.

copy/paste error, tidying up.

danmarsden commented 8 years ago

Thanks! - I use PHPStorm as my IDE and it has some nice syntax highlighting for stuff like this, not sure what you're using - everyone seems to have a different preference! :-)