diff --git a/templates/partials/packages/actions.html b/templates/partials/packages/actions.html index ae8cf141..016177c7 100644 --- a/templates/partials/packages/actions.html +++ b/templates/partials/packages/actions.html @@ -24,71 +24,73 @@ {{ "Search wiki" | tr }} - {% if not out_of_date %} + {% if request.user.is_authenticated() %} + {% if not out_of_date %} +
  • + + {{ "Flag package out-of-date" | tr }} + +
  • + {% else %} +
  • + + + {{ + "Flagged out-of-date (%s)" + | tr | format(date_strftime(pkgbase.OutOfDateTS, "%Y-%m-%d")) + }} + + +
  • + {% if request.user.has_credential(creds.PKGBASE_UNFLAG, approved=unflaggers) %} +
  • +
    + +
    +
  • + {% endif %} + {% endif %}
  • - - {{ "Flag package out-of-date" | tr }} - -
  • - {% else %} -
  • - - - {{ - "Flagged out-of-date (%s)" - | tr | format(date_strftime(pkgbase.OutOfDateTS, "%Y-%m-%d")) - }} - - -
  • - {% if request.user.has_credential(creds.PKGBASE_UNFLAG, approved=unflaggers) %} -
  • -
    - + {% if not voted %} + +
    + {% else %} +
    + +
    + {% endif %} +
  • +
  • + {% if notified %} +
    + +
    + {% else %} +
    + +
    + {% endif %}
  • - {% endif %} {% endif %} -
  • - {% if not voted %} -
    - -
    - {% else %} -
    - -
    - {% endif %} -
  • -
  • - {% if notified %} -
    - -
    - {% else %} -
    - -
    - {% endif %} -
  • {% if request.user.has_credential(creds.PKGBASE_EDIT_COMAINTAINERS, approved=[pkgbase.Maintainer]) %}
  • @@ -111,11 +113,13 @@
  • {% endif %} {% endif %} -
  • - - {{ "Submit Request" | tr }} - -
  • + {% if request.user.is_authenticated() %} +
  • + + {{ "Submit Request" | tr }} + +
  • + {% endif %} {% if request.user.has_credential(creds.PKGBASE_DELETE) %}
  • diff --git a/test/test_packages_routes.py b/test/test_packages_routes.py index e1c989da..4bf7a5ae 100644 --- a/test/test_packages_routes.py +++ b/test/test_packages_routes.py @@ -507,7 +507,9 @@ def test_package_requests_display( client: TestClient, user: User, package: Package, pkgreq: PackageRequest ): # Test that a single request displays "1 pending request". + cookies = {"AURSID": user.login(Request(), "testPassword")} with client as request: + request.cookies = cookies resp = request.get(package_endpoint(package)) assert resp.status_code == int(HTTPStatus.OK) @@ -530,6 +532,7 @@ def test_package_requests_display( # Test that a two requests display "2 pending requests". with client as request: + request.cookies = cookies resp = request.get(package_endpoint(package)) assert resp.status_code == int(HTTPStatus.OK)