[whups] [patch] getTicketsByProperties() fix.

Jason M. Felice jfelice at cronosys.com
Thu Aug 14 07:57:25 PDT 2003


On Wed, Aug 13, 2003 at 09:06:59PM -0400, Chuck Hagenbuch wrote:
> This would actually break how groups are going to work (they're not there yet).
> Instead, the 'user:' should be added in the arguments to this function - where
> is it missing?

get_assigned_ticket_ids() api, attached patch fixes this.

Does this mean that if you assign a ticket to a group (which contains a
user), then search for tickets owned by that user, you won't find them?
Or is that code later to be written?

I did a user/group system for another huge system and I have some experience
to contribute <g>.  I'll make that a separate email to dev.

-- 
 Jason M. Felice
 Cronosys, LLC <http://www.cronosys.com/>
 216.221.4600 x302
-------------- next part --------------
Index: whups/lib/api.php
===================================================================
RCS file: /repository/whups/lib/api.php,v
retrieving revision 1.44
diff -u -r1.44 api.php
--- whups/lib/api.php	17 Jul 2003 16:42:23 -0000	1.44
+++ whups/lib/api.php	14 Aug 2003 14:47:00 -0000
@@ -141,7 +141,7 @@
     require_once dirname(__FILE__) . '/base.php';
     global $whups;
 
-    $info = array('owner' => Auth::getAuth(),
+    $info = array('owner' => 'user:' . Auth::getAuth(),
                   'nores' => true);
     $tickets = &$whups->getTicketsByProperties($info);
     if (is_a($tickets, 'PEAR_Error')) {


More information about the whups mailing list