Merge pull request #1001 from ecds/bugfix/945-page-number

Don't add 1 to canvas position in search results (#945)

apps/ingest/models.py

@@ -452,7 +452,7 @@ def open_metadata(self):
 
     def create_canvases(self):
         # TODO: What if there are multiple sequences? Is that even allowed in IIIF?
-        for position, sc_canvas in enumerate(self.remote_manifest['sequences'][0]['canvases']):
+        for position, sc_canvas in enumerate(self.remote_manifest['sequences'][0]['canvases'], start=1):
             canvas_metadata = None
             # TODO: we will need some sort of check for IIIF API version, but not
             # everyone includes a context for each canvas.

apps/templates/snippets/volume_result.html

@@ -78,7 +78,7 @@ <h4>
             {% for canvas in volume.meta.inner_hits|group_by_canvas %}
                 <dd class="result-page">
                     <a href="{% url 'page' volume=volume.pid page=canvas.pid %}">
-                        <span class="page-number">p. {{ canvas.position|add:1 }}</span>
+                        <span class="page-number">p. {{ canvas.position }}</span>
                         {% if canvas.highlights|length %}
                             <ul class="highlights">
                             {% for fragment in canvas.highlights %}