Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dj21 render() has a new param: renderer #235

Merged
merged 6 commits into from
Aug 22, 2018

Conversation

zvolsky
Copy link
Contributor

@zvolsky zvolsky commented Jul 20, 2018

Without this under dj21rc1:

Traceback (most recent call last):
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/core/handlers/exception.py", line 34, in inner
response = get_response(request)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/core/handlers/base.py", line 156, in _get_response
response = self.process_exception_by_middleware(e, request)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/core/handlers/base.py", line 154, in _get_response
response = response.render()
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/response.py", line 106, in render
self.content = self.rendered_content
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/response.py", line 83, in rendered_content
content = template.render(context, self._request)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/backends/django.py", line 61, in render
return self.template.render(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/base.py", line 171, in render
return self._render(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/base.py", line 163, in _render
return self.nodelist.render(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/base.py", line 937, in render
bit = node.render_annotated(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/base.py", line 904, in render_annotated
return self.render(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/loader_tags.py", line 150, in render
return compiled_parent._render(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/base.py", line 163, in _render
return self.nodelist.render(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/base.py", line 937, in render
bit = node.render_annotated(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/base.py", line 904, in render_annotated
return self.render(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/loader_tags.py", line 150, in render
return compiled_parent._render(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/base.py", line 163, in _render
return self.nodelist.render(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/base.py", line 937, in render
bit = node.render_annotated(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/base.py", line 904, in render_annotated
return self.render(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/loader_tags.py", line 150, in render
return compiled_parent._render(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/base.py", line 163, in _render
return self.nodelist.render(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/base.py", line 937, in render
bit = node.render_annotated(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/base.py", line 904, in render_annotated
return self.render(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/loader_tags.py", line 62, in render
result = block.nodelist.render(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/base.py", line 937, in render
bit = node.render_annotated(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/base.py", line 904, in render_annotated
return self.render(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/loader_tags.py", line 62, in render
result = block.nodelist.render(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/base.py", line 937, in render
bit = node.render_annotated(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/base.py", line 904, in render_annotated
return self.render(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/defaulttags.py", line 209, in render
nodelist.append(node.render_annotated(context))
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/base.py", line 904, in render_annotated
return self.render(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/loader_tags.py", line 188, in render
return template.render(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/base.py", line 173, in render
return self._render(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/base.py", line 163, in _render
return self.nodelist.render(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/base.py", line 937, in render
bit = node.render_annotated(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/base.py", line 904, in render_annotated
return self.render(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/defaulttags.py", line 209, in render
nodelist.append(node.render_annotated(context))
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/base.py", line 904, in render_annotated
return self.render(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/defaulttags.py", line 209, in render
nodelist.append(node.render_annotated(context))
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/base.py", line 904, in render_annotated
return self.render(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/defaulttags.py", line 309, in render
return nodelist.render(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/base.py", line 937, in render
bit = node.render_annotated(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/base.py", line 904, in render_annotated
return self.render(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/defaulttags.py", line 309, in render
return nodelist.render(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/base.py", line 937, in render
bit = node.render_annotated(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/base.py", line 904, in render_annotated
return self.render(context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/base.py", line 993, in render
return render_value_in_context(output, context)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/template/base.py", line 972, in render_value_in_context
value = str(value)
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/utils/html.py", line 397, in
klass.str = lambda self: mark_safe(klass_str(self))
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/forms/boundfield.py", line 33, in str
return self.as_widget()
File "/home/vagrant/venv37/lib/python3.7/site-packages/django/forms/boundfield.py", line 93, in as_widget
renderer=self.form.renderer,
TypeError: render() got an unexpected keyword argument 'renderer'

@crucialfelix
Copy link
Owner

Thanks.

Copy link
Owner

@crucialfelix crucialfelix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah I was wondering how that worked with older versions too. I need to check if the tests cover that.

@nortigo
Copy link

nortigo commented Aug 6, 2018

Hi, do you have idea when this fix will be released ? Thank you :)

@crucialfelix
Copy link
Owner

crucialfelix commented Aug 6, 2018 via email

@nortigo
Copy link

nortigo commented Aug 6, 2018

No problem, thank you for the quick follow up. Enjoy your holidays!

@nortigo
Copy link

nortigo commented Aug 14, 2018

Maybe I am not good with pip but it seems that @zvolsky's branch is not working.

Here is the command I executed:

pip install -e git+git://github.com/zvolsky/django-ajax-selects.git@zvolsky_fix_renderer#egg=django-ajax-selects

Pip freeze gives me this output:

-e git://github.com/zvolsky/django-ajax-selects.git@a677609287f974a4ea672c069d613e1270500d62#egg=django_ajax_selects

@crucialfelix
Copy link
Owner

crucialfelix commented Aug 14, 2018 via email

@nortigo
Copy link

nortigo commented Aug 14, 2018

It gives me the same error as previously mentioned. That's why I published the command I executed because I don't understand why it still not working. Maybe @zvolsky forgot some places? I'm at work right now but as soon as I'm back home, I'll post the exact error message I got.

@crucialfelix
Copy link
Owner

Tomorrow morning I'll get this solved and released. Big client work today. Thanks for your patience.

@nortigo
Copy link

nortigo commented Aug 20, 2018

Thanks a lot for your support.

@crucialfelix crucialfelix merged commit 1b362f0 into crucialfelix:develop Aug 22, 2018
@crucialfelix
Copy link
Owner

Thanks very much for the PR !

The tests now run against all versions and they do check all the render() of fields. Doing the release shortly.

@crucialfelix
Copy link
Owner

@jonathanmorgan
Copy link

Working great for me with django 2.1. Thank you!

@nortigo
Copy link

nortigo commented Aug 22, 2018

@crucialfelix Thank you for the release but it still does not work for me.
I deleted my virtual env, removed the pip's cache, created a new virtual env and installed the package.

Here is the stacktrace:

Traceback (most recent call last):
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/core/handlers/exception.py", line 34, in inner
    response = get_response(request)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/core/handlers/base.py", line 156, in _get_response
    response = self.process_exception_by_middleware(e, request)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/core/handlers/base.py", line 154, in _get_response
    response = response.render()
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/response.py", line 106, in render
    self.content = self.rendered_content
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/response.py", line 83, in rendered_content
    content = template.render(context, self._request)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/backends/django.py", line 61, in render
    return self.template.render(context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/base.py", line 171, in render
    return self._render(context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/base.py", line 163, in _render
    return self.nodelist.render(context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/base.py", line 937, in render
    bit = node.render_annotated(context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/base.py", line 904, in render_annotated
    return self.render(context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/loader_tags.py", line 150, in render
    return compiled_parent._render(context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/base.py", line 163, in _render
    return self.nodelist.render(context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/base.py", line 937, in render
    bit = node.render_annotated(context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/base.py", line 904, in render_annotated
    return self.render(context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/loader_tags.py", line 150, in render
    return compiled_parent._render(context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/base.py", line 163, in _render
    return self.nodelist.render(context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/base.py", line 937, in render
    bit = node.render_annotated(context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/base.py", line 904, in render_annotated
    return self.render(context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/loader_tags.py", line 62, in render
    result = block.nodelist.render(context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/base.py", line 937, in render
    bit = node.render_annotated(context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/base.py", line 904, in render_annotated
    return self.render(context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/loader_tags.py", line 62, in render
    result = block.nodelist.render(context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/base.py", line 937, in render
    bit = node.render_annotated(context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/base.py", line 904, in render_annotated
    return self.render(context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/defaulttags.py", line 209, in render
    nodelist.append(node.render_annotated(context))
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/base.py", line 904, in render_annotated
    return self.render(context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/loader_tags.py", line 188, in render
    return template.render(context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/base.py", line 173, in render
    return self._render(context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/base.py", line 163, in _render
    return self.nodelist.render(context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/base.py", line 937, in render
    bit = node.render_annotated(context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/base.py", line 904, in render_annotated
    return self.render(context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/defaulttags.py", line 209, in render
    nodelist.append(node.render_annotated(context))
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/base.py", line 904, in render_annotated
    return self.render(context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/defaulttags.py", line 209, in render
    nodelist.append(node.render_annotated(context))
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/base.py", line 904, in render_annotated
    return self.render(context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/defaulttags.py", line 309, in render
    return nodelist.render(context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/base.py", line 937, in render
    bit = node.render_annotated(context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/base.py", line 904, in render_annotated
    return self.render(context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/defaulttags.py", line 309, in render
    return nodelist.render(context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/base.py", line 937, in render
    bit = node.render_annotated(context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/base.py", line 904, in render_annotated
    return self.render(context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/base.py", line 993, in render
    return render_value_in_context(output, context)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/template/base.py", line 972, in render_value_in_context
    value = str(value)
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/utils/html.py", line 397, in <lambda>
    klass.__str__ = lambda self: mark_safe(klass_str(self))
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/forms/boundfield.py", line 33, in __str__
    return self.as_widget()
  File "/home/frujo/projects/venvs/oscar/lib/python3.6/site-packages/django/forms/boundfield.py", line 93, in as_widget
    renderer=self.form.renderer,
TypeError: render() got an unexpected keyword argument 'renderer'

Here is the portion of the code that I use for the Ajax Select:

form = make_ajax_form(BusinessLocation, {
    'city': 'cities',
    'postal_code': 'postal_codes'
})

The Lookups:

@register('cities')
class CityLookup(LookupChannel):
    model = City

    def get_query(self, q, request):
        return self.model.objects.filter(name__istartswith=q)


@register('postal_codes')
class PostalCodeLookup(LookupChannel):
    model = PostalCode

    def get_query(self, q, request):
        return self.model.objects.filter(postal_code__istartswith=q)

@nortigo
Copy link

nortigo commented Aug 22, 2018

@crucialfelix Nevermind. I found the problem. I was using another plugin in my form on another field. Everything's working perfectly! Thank you again for your support and the release.

@crucialfelix
Copy link
Owner

Damn... Oh ok... Great ! Glad to hear it !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants