Shouldn't it inherit from EffectBase?