Skip to content

Jason Platts
jason-platts

Organizations

@moodleou @JISC-DOULS
@jason-platts
OU Blogs creates grade entry
@jason-platts

I've now fixed this and it should be upstream next time we push. Have attached a zip of the patch for this for use until then. blogfix56.zip

@jason-platts

Thanks for reporting this. I see the same thing myself so will investigate.

@jason-platts
Participation summary on user outline report not working/inconsistent with complete report
@jason-platts

Fixed - thanks for your patch.

@jason-platts
Fix Individual Activity Reports summaries
jason-platts commented on pull request moodleou/moodle-mod_ouwiki#44
@jason-platts

Many thanks for this. I think your changes look ok. They certainly seem to work for me, though we don't use these reports so I'm not 100% sure that…

@jason-platts
externallib.php: fix invalid OPTIONAL parameter for external function params
jason-platts commented on pull request moodleou/moodle-mod_oublog#55
@jason-platts

Thanks for this. I've integrated and will be upstream next time we push.

@jason-platts
Replace 'continue' outside of loop
jason-platts commented on pull request moodleou/moodle-mod_ouwiki#45
@jason-platts

Thanks for spotting this. I've integrated this patch and it will be upstream next time we push.

@jason-platts
Fix deprecation warning in Moodle 2.9
jason-platts commented on pull request moodleou/moodle-mod_ouwiki#40
@jason-platts

Thanks for the fix. This should be alright on master & 2.9 branch now as we fixed this (though I see the fix was not as good as your patch!).

@jason-platts
Plugin forumngfeature_copy does not declare valid $plugin->component in its version.php.
@jason-platts

Fixed by #56 : Will be upstream in the future, till then you can use the patch.

@jason-platts
Updated version.php for Moodle 3.0
jason-platts commented on pull request moodleou/moodle-mod_forumng#56
@jason-platts

Thanks. I finally got round to adding you patch so it will be upstream next time we push.

@jason-platts
Bit of an Odd One
@jason-platts

Thanks! Sorry, hadn't spotted this breaks unit tests as hadn't realised there was a core test that tests all the external lib files. I'll use your …

jason-platts commented on pull request moodleou/moodle-mod_ouwiki#44
@jason-platts

Excellent, thanks lots for this. I'll try and take a look as it looks much better than what we had previously. One minor (picky) point: could you r…

@jason-platts

A change to make digestmailheader string into two strings, one for html and one for plain text would be the easiest way to deal with this. But then…

@jason-platts
Fixit #2
jason-platts commented on pull request moodleou/moodle-mod_forumng#58
@jason-platts

Yes, thanks. I will be merging #56 to fix this issue (I don't know when, so you can use Alistair's commit till then).

@jason-platts

Sorry I totally gave up trying to fix this as I got stuck. My hack (which I seem to have lost) was amending service_class_method_body() so that $pa…

@jason-platts

Thanks. I see this as well. We hadn't spotted this as it actually works as expected in our theme (so maybe it works ok on other older themes?). As …

Something went wrong with that request. Please try again.