Skip to content

CMSPluginBase.render() misses "instance" argument #272

Closed
ulope opened this Issue Nov 26, 2009 · 1 comment

2 participants

@ulope
ulope commented Nov 26, 2009

The default implementation of render() in CMSPluginBase has the signature

def render(self, context, placeholder):

while it should have:

def render(self, context, instance, placeholder):

This prevents the explainatory NotImplementedError from being raised, instead a (for beginners I assume confusing) TypeError is shown.

@digi604
Divio AG member
digi604 commented Nov 30, 2009

closed by d6f6ded signature of render function was not correct in pluginbase

@christianbertschy christianbertschy pushed a commit to christianbertschy/django-cms-2.0 that referenced this issue Oct 1, 2012
@digi604 digi604 fixes #272 signature of render function was not correct in pluginbase d6f6ded
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.