{"id":806357,"url":"http://patchwork.ozlabs.org/api/1.0/patches/806357/?format=json","project":{"id":16,"url":"http://patchwork.ozlabs.org/api/1.0/projects/16/?format=json","name":"Patchwork","link_name":"patchwork","list_id":"patchwork.lists.ozlabs.org","list_email":"patchwork@lists.ozlabs.org","web_url":"http://jk.ozlabs.org/projects/patchwork/","scm_url":"git://github.com/getpatchwork/patchwork","webscm_url":"https://github.com/getpatchwork/patchwork"},"msgid":"<20170828041127.20127-1-dja@axtens.net>","date":"2017-08-28T04:11:27","name":"Handle EmptyPage exceptions","commit_ref":null,"pull_url":null,"state":"accepted","archived":false,"hash":"951cd0fb83d475059ded30e74b49571557312391","submitter":{"id":65792,"url":"http://patchwork.ozlabs.org/api/1.0/people/65792/?format=json","name":"Daniel Axtens","email":"dja@axtens.net"},"delegate":null,"mbox":"http://patchwork.ozlabs.org/project/patchwork/patch/20170828041127.20127-1-dja@axtens.net/mbox/","series":[{"id":65,"url":"http://patchwork.ozlabs.org/api/1.0/series/65/?format=json","date":"2017-08-28T04:11:27","name":"Handle EmptyPage exceptions","version":1,"mbox":"http://patchwork.ozlabs.org/series/65/mbox/"}],"check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/806357/checks/","tags":{},"headers":{"Return-Path":"<patchwork-bounces+incoming=patchwork.ozlabs.org@lists.ozlabs.org>","X-Original-To":["incoming@patchwork.ozlabs.org","patchwork@lists.ozlabs.org"],"Delivered-To":["patchwork-incoming@bilbo.ozlabs.org","patchwork@lists.ozlabs.org"],"Received":["from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3])\n\t(using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xgdd804W0z9sN7\n\tfor <incoming@patchwork.ozlabs.org>;\n\tMon, 28 Aug 2017 14:11:56 +1000 (AEST)","from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3])\n\tby lists.ozlabs.org (Postfix) with ESMTP id 3xgdd75cFDzDqXp\n\tfor <incoming@patchwork.ozlabs.org>;\n\tMon, 28 Aug 2017 14:11:55 +1000 (AEST)","from mail-pg0-x242.google.com (mail-pg0-x242.google.com\n\t[IPv6:2607:f8b0:400e:c05::242])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128\n\tbits)) (No client certificate requested)\n\tby lists.ozlabs.org (Postfix) with ESMTPS id 3xgdd20nQTzDqMh\n\tfor <patchwork@lists.ozlabs.org>;\n\tMon, 28 Aug 2017 14:11:50 +1000 (AEST)","by mail-pg0-x242.google.com with SMTP id r133so5427300pgr.2\n\tfor <patchwork@lists.ozlabs.org>;\n\tSun, 27 Aug 2017 21:11:50 -0700 (PDT)","from localhost.localdomain (124-171-202-56.dyn.iinet.net.au.\n\t[124.171.202.56]) by smtp.gmail.com with ESMTPSA id\n\th81sm20712453pfj.72.2017.08.27.21.11.45\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);\n\tSun, 27 Aug 2017 21:11:47 -0700 (PDT)"],"Authentication-Results":["ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=axtens.net header.i=@axtens.net\n\theader.b=\"Ds/3H4B9\"; dkim-atps=neutral","lists.ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=axtens.net header.i=@axtens.net\n\theader.b=\"Ds/3H4B9\"; dkim-atps=neutral","lists.ozlabs.org; dkim=pass (1024-bit key;\n\tunprotected) header.d=axtens.net header.i=@axtens.net\n\theader.b=\"Ds/3H4B9\"; dkim-atps=neutral"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=axtens.net; s=google;\n\th=from:to:cc:subject:date:message-id;\n\tbh=19cXohMEj9q5TT7fwsCYDtHm2CwYXXRk8laSgdZO+RU=;\n\tb=Ds/3H4B9jUTThyM0sYCyVn0ipz9gLHYplQV8MkVqyphCUo1YT3sLgdGPRnFPmNFzmR\n\trjL7ajWayvZk0LaJKmDBleAXXsggztDRxW1k83HJCLLDxKHD6BWah2CSdd85eXj5qTEM\n\tMQFUVK/PZ2r8JuqlJEwG+d9iZy0AkMkr8SR24=","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:from:to:cc:subject:date:message-id;\n\tbh=19cXohMEj9q5TT7fwsCYDtHm2CwYXXRk8laSgdZO+RU=;\n\tb=naHkRVv9TkNEdWPe1QN9NUMiAyXLbpNMUYU9ozkE26jz3UZXGQNuG389dH+dU6Uw85\n\tBlSoiMpDX7Wv/edqqisy4dlSrX391Fqlk69ivYVa6JNseQ5T8XTDce4A8fb2C32Sxf3K\n\tj+Gu6F6c+0E+C8NFeuBUTnLlBNxf8qvc8lJzg3lBtuLQpyW2Zexw1atsZyBkKELNHcqx\n\t5bwA84/YlrhmR0twp+/ybHJkjxl3BxBgWdZMO7tgVVOriR+cvDSQShQhVCyqAej3hL2i\n\tGs2Na47ipbrjfBqMPTfguXTzR29SE3iP7b4JZpfnVRWqZ43vUZDw3Qt8WZwBbYoGNuG4\n\tHNLg==","X-Gm-Message-State":"AHYfb5hx5TATAfpKKgdx21vT8t/3ztGG6oK3PG4puKbcnKyvg2aTOemn\n\tq5m3G2dyDU4ZaSMpO32nUQ==","X-Received":"by 10.84.210.236 with SMTP id a99mr7040021pli.151.1503893507942; \n\tSun, 27 Aug 2017 21:11:47 -0700 (PDT)","From":"Daniel Axtens <dja@axtens.net>","To":"patchwork@lists.ozlabs.org","Subject":"[PATCH] Handle EmptyPage exceptions","Date":"Mon, 28 Aug 2017 14:11:27 +1000","Message-Id":"<20170828041127.20127-1-dja@axtens.net>","X-Mailer":"git-send-email 2.11.0","X-BeenThere":"patchwork@lists.ozlabs.org","X-Mailman-Version":"2.1.23","Precedence":"list","List-Id":"Patchwork development <patchwork.lists.ozlabs.org>","List-Unsubscribe":"<https://lists.ozlabs.org/options/patchwork>,\n\t<mailto:patchwork-request@lists.ozlabs.org?subject=unsubscribe>","List-Archive":"<http://lists.ozlabs.org/pipermail/patchwork/>","List-Post":"<mailto:patchwork@lists.ozlabs.org>","List-Help":"<mailto:patchwork-request@lists.ozlabs.org?subject=help>","List-Subscribe":"<https://lists.ozlabs.org/listinfo/patchwork>,\n\t<mailto:patchwork-request@lists.ozlabs.org?subject=subscribe>","Cc":"jk@ozlabs.org","MIME-Version":"1.0","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"patchwork-bounces+incoming=patchwork.ozlabs.org@lists.ozlabs.org","Sender":"\"Patchwork\"\n\t<patchwork-bounces+incoming=patchwork.ozlabs.org@lists.ozlabs.org>"},"content":"If a user asks for a page beyond the range of pages, an EmptyPage\nexception is thrown. Catch this and clamp the page number\nappropriately.\n\nReported-by: Jeremy Kerr <jk@ozlabs.org>\nSigned-off-by: Daniel Axtens <dja@axtens.net>\n---\n patchwork/paginator.py | 6 ++++++\n 1 file changed, 6 insertions(+)","diff":"diff --git a/patchwork/paginator.py b/patchwork/paginator.py\nindex 609e908c2cbf..e4cf556ebd96 100644\n--- a/patchwork/paginator.py\n+++ b/patchwork/paginator.py\n@@ -55,6 +55,12 @@ class Paginator(paginator.Paginator):\n         except ValueError:\n             page_no = 1\n             self.current_page = self.page(page_no)\n+        except paginator.EmptyPage:\n+            if page_no < 1:\n+                page_no = 1\n+            else:\n+                page_no = self.num_pages\n+            self.current_page = self.page(page_no)\n \n         self.leading_set = self.trailing_set = []\n \n","prefixes":[]}