Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Feature: Extend plugin framework #368

Closed
echiu64 opened this Issue Sep 13, 2011 · 5 comments

Comments

Projects
None yet
2 participants
Contributor

echiu64 commented Sep 13, 2011

  1. Create base Plugin class for all plugins to extend
  2. Base class will have methods:
    def isReloadable:
        return True/False
    def isStoppable:
        return True/False
    def start:
        hook('plugin.x.start')
        # start plugin
    def stop:
        hook('plugin.x.stop')
        # stop plugin
    def reload:
        hook('plugin.x.reload')
        # reload plugin configuration

Hooks are added in so that any dependent plugin/system can be notified when a plugin starts/stops.

Contributor

echiu64 commented Sep 13, 2011

Delagger and lpbot are candidates to be moved into plugins

Owner

c00w commented Sep 13, 2011

I like moving lpbot to be a plugin. Moving the delagger to be a plugin sounds like a bad idea since someone might disable it.

Owner

c00w commented Sep 14, 2011

lpbot is moved.

Contributor

echiu64 commented Sep 14, 2011

That was quick!

Owner

c00w commented Apr 9, 2012

I'm actually getting rid of plugins. Not a lot of people wrote them and they tended to be much to complicated. Plus the callback hooks were quite hard to debug.

@c00w c00w closed this Apr 9, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment