Fixed a bug when not specifying a limit when using the more_like_this template tag. Thanks to symroe for the original patch.

Author: toastdriven

AUTHORS

@@ -41,3 +41,4 @@ Thanks to
     * piquadrat for a patch regarding the more_like_this template tag.
     * dedsm for a patch regarding the pickling of SearchResult objects.
     * EmilStenstrom for a patch to the Highlighter.
+    * symroe for a patch regarding the more_like_this template tag.

haystack/templatetags/more_like_this.py

@@ -11,7 +11,10 @@ def __init__(self, model, varname, for_types=None, limit=None):
         self.model = template.Variable(model)
         self.varname = varname
         self.for_types = for_types
-        self.limit = int(limit)
+        self.limit = limit
+        
+        if not self.limit is None:
+            self.limit = int(self.limit)
 
     def render(self, context):
         try:

tests/solr_tests/tests/templatetags.py

@@ -53,3 +53,11 @@ def test_more_like_this_with_limit(self):
             'entry': mock,
         }
         self.assertEqual(self.render(template, context), u'core.mockmodel.2 core.mockmodel.18 core.mockmodel.23 core.mockmodel.15 core.mockmodel.21 ')
+    
+    def test_more_like_this_without_limit(self):
+        mock = MockModel.objects.get(pk=3)
+        template = """{% load more_like_this %}{% more_like_this entry as related_content %}{% for rc in related_content %}{{ rc.id }} {% endfor %}"""
+        context = {
+            'entry': mock,
+        }
+        self.assertEqual(self.render(template, context), u'core.mockmodel.2 core.mockmodel.18 core.mockmodel.23 core.mockmodel.15 core.mockmodel.21 core.mockmodel.13 core.mockmodel.17 core.mockmodel.16 core.mockmodel.20 core.mockmodel.1 core.mockmodel.22 core.mockmodel.19 core.mockmodel.8 core.mockmodel.6 core.mockmodel.4 core.mockmodel.11 core.mockmodel.14 core.mockmodel.12 core.mockmodel.9 core.mockmodel.7 core.mockmodel.10 core.mockmodel.5 ')