[Rt-commit] rt branch, 4.2/confirm-loaded-queue, repushed
Jim Brandt
jbrandt at bestpractical.com
Tue May 13 16:50:54 EDT 2014
The branch 4.2/confirm-loaded-queue was deleted and repushed:
was 36c6c285e87cf96d5b6a84695ef9881e2cf33e46
now a38b879928f021e6c28ee2149a0c9661a6eb07ff
1: 2c714ae = 1: 2c714ae Confirm loaded queue object before setting context object
2: 36c6c28 ! 2: 1e4bab3 Explicitly exclude 0 from queue lookup
@@ -18,8 +18,9 @@
- if ( defined $args{'Queue'} && ($args{'Queue'} =~ /\D/ || !$self->ContextObject) ) {
+ # Exclude 0 since it is a valid parameter, but will not load a valid queue
+ if ( defined $args{'Queue'}
-+ && $args{'Queue'} != 0
++ && ($args{'Queue'} =~ /^\d+$/ ? $args{'Queue'} != 0 : 1)
+ && ($args{'Queue'} =~ /\D/ || !$self->ContextObject) ) {
my $QueueObj = RT::Queue->new( $self->CurrentUser );
my ($ret, $msg) = $QueueObj->Load( $args{'Queue'} );
+
-: ------- > 3: a38b879 Tests for LoadByName with queue 0
More information about the rt-commit
mailing list