MoodleMobile Text Messages

Messaging outputs ::: message_pageone
Maintained by Me !Tim Williams, ian hatton
The MoodleMobille messaging outputs plugin allows PageOne/JanetTXT subscribers to send standard moodle notifications via text message.
Latest release:
16 sites
21 downloads
5 fans
Current versions available: 2
The MoodleMobile messaging plug-in allows you to send allows standard moodle notifications to be sent via text message. With no ongoing monthly cost and only a small set-up charge to enable your Janet txt account, MoodleMobile is a great way to integrate text messaging into your institution. Simply contact PageOne to enable your account and then download the MoodleMobile plug-in.

Screenshots

Screenshot #0

Contributors

Me !
Tim Williams (Lead maintainer)
Please login to view contributors details and/or to contact them

Comments RSS

Comments

  • Thu, Oct 22, 2015, 3:51 PM
    Hi there,

    Thanks for sharing your code. Please provide a link to the public source code repository and bug tracker related to your plugin.

    Also - the name you have used here does not seem correct - "MoodleMobile" - this could cause significant confusion with the Mobile app provided by Moodle. I would suggest renaming it to "PageOne Text messaging" or something similar.

    A quick look at your code shows that it does not follow our guidelines very well - please install the local_codechecker and local_moodlecheck plugins and use these to validate the structure of your code. I also see some backticks within in-line SQL which are not cross-db compatible.

    once you have tidied these things up please submit a new version for us to look at - thanks!
  • Fri, Oct 23, 2015, 12:03 AM
  • Fri, Nov 27, 2015, 4:39 PM

    Thanks for an updated version. Looking better now.

    When configuring the plugin, I can see several PHP warnings raised. Please make sure you are developing your plugin with developer debugging enabled and displayed.

    Warning: array_key_exists(): The first argument should be either a string or an integer in .../lib/adminlib.php on line 3068
    

    I believe this is caused by a bug in your settings.php file where in admin_setting_configselect() the options are correct integers (0 or 1) but you set the default value to 'false' (instead of 0). As a result, the warning is raised and no information about the default value is displayed.

    I agree with Dan that the name could be less confusing, but I understand your reasoning and background sent via e-mail. You are cleared to land now, welcome to the Plugins directory.

Please login to post comments