BaseGeometryWidget.subwidgets() crashes with KeyError 'widget'

[Dariusz Paluch]

BaseGeometryWidget.get_context() doesn't execute super().get_context(self, name, value, attrs) so key 'widget' used by method subwidgets() doesn't exist.

[Tim Graham]

Can you provide a test case to reproduce the issue?

[Dariusz Paluch]

Replying to Tim Graham:

Can you provide a test case to reproduce the issue?

Just use any BaseGeometryWidget based form field in form.

[Tim Graham]

Here's a snippet to reproduce:

>>> from django.contrib.gis.forms import OpenLayersWidget
>>> widget = forms.OpenLayersWidget()
>>> list(widget.subwidgets('name', 'value'))
Traceback (most recent call last):
...
  File "django/django/forms/widgets.py", line 177, in subwidgets
    yield context['widget']
KeyError: 'widget

[Tim Graham]

Dariusz, could you your application with this ​PR. I'm not certain that it's correct.

[dpaluch-rp]

Replying to Tim Graham:

Dariusz, could you your application with this ​PR. I'm not certain that it's correct.

I'm fixed my application the same way.

`class GoogleMapWidget(forms.BaseGeometryWidget):

map_width = 800
map_height = 500
template_name = 'gis/google.html'

def serialize(self, value):

return value.geojson if value else

def get_context(self, name, value, attrs):

# TODO: Bug Django 1.11, BaseGeometryWidget.get_context don't call super(), so context don't have "widget" key
context = super().get_context(name, value, attrs)
super_context = super(forms.BaseGeometryWidget, self).get_context(name, value, attrs)
super_context.update(context)
return super_context

`

[GitHub <noreply@…>]

In d2cb7a2b:

Fixed #28039 -- Fixed crash in BaseGeometryWidget.subwidgets().

[Tim Graham <timograham@…>]

In 5e2bbcd7:

[1.11.x] Fixed #28039 -- Fixed crash in BaseGeometryWidget.subwidgets().

Backport of d2cb7a2bc11f111be04a29b5e4f92a183b18ba88 from master