Fix Departments pagination automatic filter application (#1078)

## Fixes issue
Fixes #1077 

## Description of Changes
Changes check if there are "filters" being applied to the list given in
the `departments` view. If there are no filters applied then we generate
a "next" link that does not automatically apply filters.

## Tests and linting
 - [x] This branch is up-to-date with the `develop` branch.
 - [x] `pytest` passes on my local development environment.
 - [x] `pre-commit` passes on my local development environment.

---------

Co-authored-by: sea-kelp <66500457+sea-kelp@users.noreply.github.com>

Author: Spraynard

OpenOversight/app/main/views.py

@@ -4,6 +4,7 @@
 from datetime import datetime
 from http import HTTPMethod, HTTPStatus
 from traceback import format_exc
+from typing import Optional
 
 from flask import (
     Response,
@@ -856,7 +857,7 @@ def redirect_list_officer(
     unique_internal_identifier=None,
     unit=None,
     current_job=None,
-    require_photo: bool = False,
+    require_photo: Optional[bool] = None,
 ):
     flash(FLASH_MSG_PERMANENT_REDIRECT)
     return redirect(
@@ -896,7 +897,7 @@ def list_officer(
     unique_internal_identifier=None,
     unit=None,
     current_job=None,
-    require_photo: bool = False,
+    require_photo: Optional[bool] = None,
 ):
     form = BrowseForm()
     form.rank.query = (

OpenOversight/tests/test_functional.py

@@ -114,16 +114,25 @@ def test_officer_browse_pagination(mockdata, browser, server_port):
     total = Officer.query.filter_by(department_id=AC_DEPT).count()
 
     # first page of results
-    browser.get(f"http://localhost:{server_port}/departments/{AC_DEPT}?page=1")
+    browser.get(
+        f"http://localhost:{server_port}/departments/{AC_DEPT}?page=1&gender=Not+Sure"
+    )
     wait_for_element(browser, By.TAG_NAME, "body")
     page_text = browser.find_element_by_tag_name("body").text
     expected = f"Showing 1-{current_app.config[KEY_OFFICERS_PER_PAGE]} of {total}"
     assert expected in page_text
 
+    # check that "Next" pagination link does not automatically add require_photo parameter
+    next_link = browser.find_elements(By.XPATH, '//li[@class="next"]/a')[
+        0
+    ].get_attribute("href")
+    assert "gender" in next_link
+    assert "require_photo" not in next_link
+
     # last page of results
     last_page_index = (total // current_app.config[KEY_OFFICERS_PER_PAGE]) + 1
     browser.get(
-        f"http://localhost:{server_port}/departments/{AC_DEPT}?page={last_page_index}"
+        f"http://localhost:{server_port}/departments/{AC_DEPT}?page={last_page_index}&gender=Not+Sure"
     )
     wait_for_element(browser, By.TAG_NAME, "body")
     page_text = browser.find_element_by_tag_name("body").text
@@ -135,6 +144,13 @@ def test_officer_browse_pagination(mockdata, browser, server_port):
     expected = f"Showing {start_of_page}-{total} of {total}"
     assert expected in page_text
 
+    # check that "Previous" pagination link does not automatically add require_photo parameter
+    previous_link = browser.find_elements(By.XPATH, '//li[@class="previous"]/a')[
+        0
+    ].get_attribute("href")
+    assert "gender" in previous_link
+    assert "require_photo" not in previous_link
+
 
 def test_find_officer_can_see_uii_question_for_depts_with_uiis(
     mockdata, browser, server_port