Reports: Watershed Report Link

report_watershed
Maintained by Picture of Andrew Downes Andrew Downes
Watershed integration plugin for Moodle.
12 sites
31 downloads
4 fans

This plugin provides an integration between Watershed LRS and MoodleThis plugin requires moodle-mod_tincanlaunch 1.3RC1 or higher to be installed.

You can find out more about Watershed at https://wastershedlrs.com and full documentation is maintained on the Watershed help site.


Tags: xAPI, Experience API, Tin Can API, Learning Evaluation, Analytics, Reports


Potential privacy issues

Sends some user information relating to persona association to Watershed LRS. This enables Watershed to associate a user's email address to their Moodle account id if it has data about both.

Screenshots

Screenshot #0

Contributors

Picture of Andrew Downes
Andrew Downes (Lead maintainer)
Please login to view contributors details and/or to contact them

Comments RSS

Show comments
  • Picture of Dan Marsden
    Mon, 13 Jun 2016, 7:05 AM
    Hi Andrew,

    The plugin seems to allow Moodle's full session id to be disclosed - passed in a $_GET call and sent to the external service? - that looks pretty nasty to me. I'd also suggest renaming your config.php file to config-dist.php to help avoid people from adding their credentials into their local git repo and then accidentally sharing the repo (for example in github) I've reported those issues into your github tracker - It would also be good to run the moodle coding guideline tools over the code - Have you seen the travis-ci integration to help run these automatically? - see (https://moodle.org/mod/forum/discuss.php?d=323384)
  • Picture of Dan Marsden
    Mon, 13 Jun 2016, 7:06 AM
    it would also be good to add some more detail in the plugin description here and some screenshots of the reports that might be available.
  • Picture of Andrew Downes
    Tue, 21 Jun 2016, 7:11 PM
    These issues have now been resolved on the Github. The only thing is that the plugin makes use for PHP's error_log function which the code checker does not like for some reason.

    What's the next step? Should I push a new release here, or wait for you to review again first?
  • Picture of David Mudrák
    Tue, 21 Jun 2016, 8:19 PM

    Add a new version here and then use the "Request re-approval" button.

  • Picture of David Mudrák
    Wed, 10 Aug 2016, 7:54 PM

    Thanks Andrew for sharing another plugin with the community, and thanks Dan for the code peer-review. While attempting to install the report, I ran into a fatal error due to missing dependencies and reported the issue as #5 in your tracker. With the hope that issue will be fixed asap, I am going to approve the plugin now. You are cleared to land, welcome to the Plugins directory.

  • Picture of dave fay
    Fri, 30 Sep 2016, 5:14 AM
    Debug info:
    Error code: codingerror
    Stack trace:

    line 289 of /lib/ddl/sql_generator.php: coding_exception thrown
    line 221 of /lib/ddl/mysql_sql_generator.php: call to sql_generator->getCreateTableSQL()
    line 250 of /lib/ddl/sql_generator.php: call to mysql_sql_generator->getCreateTableSQL()
    line 415 of /lib/ddl/database_manager.php: call to sql_generator->getCreateStructureSQL()
    line 370 of /lib/ddl/database_manager.php: call to database_manager->install_from_xmldb_structure()
    line 514 of /lib/upgradelib.php: call to database_manager->install_from_xmldb_file()
    line 1742 of /lib/upgradelib.php: call to upgrade_plugins()
    line 683 of /admin/index.php: call to upgrade_noncore()

    that was what i saw when installed on moddle3.1
  • Picture of Andrew Downes
    Fri, 30 Sep 2016, 6:47 PM
    Hi Dave,

    I have replicated and fixed the error and pushed a new release. Can you download that new release and try again?

    Thanks for giving me the full stack trace, though ironically the useful bit of error message was the text immediately above what you copied smile

    Note that this plugin only works with Watershed LRS. Do you have a Watershed account that you're using this plugin with? See https://watershedlrs.com or mail andrew.downes@watershedlrs.com

    Andrew
Please login to post comments