elearning-bs23/moodle-atto_brightcove

Moodle Atto Brightcove video plugin

moodle-atto_brightcove

This plugin will help you to insert the Brightcove video in moodle atto editor.

Dependency

• Moodle-media_bcplayer to preview the video

Instalation

Installing directly from the Moodle plugins directory

• Login as an admin and go to Site administration > Plugins > Install plugins. (If you can't find this - location, then plugin installation is prevented on your site.)
• Click the button 'Install plugins from Moodle plugins directory'.
• Select your current Moodle version, then search for a plugin with an Install button, click the Install - button, then click Continue.
• Confirm the installation request
• Check the plugin validation report

Installing via uploaded ZIP file

• Go to the Moodle plugins directory, select your current Moodle version, then download plugin with a Download button and download the ZIP file.
• go to [moodledirectory]/lib/editor/atto/plugins/
• Create new folder named brightcove
• extract zip content in it`

Instalation via Github

• cd [moodledirectory]/lib/editor/atto/plugins

• clone plugin from github git clone https://github.com/eLearning-BS23/moodle-atto_brightcove.git brightcove

• Go to your moodle website from browser

• Confirm the installation request

• Check the plugin validation report

Install Dependency

• After install this plugin you have to install Brightcove player plugin

usages

• go to atto editor
• you can see brightcove logo in toolbar
• click on the logo
• Pop-up will be shown
• insert your account id, video id and optional player id
• done!

Settings

• Go to site adminishtration > plugins overview > Brightcove video
• Go to settings
• you can define the brightcove account id/ player id for default use

moodle-atto_brightcove's Issues

hard-coded default account id

https://github.com/eLearning-BS23/moodle-atto_brightcove/blob/master/settings.php#L31

that looks incorrect to me. You should probably leave that empty, and ideally make sure that the plugin becomes disabled when it is empty.

hard-coded english language strings

https://github.com/eLearning-BS23/moodle-atto_brightcove/blob/master/yui/src/button/js/button.js#L44

Note: correct use of language strings is a blocker for approval in the plugins database.

Merge Moodle Media Bc player with Atto Brightcove.

This plugin (media_bcplayer) doesn't appear to actually be a Moodle media plugin at all - all the plugin appears to be doing is overriding the header of all pages and injecting some custom JS into the header of all pages. It still contains the weird observer.php class (which I don't think is actually doing anything so should just be deleted.)

It seems to me that you could build this all into the atto_brightcove plugin by adding a before_standard_html_head or before_footer callback to the atto_brightcove lib.php file.

for now as the observer class is still in place, and the fact this plugin doesn't actually appear to be implemented as a media player plugin in moodle I'm going to flag it as requiring more work.

The good news is that I'm happy with the atto_brightcove plugin code at the moment, so if you move the brightcove JS from this media plugin into the atto_brightcove plugin this might be ready to go without needing a lot more time to review.

thanks for your patience with the review process!