From 75413ba57ce8a5be9046438e194371d8c0490b16 Mon Sep 17 00:00:00 2001
From: Erich Eckner <git@eckner.net>
Date: Wed, 19 Jun 2019 09:51:18 +0200
Subject: buildmaster/build-list.php: changing search criteria should not
 nullify the ordering options

---
 buildmaster/build-list.php | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/buildmaster/build-list.php b/buildmaster/build-list.php
index 968b357..e8b8b86 100644
--- a/buildmaster/build-list.php
+++ b/buildmaster/build-list.php
@@ -244,7 +244,7 @@ $columns = array(
 );
 
 if (!array_key_exists("sort", $_GET))
-  $_GET["sort"]="trials";
+  $_GET["sort"] = "trials";
 
 if (substr($_GET["sort"],0,1) == "-") {
   $direction = " DESC";
@@ -449,13 +449,12 @@ while($row = $result->fetch_assoc()) {
 
 print_header("List of Package Builds");
 
-// TODO: changing search criteria should not nullify the ordering options
 ?>
       <a href="https://buildmaster.archlinux32.org/build-logs/">build logs</a>
       <div id="pkglist-search" class="box filter-criteria">
         <h2>Package Build Search</h2>
         <form id="pkg-search" method="get" action="/buildmaster/build-list.php">
-          <p><input id="id_sort" name="sort" type="hidden" /></p>
+          <p><input id="id_sort" name="sort" type="hidden" value="<?php print $_GET["sort"]; ?>"/></p>
           <fieldset>
             <legend>Enter search criteria</legend>
 <?php
-- 
cgit v1.2.3