bugfix: ModelTableOptions() should call TableOptions.__init__

adammck · miracle2k · commit 41f8171e10f3 · 2010-04-21T09:15:55.000+02:00
any options specified in the Meta of a ModelTable were not being
passed along to the constructor of TableOptions. this wasn't noticed
before, but now that order_by is an option of TableBase, it became
obvious.
diff --git a/django_tables/models.py b/django_tables/models.py
@@ -9,7 +9,7 @@
 
 class ModelTableOptions(TableOptions):
     def __init__(self, options=None):
-        super(ModelTableOptions, self).__init__()
+        super(ModelTableOptions, self).__init__(options)
         self.model = getattr(options, 'model', None)
         self.columns = getattr(options, 'columns', None)
         self.exclude = getattr(options, 'exclude', None)
diff --git a/tests/test_models.py b/tests/test_models.py
@@ -212,6 +212,9 @@ class Meta:
             model = Country
             order_by = '-name'
 
+    # the order_by option is provided by TableOptions
+    assert_equal('-name', SortedCountryTable()._meta.order_by)
+
     # the default order can be inherited from the table
     assert_equal(('-name',), SortedCountryTable().order_by)
     assert_equal(4, SortedCountryTable().rows[0]['id'])
