Index: modules/project_issue/issue.inc
===================================================================
RCS file: /cvs/drupal-contrib/contributions/modules/project_issue/issue.inc,v
retrieving revision 1.172.2.22.2.2
diff -u -p -r1.172.2.22.2.2 issue.inc
--- modules/project_issue/issue.inc	12 Nov 2006 09:22:15 -0000	1.172.2.22.2.2
+++ modules/project_issue/issue.inc	16 Nov 2006 00:52:51 -0000
@@ -1,6 +1,6 @@
 <?php
 // $Id: issue.inc,v 1.172.2.22.2.2 2006/11/12 09:22:15 dww Exp $
-// $Name:  $
+// $Name: DRUPAL-4-7--2 $
 
 function project_issue_page() {
   switch ($_POST['op'] ? $_POST['op'] : arg(2)) {
@@ -66,31 +66,18 @@ function project_issue_page() {
 /**
  * Decides what to do with search page request.
  *
- * - if this is not a POST, return project_project_retrieve() after
- *   dealing with $_SESSION['project_issue_search']
+ * - if this is not a POST, just return project_project_retrieve()
  * - if this is a POST, dispatch to handler function based on form id
  *
  * @param $url_arg
  *   Value returned from arg(2)
  */
 function project_issue_build_form_url($url_arg) {
-  $adv_filters = array('text', 'attachment', 'components', 'submitted', 'assigned', 'participated', 
-                       'states', 'categories', 'priorities');
+  $adv_filters = array('text', 'attachment', 'components', 'submitted', 'assigned', 'participated', 'states', 'categories', 'priorities');
   $filters = array_slice($adv_filters, 5);
 
-  // this isn't a POST, so check session and return project
+  // this isn't a POST, just return project
   if (empty($_POST)) {
-    if (isset($_SESSION['project_issue_search'])) {
-      // put session vars in $_GET for project_issue_query_parse 
-      foreach ($filters as $filter) {
-        if (isset($_SESSION['project_issue_search'][$filter])) {
-          $_GET[$filter] = $_SESSION['project_issue_search'][$filter];
-        }
-      }
-
-      // clean up after ourselves
-      unset($_SESSION['project_issue_search']);
-    }
     return project_project_retrieve($url_arg);
   }
   
@@ -130,30 +117,42 @@ function project_issue_quick_search($url
 
     // put $_POST['edit'] where pager will see them 
     foreach ($filters as $filter) {
-      if (isset($_POST['edit'][$filter])) {
+      if (!empty($_POST['edit'][$filter])) {
         $_REQUEST[$filter] = $_POST['edit'][$filter];
       }
     }
 
     // unset $_REQUEST['edit'] so the pager links aren't full of cruft
     unset($_REQUEST['edit']);
+    unset($_REQUEST['op']);
     return isset($project) ? $project : NULL;
   }
 
   // redirecting, so put filters in session
   foreach ($filters as $filter) {
-    if (isset($_POST['edit'][$filter])) {
-      $_SESSION['project_issue_search'][$filter] = $_POST['edit'][$filter];
+    if (!empty($_POST['edit'][$filter])) {
+      $issue_filters[$filter] = $_POST['edit'][$filter];
     }
   }
-  $_REQUEST['destination'] = $destination;
-  drupal_goto();
+  drupal_goto($destination, drupal_query_string_encode($issue_filters));
 }
 
 /**
  * handles the advanced search form
  */
 function project_issue_advanced_search($url_arg, $filters) {
+  // Figure out what issue filters are in place for the query we're showing
+  foreach ($filters as $filter) {
+    if (!empty($_POST['edit'][$filter])) {
+      if (is_array($_POST['edit'][$filter])) {
+        $issue_filters[$filter] = implode(',', $_POST['edit'][$filter]);
+      }
+      else {
+        $issue_filters[$filter] = $_POST['edit'][$filter];
+      }
+    }
+  }
+
   // do we have just one project to search for?
   if (isset($_POST['edit']['projects']) && count($_POST['edit']['projects']) == 1) {
     // try to find the project, and 404 if not found
@@ -161,24 +160,19 @@ function project_issue_advanced_search($
       drupal_not_found();
       exit;
     }
-
-    // redirecting, so put filters in session
-    foreach ($filters as $filter) {
-      if (isset($_POST['edit'][$filter])) {
-        if (is_array($_POST['edit'][$filter])) {
-          $_SESSION['project_issue_search'][$filter] = implode(',', $_POST['edit'][$filter]);
-        }
-        else {
-          $_SESSION['project_issue_search'][$filter] = $_POST['edit'][$filter];
-        }
-      }
-    }
-    $_REQUEST['destination'] = 'project/issues/'. $project->uri;
-    drupal_goto();
+    $destination = 'project/issues/'. $project->uri;
+    drupal_goto($destination, drupal_query_string_encode($issue_filters));
   }
 
+  // Put the issue filters where the pager links can find them.
+  foreach ($issue_filters as $name => $val) {
+    $_REQUEST[$name] = $val;
+  }
   // unset $_REQUEST['edit'] so the pager links aren't full of cruft
   unset($_REQUEST['edit']);
+  // Get rid of the silly 'op', since it's not a filter.
+  unset($_REQUEST['op']);
+
   return project_project_retrieve($url_arg);
 }