Refactored product app to provide a ProductApplication which contains all the url patterns, this makes it possible to extend its functionality without requiring the import_modules trick. Using Django's existing get_model method where needed.

oscar/apps/product/__init__.py

@@ -0,0 +1,21 @@
+from django.conf.urls.defaults import patterns, url, include        
+from oscar.core.application import Application
+from oscar.apps.product.views import ItemDetailView, ItemClassListView, ProductListView
+
+
+class ProductApplication(Application):
+    name = 'products'
+    detail_view = ItemDetailView
+    class_list_view = ItemClassListView
+    list_view = ProductListView
+
+    def get_urls(self):
+        urlpatterns = patterns('',
+            url(r'^$', self.list_view.as_view(), name='list'),
+            url(r'^(?P<item_class_slug>[\w-]+)/$', self.class_list_view.as_view(), name='class-list'),            
+            url(r'^(?P<item_class_slug>[\w-]+)/(?P<item_slug>[\w-]*)-(?P<item_id>\d+)/$', self.detail_view.as_view(), name='detail'),
+        )
+        return urlpatterns
+
+
+application = ProductApplication()

oscar/apps/product/abstract_models.py

@@ -38,8 +38,9 @@ class AbstractItemClass(models.Model):
             self.slug= slugify(self.name)
         super(AbstractItemClass, self).save(*args, **kwargs)
 
+    @models.permalink
     def get_absolute_url(self):
-        return reverse('oscar-product-item-class', kwargs={'item_class_slug': self.slug})
+        return ('products:class-list', (), {'item_class_slug': self.slug})
 
     def __unicode__(self):
         return self.name
@@ -185,7 +186,7 @@ class AbstractItem(models.Model):
     @models.permalink
     def get_absolute_url(self):
         u"""Return a product's absolute url"""
-        return ('oscar-product-item', (), {
+        return ('products:detail', (), {
             'item_class_slug': self.get_item_class().slug, 
             'item_slug': self.slug,
             'item_id': self.id})

oscar/apps/product/urls.py

@@ -1,11 +0,0 @@
-from django.conf.urls.defaults import *
-
-from oscar.core.decorators import class_based_view
-from oscar.core.loading import import_module
-import_module('product.views', ['ItemDetailView', 'ProductListView', 'ItemClassListView'], locals())
-
-urlpatterns = patterns('oscar.product.views',
-    url(r'(?P<item_class_slug>[\w-]+)/(?P<item_slug>[\w-]*)-(?P<item_id>\d+)/$', ItemDetailView.as_view(), name='oscar-product-item'),
-    url(r'(?P<item_class_slug>[\w-]+)/$', ItemClassListView.as_view(), name='oscar-product-item-class'),
-    url(r'^$', ProductListView.as_view(), name='oscar-products'),
-)

oscar/apps/product/views.py

@@ -9,11 +9,14 @@ from django.template.response import TemplateResponse
 
 from oscar.core.loading import import_module
 
-product_models = import_module('product.models', ['Item', 'ItemClass'])
 product_signals = import_module('product.signals', ['product_viewed', 'product_search'])
 basket_forms = import_module('basket.forms', ['FormFactory'])
 history_helpers = import_module('customer.history_helpers', ['receive_product_view'])
 
+from django.db.models import get_model
+
+item_model = get_model('product','item')
+item_class_model = get_model('product', 'itemclass')
 
 class ItemDetailView(DetailView):
     u"""View a single product."""
@@ -41,7 +44,7 @@ class ItemDetailView(DetailView):
         
         We cache the object as this method gets called twice."""
         if not self._item:
-            self._item = get_object_or_404(product_models.Item, pk=self.kwargs['item_id'])
+            self._item = get_object_or_404(item_model, pk=self.kwargs['item_id'])
         return self._item
     
     def get_context_data(self, **kwargs):
@@ -60,8 +63,8 @@ class ItemClassListView(ListView):
     paginate_by = 20
 
     def get_queryset(self):
-        item_class = get_object_or_404(product_models.ItemClass, slug=self.kwargs['item_class_slug'])
-        return product_models.Item.browsable.filter(item_class=item_class)
+        item_class = get_object_or_404(item_class_model, slug=self.kwargs['item_class_slug'])
+        return item_model.browsable.filter(item_class=item_class)
 
 
 class ProductListView(ListView):
@@ -84,9 +87,9 @@ class ProductListView(ListView):
             # Send signal to record the view of this product
             product_signals.product_search.send(sender=self, query=q, user=self.request.user)
             
-            return product_models.Item.browsable.filter(title__icontains=q)
+            return item_model.browsable.filter(title__icontains=q)
         else:
-            return product_models.Item.browsable.all()
+            return item_model.browsable.all()
         
     def get_context_data(self, **kwargs):
         context = super(ProductListView, self).get_context_data(**kwargs)

oscar/core/application.py

@@ -0,0 +1,15 @@
+class Application(object):
+    name = None
+    
+    def __init__(self, app_name=None):
+        self.app_name = app_name
+    
+    def get_urls(self):
+        u"""
+        Return the url patterns for this app, MUST be implemented in the subclass
+        """
+        pass
+    
+    @property
+    def urls(self):
+        return self.get_urls(), self.app_name, self.name

oscar/urls.py

@@ -1,8 +1,10 @@
 from django.conf.urls.defaults import *
 from django.conf import settings
 
+from oscar.apps.product import application as products_app
+
 urlpatterns = patterns('',
-    (r'product/', include('oscar.apps.product.urls')),
+    (r'products/', include(products_app.urls)),
     (r'basket/', include('oscar.apps.basket.urls')),
     (r'checkout/', include('oscar.apps.checkout.urls')),
     (r'order-management/', include('oscar.apps.order_management.urls')),