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, so looks promise.

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

Unfortunelly my apps is not django 2.0 ready because of GIS usage, so I can't test your commit.

[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