Details
-
Type:
Improvement
-
Status:
Resolved
-
Priority:
Minor
-
Resolution: Incomplete
-
Affects Version/s: 1.8, 1.8.1, 1.8.2, 1.8.3, 1.8.4, 1.8.5, 1.8.6, 1.8.7, 1.8.8, 1.9, 1.9.1, 1.9.2, 1.9.3, 1.9.4
-
Fix Version/s: None
-
Component/s: Block: {none of the above}
-
Labels:None
-
Affected Branches:MOODLE_18_STABLE, MOODLE_19_STABLE
Description
This block allows Moodle administrators and teachers to relate a course of Moodle to a Youtube RSS channel (which can be based on Labels, Users, Subjects, Youtube Updates, etc.) and reproduce these videos in the same block.
Andreu - Thanks for sharing this block with the Moodle community and please accept my apologies for the delayed response. My summer work has kept me from getting as much accomplished for Moodle as I would like. In any case, I am starting to review your code. Please know that my comments are meant to be constructive and help make your code better or make the experience for new users of your code go more smoothly so use them in so far as they are helpful to you. I recommend testing code with Moodle debugging set to show all reasonable errors and notices (not developer mode). Here are a few notices I received when I first added the block to the front page of my test site:
Notice: Trying to get property of non-object in /home/arborrow/Moodle/code/19stable/blocks/youtube_list/block_youtube_list.php on line 27
Warning: DOMDocument::load() [domdocument.load]: Empty string supplied as input in /home/arborrow/Moodle/code/19stable/blocks/youtube_list/block_youtube_list.php on line 27
Notice: Undefined property: stdClass::$footer in /home/arborrow/Moodle/code/19stable/blocks/moodleblock.class.php on line 355
These are usually minor errors from not checking that a particular variable is defined or not initializing a variable. I'm happy to help you debug them and come up with patches if you would like. Peace - Anthony