Prevent KeyError when page_template not supplied

[jnns]

When page_template is not supplied as a kwarg via .as_View() a KeyError is raised because .pop() is used on a dictionary with no default value specified.

[jnns]

Indeed, page_template is passed to get_context_data(). But what if one overwrites the GenericView's get() method? Additionally, the following condition would be useless otherwise because page_template could never be None:

   if page_template is None:
            if hasattr(queryset, 'model'):
                page_template = self.get_page_template(**kwargs)
            else:
                raise ImproperlyConfigured(
                    'AjaxListView requires a page_template')

[frankban]

It seems to me that that condition still makes sense, because page_template can be passed and can be None (i.e. when self.page_template is None). However, I see you point re overriding get. I'd like to merge your change, but It's still not clear to me why we should do the same for the queryset. The corresponding Django generic view uses the same logic to populate the context with the queryset, essentially """queryset = kwargs.pop('object_list')""". Could you please explain why do you think that's required or, if I convinced you, change the merge proposal accordingly? After that, I'll merge it and also add a test for the page_template issue.

[jnns]

You're right. It doesn't make sense for queryset so I edited the commit to only add the default None for page_template.