search

kaltura / all-in-one-video-pack.wordpress

A Wordpress Plugin to simplify adding Kaltura to your Blog
https://wordpress.org/plugins/all-in-one-video-pack/
19 stars 31 forks source link

first pass at removing the sidebar widget #4

Closed jondavidjohn closed 9 years ago

jondavidjohn commented 11 years ago

This attempts to remove references from the sidebar widget plugin, that attempts to directly access/manipulate the database.

We suggest this functionality get moved into a seperate "add-on" plugin that is wrapped in the following to make sure the main plugin is loaded/activated


if (class_exists('Kaltura_AllInOneVideoPackPlugin')) {

    // Load up sidebar widget plugin

}

This way we won't have to maintain a seperate WordPress VIP branch and the sidebar widget would be optional with a note about why.

kaltura-hooks commented 9 years ago

Hi @jondavidjohn, Thank you for contributing this pull request! Please sign the Kaltura CLA so we can review and merge your contribution. Learn more at http://bit.ly/KalturaContrib

rkreich commented 9 years ago

This issue was already handled under the postmedia branch