[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