[Rt-commit] rt branch, 3.8-trunk, updated. rt-3.8.6-89-ge221be4

Emmanuel Lacour elacour at bestpractical.com
Thu Nov 12 12:17:59 EST 2009


The branch, 3.8-trunk has been updated
       via  e221be4bb8168d36c6cf40f83f3d02d9d738273a (commit)
      from  59000984a6f50c1290a81d116c38a0a87534eae0 (commit)

Summary of changes:
 lib/RT/User_Overlay.pm |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

- Log -----------------------------------------------------------------
commit e221be4bb8168d36c6cf40f83f3d02d9d738273a
Author: Emmanuel Lacour <elacour at easter-eggs.com>
Date:   Thu Nov 12 18:16:48 2009 +0100

    Fix warning message
    
        It's not possible to concatenate strings with  conditions like "() ? :"

diff --git a/lib/RT/User_Overlay.pm b/lib/RT/User_Overlay.pm
index a7982e5..db3964c 100755
--- a/lib/RT/User_Overlay.pm
+++ b/lib/RT/User_Overlay.pm
@@ -1140,7 +1140,7 @@ sub SetDisabled {
     my $set_err = $self->PrincipalObj->SetDisabled($val);
     unless ($set_err) {
         $RT::Handle->Rollback();
-        $RT::Logger->warning("Couldn't ".($val == 1) ? "disable" : "enable"." user ".$self->PrincipalObj->Id);
+        $RT::Logger->warning(sprintf("Couldn't %s user %s", ($val == 1) ? "disable" : "enable", $self->PrincipalObj->Id));
         return (undef);
     }
     $self->_NewTransaction( Type => ($val == 1) ? "Disabled" : "Enabled" );

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


More information about the Rt-commit mailing list