[Bps-public-commit] rt-extension-automaticassignment branch, master, updated. 6c18329df440adb1f86b5e4657c0c5d28a11c317

Shawn Moore shawn at bestpractical.com
Wed Jul 20 18:12:55 EDT 2016


The branch, master has been updated
       via  6c18329df440adb1f86b5e4657c0c5d28a11c317 (commit)
      from  053cfce659a4ef7316435fc34e5698c78f081a0b (commit)

Summary of changes:
 lib/RT/Extension/AutomaticAssignment/Chooser/Ownership.pm | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

- Log -----------------------------------------------------------------
commit 6c18329df440adb1f86b5e4657c0c5d28a11c317
Author: Shawn M Moore <shawn at bestpractical.com>
Date:   Wed Jul 20 22:09:04 2016 +0000

    Small fixes

diff --git a/lib/RT/Extension/AutomaticAssignment/Chooser/Ownership.pm b/lib/RT/Extension/AutomaticAssignment/Chooser/Ownership.pm
index d27cb24..09b3bb9 100644
--- a/lib/RT/Extension/AutomaticAssignment/Chooser/Ownership.pm
+++ b/lib/RT/Extension/AutomaticAssignment/Chooser/Ownership.pm
@@ -29,7 +29,7 @@ sub ChooseOwnerForTicket {
 
     if ($config->{ties}) {
         for my $statuses (@{ $config->{ties} }) {
-            if (ref($statuses) {
+            if (ref($statuses)) {
                 # multiple statuses at the same round (like new/open above)
                 # arbitrarily map to the first status in the list (new)
                 my $primary = $statuses->[0];
@@ -51,15 +51,15 @@ sub ChooseOwnerForTicket {
         # then simplify by selecting the user with the fewest
         # active-status tickets. we map everything to a single
         # "active" round
-        @statuses = $queue->ActiveStatusArray;
-        $primary_status{$_} = 'active' for @statuses;
+        @all_statuses = $ticket->QueueObj->ActiveStatusArray;
+        $primary_status{$_} = 'active' for @all_statuses;
         @rounds = 'active';
     }
 
     # limit to all the statuses we've seen thus far
     # we certainly don't want to look at all resolved tickets (unless
     # directed to)
-    for my $status (@statuses) {
+    for my $status (@all_statuses) {
         $tickets->LimitStatus(VALUE => $status);
     }
 

-----------------------------------------------------------------------


More information about the Bps-public-commit mailing list