[patch] Make generic views work with magic-removal models

[Esaj]

AttributeError: 'module' object has no attribute 'get_module'

Exception Location: /usr/lib/python2.3/site-packages/django/views/generic/list_detail.py in object_list, line 37

models.get_module(app, module_name) appears to be commented out in django/db/models/__init__.py

[Esaj]

Looks like the generic views need to be modified to use a model arg for specifying which model class to use. Presumably app_label and module_name need to be kept for selecting the template using <app_label>/<module_name>.

[Adrian Holovaty]

Hmmm, we should remove the <app_label>/<module_name> thing as well, because there's no longer a concept of "module_name" in magic-removal. What do you think would be a good template-directory naming convention?

[Esaj]

We could keep app_label and just use the lowercase version of the model class name instead of module_name. In case of namespace clashes, perhaps we should have an optional template_prefix arg which would override the lowercase classname and give us <app_label>/<template_prefix>_xxx.

An alternative would be to try and use the full path of the model class name (replacing . with /) but that would get a bit long and unwieldy, although it might reduce the chance of namespace clashes.

[Esaj]

Use 'model' arg instead of 'module_name' for generic views

[Adrian Holovaty]

Fixed in [1933].